From 125e1c86a3fb24c990d24f8a171eb05bb18d04ea Mon Sep 17 00:00:00 2001
From: Connor Turland <connorturland@gmail.com>
Date: Mon, 19 Dec 2016 14:57:53 -0500
Subject: [PATCH] chat box opens and closes properly

---
 frontend/src/Metamaps/Views/ChatView.js  | 33 +++++++-----------------
 frontend/src/components/MapChat/index.js | 19 +++++++++++---
 2 files changed, 26 insertions(+), 26 deletions(-)

diff --git a/frontend/src/Metamaps/Views/ChatView.js b/frontend/src/Metamaps/Views/ChatView.js
index a3398ed6..e961ed69 100644
--- a/frontend/src/Metamaps/Views/ChatView.js
+++ b/frontend/src/Metamaps/Views/ChatView.js
@@ -16,13 +16,13 @@ import MapChat from '../../components/MapChat'
 const linker = new Autolinker({ newWindow: true, truncate: 50, email: false, phone: false })
 
 const ChatView = {
+  isOpen: false,
   init: function(messages, mapper, room, opts = {}) {
     const self = ChatView
     self.room = room
     self.mapper = mapper
     self.messages = messages
 
-    self.isOpen = false
     self.alertSound = true // whether to play sounds on arrival of new messages or not
     self.cursorsShowing = true
     self.videosShowing = true
@@ -44,13 +44,13 @@ const ChatView = {
   render: () => {
     const self = ChatView
     ReactDOM.render(React.createElement(MapChat, {
-      isOpen: self.isOpen,
+      onOpen: self.onOpen,
+      onClose: self.onClose,
       leaveCall: Realtime.leaveCall,
       joinCall: Realtime.joinCall,
       participants: self.participants,
       messages: self.messages.models.map(m => m.attributes),
       unreadMessages: self.unreadMessages,
-      buttonClick: self.buttonClick,
       videoToggleClick: self.videoToggleClick,
       cursorToggleClick: self.cursorToggleClick,
       soundToggleClick: self.soundToggleClick,
@@ -62,13 +62,13 @@ const ChatView = {
       handleInputMessage: self.handleInputMessage
     }), document.getElementById('chat-box-wrapper'))
   },
-  show: () => {
-    ChatView.show = true
-    ChatView.render()
+  onOpen: () => {
+    ChatView.isOpen = true
+    $(document).trigger(ChatView.events.openTray)
   },
-  hide: () => {
-    ChatView.show = false
-    ChatView.render()
+  onClose: () => {
+    ChatView.isOpen = false
+    $(document).trigger(ChatView.events.closeTray)
   },
   addParticipant: participant => {
     // TODO this should probably be using a Backbone collection????
@@ -107,6 +107,7 @@ const ChatView = {
 
     self.messages.push(m)
     // TODO what is scrollMessages?
+    // scrollMessages scrolls to the bottom of the div when there's new messages“
     // if (!isInitial) self.scrollMessages(200)
     self.render()
 
@@ -137,30 +138,16 @@ const ChatView = {
     // TODO refactor
     // this.$participants.find('.participant-' + id).removeClass('pending')
   },
-  buttonClick: () => {
-    const self = ChatView
-    if (self.isOpen) self.close()
-    else if (!self.isOpen) self.open()
-  },
   close: () => {
-    this.isOpen = false
-
     // TODO how to do focus with react
     // this.$messageInput.blur()
-
-    $(document).trigger(ChatView.events.closeTray)
-    ChatView.render()
   },
   open: () => {
     ChatView.unreadMessages = 0
-    this.isOpen = true
-
     // TODO how to do focus with react
     // this.$messageInput.focus()
     // TODO reimplement scrollMessages
     // this.scrollMessages(0)
-
-    $(document).trigger(ChatView.events.openTray)
     ChatView.render()
   },
   videoToggleClick: function() {
diff --git a/frontend/src/components/MapChat/index.js b/frontend/src/components/MapChat/index.js
index 30be2f82..c095be5e 100644
--- a/frontend/src/components/MapChat/index.js
+++ b/frontend/src/components/MapChat/index.js
@@ -8,10 +8,22 @@ class MapChat extends Component {
     super(props)
 
     this.state = {
+      open: false,
       messageText: ''
     }
   }
 
+  toggleDrawer = () => {
+    if (this.state.open) {
+      this.setState({open: false})
+      this.props.onClose()
+    }
+    else if (!this.state.open) {
+      this.setState({open: true})
+      this.props.onOpen()
+    }
+  }
+
   handleChange = key => e => {
     this.setState({
       [key]: e.target.value
@@ -28,7 +40,7 @@ class MapChat extends Component {
   }
 
   render = () => {
-    const rightOffset = this.props.isOpen ? '-300px' : '0'
+    const rightOffset = this.state.open ? '0' : '-300px'
     const { videosShowing, cursorsShowing, alertSound } = this.props
     return (
       <div className="chat-box"
@@ -57,7 +69,7 @@ class MapChat extends Component {
           CHAT
           <div className={`sound-toggle ${alertSound ? 'active' : ''}`}></div>
         </div>
-        <div className="chat-button">
+        <div className="chat-button" onClick={this.toggleDrawer}>
           <div className="tooltips">Chat</div>
           <Unread count={this.props.unreadMessages} />
         </div>
@@ -80,7 +92,8 @@ class MapChat extends Component {
 }
 
 MapChat.propTypes = {
-  isOpen: PropTypes.bool,
+  onOpen: PropTypes.func,
+  onClose: PropTypes.func,
   leaveCall: PropTypes.func,
   joinCall: PropTypes.func,
   participants: PropTypes.arrayOf(PropTypes.shape({