Bug 113552

SVN commit 465900 by amantia:

Be consistent and don't use animation in any case when a bunch of minimized windows (with Show Desktop) are restored.
But I don't really understand why they should be restored when you invoke Configure Desktop...

CCBUG: 110794
CCBUG: 113552

 M  +1 -1      workspace.cpp  


--- branches/KDE/3.5/kdebase/kwin/workspace.cpp #465899:465900
@@ -2562,7 +2562,7 @@
         for( ClientList::ConstIterator it = showing_desktop_clients.begin();
              it != showing_desktop_clients.end();
              ++it )
-            (*it)->unminimize();
+            (*it)->unminimize(true);
         }
     showing_desktop_clients.clear();
     --block_showing_desktop;

The same behavior occurs after pressing [Del] on a desktop icon so that the alert window appears.

I am using KDE 3.5 beta2 and this patch seems to have made it in, since unminimizing is much faster than in beta1.

Nevertheless, we necessarily have to fix this for 3.5, this in unacceptable. Is anyone able to explain why windows are unminimized at all?

In KDE 3.5 beta2 still happens (though unminimizing is instant). 
I have also noticed it happens when minimizing all windows, then trying to run application shortcut from the desktop.

Still present in KDE 3.5 RC1.

Have a look at Bug 94199 and Bug 108899. I think this behaviour is intended. I would like the other behaviour better too, but maybe we are outnumbered ;-)

also, rmb->run command from desktop's rmb menu causes this bug. this bug still present in KDE 3.5.0. this is an annoying small bug :(

*** Bug 123668 has been marked as a duplicate of this bug. ***

*** Bug 125286 has been marked as a duplicate of this bug. ***

SVN commit 528391 by lunakl:

Don't deactivate the showing desktop state when a new window is shown
that belongs to the desktop.
BUG: 113552



 M  +10 -1     client.cpp  
 M  +10 -1     manage.cpp  


--- branches/KDE/3.5/kdebase/kwin/client.cpp #528390:528391
@@ -895,7 +895,16 @@
         }
     if( show )
         {
-        if( workspace()->showingDesktop())
+        bool belongs_to_desktop = false;
+        for( ClientList::ConstIterator it = group()->members().begin();
+             it != group()->members().end();
+             ++it )
+            if( (*it)->isDesktop())
+                {
+                belongs_to_desktop = true;
+                break;
+                }
+        if( !belongs_to_desktop && workspace()->showingDesktop())
             workspace()->resetShowingDesktop( true );
         if( isShade())
             setMappingState( IconicState );
--- branches/KDE/3.5/kdebase/kwin/manage.cpp #528390:528391
@@ -455,7 +455,16 @@
         if( !isOnCurrentDesktop() && !isMapped && !session && ( allow || workspace()->sessionSaving()))
             workspace()->setCurrentDesktop( desktop());
 
-        if( workspace()->showingDesktop())
+        bool belongs_to_desktop = false;
+        for( ClientList::ConstIterator it = group()->members().begin();
+             it != group()->members().end();
+             ++it )
+            if( (*it)->isDesktop())
+                {
+                belongs_to_desktop = true;
+                break;
+                }
+        if( !belongs_to_desktop && workspace()->showingDesktop())
             workspace()->resetShowingDesktop( false );
 
         if( isOnCurrentDesktop() && !isMapped && !allow )

So, contrary to comment #5, the former behaviour is no longer intended?