Skip to content
This repository has been archived by the owner on Aug 29, 2020. It is now read-only.

Fix the chat fade bug... #85

Merged
merged 2 commits into from
Jun 3, 2014
Merged

Fix the chat fade bug... #85

merged 2 commits into from
Jun 3, 2014

Conversation

Hoikas
Copy link
Member

@Hoikas Hoikas commented Jun 2, 2014

... This fixes the bug that many users complained about regarding the chat fading out while they were typing. The issue was that we were inappropriately resetting the fade status anytime the chat buffer changed (someone talked)--thus, fade out. I couldn't throw that away entirely because scrolling the chat buffer should reset the fade state. So, I made a smarter ResetFadeState that checks the chat input state as well. It decreases the general fiddly-ness of managing the fade state.

Also, I made the chat fade smoother because it looked nasty before.

@Deledrius
Copy link
Member

Haven't tested this, but it looks sane from here. 👍

Here's the rub: we were doing entirely too much manual chugging and
plugging on the chat fade status, so some corner-cases slipped through. In this case, we absolutely DO NOT WANT to start the fade timer if the user has entered any chat (self.chatMgr.isChatting). We were doing just that any time the chat
history display was modified (oops!).

See also:
- http://forum.guildofwriters.org/viewtopic.php?f=117&t=6076
... The old one was rough like sandpaper and looked like crap. This is
much nicer.
@cwalther
Copy link
Contributor

cwalther commented Jun 2, 2014

Kae mentioned some oddity with text shadows and fading. Has anyone reproduced and/or investigated that?

@Hoikas
Copy link
Member Author

Hoikas commented Jun 2, 2014

Reproduced consistently, but that bug is beyond the scope of this PR. The fading is done by changing the GUI control's alpha in python and the shadows don't respect this. I assume it's related to the premultiplied alpha stuff?

@cwalther
Copy link
Contributor

cwalther commented Jun 2, 2014

I’ll take a look when I get around to it (not before the weekend). It sounds very weird because characters and shadows are both just pixels in the same texture, there is no reason why one should be treated differently than the other.

@Hoikas
Copy link
Member Author

Hoikas commented Jun 2, 2014

That was my thought as well. Fortunately, this PR makes fixing that issue fairly low-priority. The only observable instance of it now is when the player lists are fading (fortunately, at the end of the animation, that UI item is disabled altogether, so it's not a big deal).

@Mystler
Copy link
Contributor

Mystler commented Jun 3, 2014

Seems to work fine. 👍
The shadows are indeed not fading, but it's not really annoying and probably not that important for now.

Deledrius added a commit that referenced this pull request Jun 3, 2014
@Deledrius Deledrius merged commit 277bc25 into H-uru:master Jun 3, 2014
@Hoikas Hoikas deleted the chatfade branch June 3, 2014 21:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants