Review Request 125178: Don't fade animate icon in plasma tooltips

[David Rosca]

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

(Updated Sept. 14, 2015, 9:12 a.m.)


Status
------

This change has been marked as submitted.


Review request for Plasma.


Changes
-------

Submitted with commit 4b4461d76b7999ce309cd9b35c2e510d8a5aa038 by David Rosca to branch master.


Repository: plasma-framework


Description
-------

PlasmaCore.IconItem has a fade animation when changing source.
This makes the plasma tooltip show old icon and then fade away to new icon when changing tooltip contents. It may be desirable when the tooltip is already visible and is changig its position and content to the new item (even though it still looks weird - text is changed immediately and icon only after a delay), but it is wrong when showing new tooltip.


Diffs
-----

  src/declarativeimports/core/private/DefaultToolTip.qml 9c4a2cd 

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


Testing
-------

This weirdness is gone \o/


Thanks,

David Rosca

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150914/5231b0f2/attachment.html>