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

(Updated Jan. 18, 2014, 5:47 p.m.)


Review request for KDE Frameworks, Albert Astals Cid and Aurélien Gâteau.


Changes
-------

add showAnimationFinished();


Repository: kwidgetsaddons


Description (updated)
-------

This patch adds the signal KMessageWidget::hideAnimationFinished(), indicating 
that the hide animation is finished.

This is required by Kate Part's passive notification system, since the next 
pending message is shown only after the hide animation is finished.
Right now, Kate Part launches a timer of 500ms (these 500 ms are used by 
kmessagewidget's animatedHide()) to estimate when the animation is finished.

This signal cannot be replaced by e.g. deriving KMessageWidget, since 
QHideEvent is also sent in case of the parent widget is hidden.

The same holds for animatedShow(): Emit signal showAnimationFinished() when the 
show animation is finished.


Diffs (updated)
-----

  src/kmessagewidget.h a17bccf 
  src/kmessagewidget.cpp deb82a3 

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


Testing
-------

In Kate Part, not commited so far, though.


Thanks,

Dominik Haumann

_______________________________________________
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Reply via email to