D22760: [Notifications] Keep new delegate hidden until room has been cleared
Kai Uwe Broulik
noreply at phabricator.kde.org
Fri Jul 26 10:22:54 BST 2019
broulik created this revision.
broulik added a reviewer: Plasma.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
broulik requested review of this revision.
REVISION SUMMARY
When a new notification comes in, we first displace the existing ones to make room and then animate in the new one.
However, due to the `PauseAnimation` the item appears with its full opacity ontop of the existing notifications that move out of the way.
Add a `PropertyAction` to ensure the item has an `opacity` of zero initially.
TEST PLAN
5.16 branch
- Added multiple notifications to the popup, had them nicely appear instead of awkwardly overlapping visually
REPOSITORY
R120 Plasma Workspace
REVISION DETAIL
https://phabricator.kde.org/D22760
AFFECTED FILES
applets/notifications/package/contents/ui/FullRepresentation.qml
To: broulik, #plasma
Cc: plasma-devel, LeGast00n, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190726/802aeafd/attachment.html>
More information about the Plasma-devel
mailing list