KDE/kdebase/workspace/plasma/generic/applets/systemtray/core

Marco Martin notmart at gmail.com
Sat Oct 30 15:35:56 CEST 2010


SVN commit 1191305 by mart:

don't move icons in the first position when the are notificating, apparently is confusing (except the notifications applet that always remain there)
i'm not completely happy about this solutions, there gota be something better :/
CCMAIL:plasma-devel at kde.org


 M  +2 -8      task.cpp  


--- trunk/KDE/kdebase/workspace/plasma/generic/applets/systemtray/core/task.cpp #1191304:1191305
@@ -177,21 +177,15 @@
 
 void Task::resetHiddenStatus()
 {
-    if (d->status == NeedsAttention) {
-        setOrder(First);
-        if (hidden() & AutoHidden) {
+    //Apparently setOrder(First) when d->status == NeedsAttention seems the most hated feature ever :/
+    if ((d->status == Active || d->status == NeedsAttention) && (hidden() & AutoHidden)) {
             setHidden(hidden() ^ AutoHidden);
-        }
-    } else {
-        if (d->status == Active && (hidden() & AutoHidden)) {
-            setHidden(hidden() ^ AutoHidden);
         } else if (d->status == Passive) {
             setHidden(hidden() | AutoHidden);
         }
 
         setOrder(Normal);
     }
-}
 
 }
 


More information about the Plasma-devel mailing list