File 0008-Only-restart-Timer-when-the-dialog-is-visi... of Package plasma5-workspace

Overview Repositories Revisions Requests Users Attributes Meta

File 0008-Only-restart-Timer-when-the-dialog-is-visible.patch of Package plasma5-workspace (Revision cfafe49cb45681b5be8054b4ac571308)

Currently displaying revision cfafe49cb45681b5be8054b4ac571308 , Show latest

From 710837e3f82e1dc9e3a12f1dcb285c831ae5f74b Mon Sep 17 00:00:00 2001
From: Kai Uwe Broulik <kde@privat.broulik.de>
Date: Tue, 28 Apr 2015 20:46:14 +0200
Subject: [PATCH 08/12] Only restart Timer when the dialog is visible

Otherwise the Timer might fire when there are no notification properties
causing warnings.

Reviewed-by: mklapetek
---
 applets/notifications/package/contents/ui/NotificationPopup.qml | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/applets/notifications/package/contents/ui/NotificationPopup.qml b/applets/notifications/package/contents/ui/NotificationPopup.qml
index 01ac2a1458c5298bf03b2f322352ce3fb710b5ce..415a125e532ba3da6414ad40ebecfb290c075b3a 100644
--- a/applets/notifications/package/contents/ui/NotificationPopup.qml
+++ b/applets/notifications/package/contents/ui/NotificationPopup.qml
@@ -42,7 +42,9 @@ PlasmaCore.Dialog {
     }
 
     onYChanged: {
-        notificationTimer.restart();
+        if (visible) {
+            notificationTimer.restart();
+        }
     }
 
     function populatePopup(notification) {
@@ -108,7 +110,7 @@ PlasmaCore.Dialog {
             }
 
             textItem: PlasmaComponents.Label {
-                wrapMode: Text.WordWrap
+                wrapMode: Text.WrapAtWordBoundaryOrAnywhere
                 elide: Text.ElideRight
                 verticalAlignment: Text.AlignTop
                 onLinkActivated: Qt.openUrlExternally(link)
-- 
2.3.5

Places

File 0008-Only-restart-Timer-when-the-dialog-is-visible.patch of Package plasma5-workspace (Revision cfafe49cb45681b5be8054b4ac571308)

Places