Page 1 of 1

chat test ptr

Posted: Fri Nov 16, 2018 11:15 pm
by fancypearl
Patch 3.1.9

- Legacy chat re-introduced into the clients - Credit goes to OpenUL project

(tested alone for now, gui seems to work, chat shows in chat.)

Testing normal chat function.
chat works, emote works, shout works, bug works, report works, rp works, whispering to emphant works.

classic chat,
chat works, emote works, shout works, bug works, report works, rp works,whispering to emphant works.

Re: chat test ptr

Posted: Sat Nov 17, 2018 12:18 am
by fancypearl
verified with gm client, whispered back and forth confirmed working, with chat, shout, emote

Re: chat test ptr

Posted: Sat Nov 17, 2018 9:40 am
by Koi-JorKyrin
Since this also involved modifications around the window positioning code on both resolutions, I tested it in 800x600. The chat box placement looks correct and all chat functions are working. Did the same with pmare client. All normal preset chat maren features work as expected. One thing to note is I also had to add the classicChatPos to the pmare build otherwise it would fail to build.

I let the OpenUL folks know that this is what I had to do to get it to compile. They may have additional input on this to change. Someone will probably have to post that as an issue as I just realized it was posted on a closed merge request on OpenUL side:
https://github.com/openunderlight/ULClient/pull/5


cChat.cpp: Added after #else on line 66

Code: Select all

// position for chat display area - no ads 
#ifndef PMARE
const struct window_pos_t classicChatPos[MAX_RESOLUTIONS] =
{ { 0, 300, 480, 180 },{ 0, 375, 600, 225 },{ 0, 480, 768, 288 } };

const struct window_pos_t chatPos[MAX_RESOLUTIONS] = 
{ { 0, 300, 480, 155 }, { 0, 375, 600, 194 }, { 0, 480, 768, 253 } };

const struct window_pos_t entryPos[MAX_RESOLUTIONS] =
{ { 0, 455, 480, 25 }, { 0, 570, 600, 35 }, { 0, 733, 768, 40 } };
#else
const struct window_pos_t classicChatPos[MAX_RESOLUTIONS] =
{ { 0, 300, 480, 180 },{ 0, 375, 600, 225 },{ 0, 480, 768, 288 } };

const struct window_pos_t chatPos[MAX_RESOLUTIONS] =
{ { 0, 300, 480, 180 },{ 0, 375, 600, 225 },{ 0, 480, 768, 288 } };
#endif

While the option for classic chat checkbox is not present on pmare login screen and does not require it, the build process itself certainly needed the above L66 section for pmare to build successfully. I did notice some strange pmare chat window positioning as I drag the window around and attempt to maren chat at each window location. Depending where I drag the pame game window, the chat box is either on the top of the view port, or other times it is at the bottom. This happens on production side as well, so I am not concerned with it, or believe it to be due to this L66 addition.

Anyways the pmare chat box issue isn't crucial to be fixed now, but needed to point this out.

As always, thank you for the help testing fancypearl !

Re: chat test ptr

Posted: Sun Nov 18, 2018 1:38 pm
by Tary
Thanks, sidas pointed out the pmare issue the other day. Will merge into master