The change back to a MouseEventListener broke notification actions because its onClicked handler would fire before the action buttons closing the notification before it had a chance to trigger the action.
Details
Details
- Reviewers
mart - Group Reviewers
Plasma - Commits
- R120:e772deb6468b: [Notifications] Unbreak notification actions
R871:e772deb6468b: [Notifications] Unbreak notification actions
I'm not sure we can rely on the order of events but I suppose MEL filters from parent to child?
In any way we should avoid the close call after triggering an action.
Diff Detail
Diff Detail
- Repository
- R120 Plasma Workspace
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Comment Actions
Ok, no, we need a more sophisticated solution... the "configure" and "close" button are also broken.