<table><tr><td style="">broulik created this revision.<br />broulik added reviewers: Plasma, VDG.<br />Herald added a project: Plasma.<br />Herald added a subscriber: plasma-devel.<br />broulik requested review of this revision.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D25851">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>This adds a quick reply feature with a text field inline in the notification popup.<br />
An action named <tt style="background: #ebebeb; font-size: 13px;">reply</tt> will spawn the text field and a <tt style="background: #ebebeb; font-size: 13px;">NotificationReplied</tt> signal is emitted then.<br />
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).</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Needs <a href="https://phabricator.kde.org/D25832" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D25832</a> for KWin<br />
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.</p>

<p>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?<br />
<a href="https://phabricator.kde.org/F7807886" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F7807886: Screenshot_20191210_155559.png</a><br />
Default placeholder and button<br />
<a href="https://phabricator.kde.org/F7807887" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F7807887: Screenshot_20191210_155720.png</a><br />
Custom placeholder and button<br />
<a href="https://phabricator.kde.org/F7807890" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F7807890: Screenshot_20191210_155642.png</a><br />
Button enabled after typing (can also hit Return/Enter so send)<br />
<a href="https://phabricator.kde.org/F7807891" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F7807891: Screenshot_20191210_155652.png</a></p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R120 Plasma Workspace</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D25851">https://phabricator.kde.org/D25851</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>applets/notifications/notificationapplet.cpp<br />
applets/notifications/notificationapplet.h<br />
applets/notifications/package/contents/ui/NotificationItem.qml<br />
applets/notifications/package/contents/ui/NotificationPopup.qml<br />
applets/notifications/package/contents/ui/NotificationReplyField.qml<br />
applets/notifications/package/contents/ui/global/Globals.qml<br />
libnotificationmanager/dbus/org.freedesktop.Notifications.xml<br />
libnotificationmanager/notification.cpp<br />
libnotificationmanager/notification.h<br />
libnotificationmanager/notification_p.h<br />
libnotificationmanager/notifications.cpp<br />
libnotificationmanager/notifications.h<br />
libnotificationmanager/notificationsmodel.cpp<br />
libnotificationmanager/notificationsmodel.h<br />
libnotificationmanager/server.cpp<br />
libnotificationmanager/server.h<br />
libnotificationmanager/server_p.cpp<br />
libnotificationmanager/server_p.h</div></div></div><br /><div><strong>To: </strong>broulik, Plasma, VDG<br /><strong>Cc: </strong>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<br /></div>