From 749351ec6477e010a7ca8b5ddf3a22bcd70788bd Mon Sep 17 00:00:00 2001
From: Matthijs Tijink <matthijstijink@gmail.com>
Date: Sat, 24 Mar 2018 15:31:10 +0100
Subject: [PATCH] Add tooltips to the "dismiss" buttons for notifications

Summary: It also makes it clearer that the top line is not a notification, but a header.

Reviewers: #kde_connect, apol

Reviewed By: #kde_connect, apol

Subscribers: #kde_connect

Differential Revision: https://phabricator.kde.org/D11646
---
 plasmoid/package/contents/ui/DeviceDelegate.qml | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/plasmoid/package/contents/ui/DeviceDelegate.qml b/plasmoid/package/contents/ui/DeviceDelegate.qml
index b2eafa7a0..d710892d4 100644
--- a/plasmoid/package/contents/ui/DeviceDelegate.qml
+++ b/plasmoid/package/contents/ui/DeviceDelegate.qml
@@ -206,6 +206,7 @@ PlasmaComponents.ListItem
                 visible: notificationsModel.isAnyDimissable;
                 anchors.right: parent.right
                 iconSource: "window-close"
+                tooltip: i18n("Dismiss all notifications")
                 onClicked: notificationsModel.dismissAll();
             }
         }
@@ -241,6 +242,7 @@ PlasmaComponents.ListItem
                     enabled: repliable
                     anchors.right: dismissButton.left
                     iconSource: "mail-reply-sender"
+                    tooltip: i18n("Reply")
                     onClicked: dbusInterface.reply();
                 }
                 PlasmaComponents.ToolButton {
@@ -249,6 +251,7 @@ PlasmaComponents.ListItem
                     enabled: dismissable
                     anchors.right: parent.right
                     iconSource: "window-close"
+                    tooltip: i18n("Dismiss")
                     onClicked: dbusInterface.dismiss();
                 }
             }