<table><tr><td style="">ngraham added a comment.
</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/D18934">View Revision</a></tr></table><br /><div><div><p>I don't like this approach because the IconItem has no pressed state. This is already a problem for the Task Manager tooltips, and I'd like to not replicate it here too. Also I don't really like the fact that the close button is red here even before it's hovered; dismissing a notification is not a destructive action.</p>

<p>I have an idea though. Try applying this on top of your patch:</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="diff" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);"><span style="color: #000080">diff --git a/applets/notifications/package/contents/ui/NotificationItem.qml b/applets/notifications/package/contents/ui/NotificationItem.qml</span>
<span style="color: #000080">index 83bd9dcd..5386e557 100644</span>
<span style="color: #a00000">--- a/applets/notifications/package/contents/ui/NotificationItem.qml</span>
<span style="color: #00a000">+++ b/applets/notifications/package/contents/ui/NotificationItem.qml</span>
<span style="color: #800080">@@ -237,8 +237,8 @@ MouseArea {</span>
             PlasmaCore.IconItem {
                 id:closeIcon
                 anchors.fill: parent
<span style="color: #a00000">-                active: parent.containsMouse</span>
<span style="color: #a00000">-                source: "window-close"</span>
<span style="color: #00a000">+                active: parent.pressed</span>
<span style="color: #00a000">+                source: parent.containsMouse ? "window-close" : "window-close-symbolic"</span>
                 animated: false
             }
         }</pre></div>

<p>(You can do this with <tt style="background: #ebebeb; font-size: 13px;">git apply -</tt> then paste, then hit <kbd style="display: inline-block; min-width: 1em; padding: 4px 5px 5px; font-weight: normal; font-size: 0.8rem; text-align: center; text-decoration: none; line-height: 0.6rem; border-radius: 3px; box-shadow: inset 0 -1px 0 rgba(71, 87, 120, 0.08); user-select: none; background: #f7f7f7; border: 1px solid #C7CCD9;">Ctrl</kbd><span class="kbd-join" style="padding: 0 4px; color: #92969D;">+</span><kbd style="display: inline-block; min-width: 1em; padding: 4px 5px 5px; font-weight: normal; font-size: 0.8rem; text-align: center; text-decoration: none; line-height: 0.6rem; border-radius: 3px; box-shadow: inset 0 -1px 0 rgba(71, 87, 120, 0.08); user-select: none; background: #f7f7f7; border: 1px solid #C7CCD9;">d</kbd>)</p>

<p>Let me know if that appeals to you.</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/D18934">https://phabricator.kde.org/D18934</a></div></div><br /><div><strong>To: </strong>filipf, ngraham, VDG, rooty, Plasma<br /><strong>Cc: </strong>plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>