Review Request 120160: KMessageWidget: fix handling of rapid succession of animatedHide+animatedShow calls

Fabio D'Urso fabiodurso at hotmail.it
Fri Sep 12 12:16:50 UTC 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/120160/
-----------------------------------------------------------

Review request for KDE Frameworks, Christoph Feck and Aurélien Gâteau.


Repository: kwidgetsaddons


Description
-------

Hi, I've found that if you call animatedShow() immediatly after animatedHide(), without waiting for the hide animation to finish first, the hide animation goes on and, in the end, the message widget becomes hidden.

The attached patch adds checks in animatedShow and animatedHide so that, if an opposite animation is currently running, they just reverse it.


Diffs
-----

  autotests/kmessagewidgetautotest.h 062e2b3 
  autotests/kmessagewidgetautotest.cpp f46bab0 
  src/kmessagewidget.cpp bc7ba32 
  tests/kmessagewidgettest.cpp f621b5a 

Diff: https://git.reviewboard.kde.org/r/120160/diff/


Testing
-------

I've added buttons to invoke animatedShow and animatedHide in tests/kmessagewidget.cpp and used them to test the behavior.

I've also added an autotest that checks the final height and isVisible() value, both after single animatedShow/animatedHide calls and after animatedShow+animatedHide and animatedHide+animatedShow.


Thanks,

Fabio D'Urso

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140912/0518a018/attachment.html>


More information about the Kde-frameworks-devel mailing list