D20266: Add new notification plasmoid

broulik closed this revision. broulik added a comment. All merged https://cgit.kde.org/plasma-workspace.git/commit/?id=3c52fc3bc05e0f351d4998ad54edae1a4e21a043 REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D20266 To: broulik, #plasma, #vdg, mart Cc: mart,

D20266: Add new notification plasmoid

mart accepted this revision. mart added inline comments. This revision is now accepted and ready to land. INLINE COMMENTS > CompactRepresentation.qml:101 > +font.pointSize: -1 > +// FIXME fontSizeMode is awful but FontMetrics also doesn't cut > it > +font.pixe

D20266: Add new notification plasmoid

broulik updated this revision to Diff 56059. broulik added a comment. - Fix diff screwup REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D20266?vs=56057&id=56059 REVISION DETAIL https://phabricator.kde.org/D20266 AFFECTED FILES applets/notifica

D20266: Add new notification plasmoid

broulik updated this revision to Diff 56057. broulik added a comment. - Grouping fixes, unread notifications are automatically expanded now and collapsing a group scrolls to the top of it - Added "4 hours" and "until disabled" do not disturb mode - Job progress bar is hidden when completed

D20266: Add new notification plasmoid

ngraham added a comment. In D20266#445888 , @broulik wrote: > > It seems like there's no Do Not Disturb mode that stays on until manually turned off though? > > Yeah. I can add that, I guess. What should that menu entry look like? Som

D20266: Add new notification plasmoid

broulik added a comment. > It seems like there's no Do Not Disturb mode that stays on until manually turned off though? Yeah. I can add that, I guess. What should that menu entry look like? REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D20266 To: brou

D20266: Add new notification plasmoid

GB_2 added a comment. I made a better timeout indicator that you can use: F6755221: TimeoutIndicator.qml This is how it looks like: F6755222: Circular Timeout.mp4 REPOSITORY R120 Plasma Workspace REVISION DETA

D20266: Add new notification plasmoid

abetts added a comment. In D20266#443993 , @ngraham wrote: > This UI is fantastic. A huge improvement over the status quo. > > It seems like there's no Do Not Disturb mode that stays on until manually turned off though? I was thinking

D20266: Add new notification plasmoid

ngraham added a comment. This UI is fantastic. A huge improvement over the status quo. It seems like there's no Do Not Disturb mode that stays on until manually turned off though? REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D20266 To: broulik, #plas

D20266: Add new notification plasmoid

GB_2 added a comment. Amazing! INLINE COMMENTS > FullRepresentation.qml:247 > +opacity: 0.6 > +text: list.count === 0 ? i18n("No unread notifications.") : > i18n("Notifications") > +} Please remove the dot to make it consistent with other Plasmoids. REPOSITO

D20266: Add new notification plasmoid

broulik created this revision. broulik added reviewers: Plasma, VDG. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. broulik requested review of this revision. REVISION SUMMARY This completely rewrites the notification plasmoid using `libnotificationmanager` The po