Thread Tools Display Modes
07/08/14, 01:31 AM   #1
Wisshard
Join Date: Jul 2014
Posts: 4
[Request] ChatMultiWindow Update

Hey,

With five guild chats, a zone chat, occasionally a group chat and tells, it is quite hard to keep track of the chats in ESO - especially when you only have one chat window.

ChatMultiWindow provides the option to create multiple chat windows, which makes the social life in ESO a lot easier and comforting, but unfortunately it has yet to be updated to patch 2.3, and so I would greatly appreciate if some kind soul could take it upon themselves to get it up and running again.

Thanks in advance.
  Reply With Quote
07/08/14, 03:31 AM   #2
CrazyDutchGuy
 
CrazyDutchGuy's Avatar
AddOn Author - Click to view addons
Join Date: Apr 2014
Posts: 89
Personally for a few of my addons i think it is usefull. I'll see if i can get it back to working state. No promises though as i never used this addon yet :P
  Reply With Quote
07/08/14, 04:01 AM   #3
Wisshard
Join Date: Jul 2014
Posts: 4
Great! Thank you, I hope you'll be able to manage it.
  Reply With Quote
07/08/14, 05:09 AM   #4
mra4nii
AddOn Author - Click to view addons
Join Date: Apr 2014
Posts: 42
I use older, original version and i can say that it works without any problem. Need to update just API version in manifest.
I guess, patched version should work fine too.
  Reply With Quote
07/08/14, 05:16 AM   #5
Garkin
 
Garkin's Avatar
AddOn Author - Click to view addons
Join Date: Mar 2014
Posts: 832
I have just a quick look to the addon code and there is virtually nothing to fix, it should be working just fine. I will check it in game.

Only thing I do not like is replacing of the original function of CHAT_SYSTEM instead of hooking it.
Instead of the last function in the code. I'd rather use this hook (but it is matter of personal preference):
lua Code:
  1. -- Hook the original function so we can fix the anchors
  2. local TransferWindow = CHAT_SYSTEM.TransferWindow
  3. function CHAT_SYSTEM.TransferWindow(...)
  4.     -- First, do what the original function did...
  5.     TransferWindow(...)
  6.  
  7.     -- Second, fix the anchors so the text is visible
  8.     if container ~= CHAT_SYSTEM.containers[1] then
  9.         ChatMultiWindow.FixContainerAnchor(container)
  10.     end
  11. end

Last edited by Garkin : 07/08/14 at 07:59 AM.
  Reply With Quote
07/08/14, 05:18 AM   #6
Wisshard
Join Date: Jul 2014
Posts: 4
The current available version of ChatMultiWindow hasn't worked for me at all since patch 2.3 unfortunately.
  Reply With Quote
07/08/14, 05:24 AM   #7
CrazyDutchGuy
 
CrazyDutchGuy's Avatar
AddOn Author - Click to view addons
Join Date: Apr 2014
Posts: 89
I'll take a peek and see what i can do, as soon i am done with my normal payed job :P
  Reply With Quote
07/08/14, 05:30 AM   #8
Wisshard
Join Date: Jul 2014
Posts: 4
Alright, I just checked again (I've had it disabled since it stopped working post-patch 2.3) and it now appears to be working for me. Must've been some compatibility issue with with some other addon I had at the time patch 2.3 came around.

I apologize for calling attention to something that there was no problem with.

While we're here and talking about ChatMultiWindows though, I do wonder if it's possibly to add some features to it. Specifically, the option to lock any additional windows you create in place, and placing the chat windows behind trade windows.
  Reply With Quote
07/08/14, 05:12 PM   #9
Garkin
 
Garkin's Avatar
AddOn Author - Click to view addons
Join Date: Mar 2014
Posts: 832
Changes in ChatMultiWindow.txt:
- updated API version
- version bump

Warning: Spoiler


Changes in ChatMultiWindow.lua:
- version bump
- replaced function DestroyContainer(...) with CHAT_SYSTEM.DestroyContiner(...)
- correctly unregister EVENT_ADD_ON_LOADED event
- hook CHAT_SYSTEM:TransferWindow(...) instead of replacing (it's not functionality change)

Warning: Spoiler
  Reply With Quote
07/09/14, 01:20 PM   #10
CrazyDutchGuy
 
CrazyDutchGuy's Avatar
AddOn Author - Click to view addons
Join Date: Apr 2014
Posts: 89
nice work Garkin, You got more time then me too spend Can you submit it as a patch for the original addon ?
  Reply With Quote
07/09/14, 02:10 PM   #11
Garkin
 
Garkin's Avatar
AddOn Author - Click to view addons
Join Date: Mar 2014
Posts: 832
Originally Posted by CrazyDutchGuy View Post
nice work Garkin, You got more time then me too spend Can you submit it as a patch for the original addon ?
Uploaded: http://www.esoui.com/downloads/info6...atch1.2.3.html
  Reply With Quote

ESOUI » AddOns » AddOn Search/Requests » [Request] ChatMultiWindow Update


Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

vB code is On
Smilies are On
[IMG] code is On
HTML code is Off