D25851: [Notifications] Add quick reply feature

Kai Uwe Broulik noreply at phabricator.kde.org
Tue Dec 10 14:59:25 GMT 2019


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 adds a quick reply feature with a text field inline in the notification popup.
  An action named `reply` will spawn the text field and a `NotificationReplied` signal is emitted then.
  There's additional kde hints for changing the placeholder text (defaults to "Type a reply..."), submit button text (defaults to "Send") and submit button icon name (defaults to "document-send", that paper aeroplane icon).

TEST PLAN
  Needs D25832 <https://phabricator.kde.org/D25832> for KWin
  Doesn't work on Wayland because there notifications never ever ever get focus and I am sick and tired of running after KWin people to have this addressed.
  
  Notification, just has a normal button - should we have some form of indication that it will spawn a quick reply here actually instead of e.g. bringing up the app or something?
  F7807886: Screenshot_20191210_155559.png <https://phabricator.kde.org/F7807886>
  Default placeholder and button
  F7807887: Screenshot_20191210_155720.png <https://phabricator.kde.org/F7807887>
  Custom placeholder and button
  F7807890: Screenshot_20191210_155642.png <https://phabricator.kde.org/F7807890>
  Button enabled after typing (can also hit Return/Enter so send)
  F7807891: Screenshot_20191210_155652.png <https://phabricator.kde.org/F7807891>

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D25851

AFFECTED FILES
  applets/notifications/notificationapplet.cpp
  applets/notifications/notificationapplet.h
  applets/notifications/package/contents/ui/NotificationItem.qml
  applets/notifications/package/contents/ui/NotificationPopup.qml
  applets/notifications/package/contents/ui/NotificationReplyField.qml
  applets/notifications/package/contents/ui/global/Globals.qml
  libnotificationmanager/dbus/org.freedesktop.Notifications.xml
  libnotificationmanager/notification.cpp
  libnotificationmanager/notification.h
  libnotificationmanager/notification_p.h
  libnotificationmanager/notifications.cpp
  libnotificationmanager/notifications.h
  libnotificationmanager/notificationsmodel.cpp
  libnotificationmanager/notificationsmodel.h
  libnotificationmanager/server.cpp
  libnotificationmanager/server.h
  libnotificationmanager/server_p.cpp
  libnotificationmanager/server_p.h

To: broulik, #plasma, #vdg
Cc: plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20191210/3effb0c2/attachment-0001.html>


More information about the Plasma-devel mailing list