broulik created this revision. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel.
REVISION SUMMARY We've been having the configure button on notifications for a long time but so far it would only ever invoke KNotification configuration UI with no way for an application to provide its own, more versatile or extensive, UI. When an action named "settings" is present in the notification the "configure" button will instead invoke this action. TEST PLAN Before: F3677860: Screenshot_20170413_113924.png <https://phabricator.kde.org/F3677860> After: (don't mind the changed text, I wanted to text how Chrome handles line breaks in its notifications) F3677861: Screenshot_20170413_120654.png <https://phabricator.kde.org/F3677861> Yes, this seems proprietary to Chrome which sends this along for its soon-to-be-released native notifications (above screenshot is from Chrome 59 with native notifications flag enabled) *but* I have wanted to have something like this for a long time and it's a good opportunity to introduce a de-facto standard for that now. I'll look into adding support for that in KNotifications, similar to the default action that was recently added. REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D5424 AFFECTED FILES applets/notifications/package/contents/ui/NotificationDelegate.qml applets/notifications/package/contents/ui/NotificationItem.qml applets/notifications/package/contents/ui/NotificationPopup.qml applets/notifications/package/contents/ui/Notifications.qml To: broulik, #plasma, #vdg Cc: plasma-devel, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol
