https://bugs.kde.org/show_bug.cgi?id=382263
Aleix Pol <aleix...@kde.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |aleix...@kde.org
--- Comment #2 from Aleix Pol <aleix...@kde.org> ---
Here's a patch that addresses this issue, yet I don't really want it in so I'll
leave you with a patch and why I think it doesn't improve the situation.
- It breaks clicking on the notification, as the TextEdit eats all the click
events. Putting the textfield within the mousearea (which would be the right
thing to do) won't work.
- selection is fiddly
- actions in notifications shouldn't be passed as links within the text.
If you still want this:
diff --git a/applets/notifications/package/contents/ui/NotificationItem.qml
b/applets/notifications/package/contents/ui/NotificationItem.qml
index 42f20bc3..d74e5a35 100644
--- a/applets/notifications/package/contents/ui/NotificationItem.qml
+++ b/applets/notifications/package/contents/ui/NotificationItem.qml
@@ -294,14 +294,10 @@ MouseArea {
}
MouseArea {
anchors.fill: parent
- acceptedButtons: Qt.RightButton | Qt.LeftButton
+ acceptedButtons: Qt.RightButton
onClicked: {
- if (mouse.button == Qt.RightButton)
- contextMenu.open(mouse.x, mouse.y)
- else {
- notificationItem.clicked(mouse)
- }
+ contextMenu.open(mouse.x, mouse.y)
}
PlasmaComponents.ContextMenu {
--
You are receiving this mail because:
You are watching all bug changes.
