Don't rely on visibility to block the pause menu

Author: StyledStrike

lua/custom_chat/client/main.lua

@@ -453,6 +453,7 @@ local function CustomChat_Open( pcalled )
     CustomChat.frame:SetKeyboardInputEnabled( true )
     CustomChat.frame:OpenChat()
 
+    CustomChat.isOpen = true
     CustomChat.SetTyping( true )
 
     -- Make sure the gamemode and other addons know we are chatting
@@ -465,6 +466,8 @@ local function CustomChat_Close()
     CustomChat.frame:CloseChat()
     CustomChat.frame:SetMouseInputEnabled( false )
     CustomChat.frame:SetKeyboardInputEnabled( false )
+
+    CustomChat.isOpen = false
     CustomChat.SetTyping( false )
 
     gui.EnableScreenClicker( false )
@@ -516,18 +519,26 @@ local function CustomChat_HUDShouldDraw( name )
 end
 
 local function CustomChat_Think()
-    if not CustomChat.frame then return end
+    local frame = CustomChat.frame
+    if not frame then return end
 
-    if not gui.IsGameUIVisible() and not CustomChat.frame:IsVisible() then
-        CustomChat.frame:SetVisible( true )
+    if gui.IsGameUIVisible() then
+        if frame:IsVisible() then
+            -- Close and completely hide the chat
+            -- while the pause menu is visible
+            chat.Close()
+            frame:SetVisible( false )
+        end
+
+    elseif not frame:IsVisible() then
+        -- Make the chat visible otherwise
+        frame:SetVisible( true )
     end
 end
 
 local function CustomChat_OnPauseMenuShow()
-    if not CustomChat.frame then return end
-
-    if CustomChat.frame:IsVisible() then
-        CustomChat.frame:SetVisible( false )
+    if CustomChat.frame and CustomChat.isOpen then
+        chat.Close()
 
         return false
     end