----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122859/ -----------------------------------------------------------
(Updated March 10, 2016, 1:02 p.m.) Status ------ This change has been marked as submitted. Review request for Plasma. Changes ------- Submitted with commit 344d04e2306ca2fdaf17269bc218f4fad065cbbf by Kai Uwe Broulik to branch master. Repository: plasma-framework Description ------- We have a lot of reusable singletons that are just hidden when unneeded (tooltip, osd). IconItem, however, will always fade from the previous state, even if the previous action happened minutes ago. This patch makes it track its visibility and skip the fade-and-wait dance when it just became visible. It also removes a visible false call in the tooltip which I didn't know what it was for. (With it in place, the IconItem always becomes visible when moving between tooltip areas, breaking the animation altogether). @Eike: Could you check whether this makes it more viable for Kicker? Diffs ----- autotests/iconitemtest.h c7ba89f autotests/iconitemtest.cpp ae825a9 src/declarativeimports/core/iconitem.h 9613354 src/declarativeimports/core/iconitem.cpp 6c23a7e Diff: https://git.reviewboard.kde.org/r/122859/diff/ Testing ------- Moving between tray icons - icon fades, moving reaaaally rapidly causes it not to load any icon until you halt (dunno if that happened before but doesn't seem too bad) Hovering tray icon, leaving, waiting, hovering another one - icon does not fade, is there right away Notifications also don't fade from the previously issued notification icon Thanks, Kai Uwe Broulik
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel