Bug 347212

*** This bug has been marked as a duplicate of bug 346837 ***

Currently intended behavior (but not undiscussed, see linked bugs)

We need to disassemble the bug reg. reports because there're
a) actual implementation bugs
b) worries about the "behavior" of the plasmoid trigger (bug #346933)
c) concerns reg. the general panel stacking(?) (ie. whether they should generally be kept above the desktop or only opt into this)

=> please select a, b or c for you :-)

I dont understand the difference between b) and c), btw I would the old behaviour of "show desktop" :)

If the only problem is that the plasmoid isn't accessible while showing the desktop, possible solutions would be to have it keep *ITS* panel (but not others) on top of the desktop or place a copy of itself onto the desktop etc.
It does not imply the requirement to keep *ANY* panel on top of the desktop.

This item (minus the panel situation!!!) is subject to flame since 2003 and we ultimately decided to perform a clean split between the fdo compliant "show desktop" (that does not minimize as implemetation detail to avoid any confusion) and a proprietary "minimize all" solution to end this - so it's not gonna restore ;-)

No one in real word cares this "flame".

KWin devs do live in the real world, and telling the 100th time "show desktop is not minimize all and you actually wanted to use another virtual desktop itfp" to be countered by completely uninformed "but i want *this* to be *that* - i don't want to use *that* instead" is incredibly annoying.

Unless you volunteer to handle those discussions, you'll have to accept that *we* care about this flame and chose to cut it down by leaving no trail of ambiguity.

Git commit f3b69b0ed0d3fa863ed4b2233ac5ce6c5fcdd9c3 by Thomas Lübking.
Committed on 19/05/2015 at 20:01.
Pushed by luebking into branch 'Plasma/5.3'.

Showing Desktop: keep docks visible

REVIEW: 123783
Related: bug 346933, bug 346837
FIXED-IN: 5.3.1

M  +7    -0    layers.cpp
M  +9    -5    workspace.cpp

http://commits.kde.org/kwin/f3b69b0ed0d3fa863ed4b2233ac5ce6c5fcdd9c3

Git commit a16a489a43aef03e198630f80d8236b56052b6f9 by Thomas Lübking.
Committed on 19/05/2015 at 20:06.
Pushed by luebking into branch 'Plasma/5.3'.

Show Desktop: keep desktop group visible

Windows that "belong" to the desktop are
kept visible and do no break the state on
activation/mapping

REVIEW: 123783
Related: bug 346837, bug 346933

M  +6    -0    layers.cpp
M  +3    -0    workspace.cpp

http://commits.kde.org/kwin/a16a489a43aef03e198630f80d8236b56052b6f9

Git commit eb789e43e317fbb25fbd1f51d0edf9800f8c002a by Thomas Lübking.
Committed on 19/05/2015 at 20:09.
Pushed by luebking into branch 'Plasma/5.3'.

Show Desktop: activateNextClient prefers desktop

othrwise closing a keepabove or desktop group
window would activate some random window and break
the state as a side-effect

REVIEW: 123783
Related: bug 346837, bug 346933

M  +5    -2    activation.cpp

http://commits.kde.org/kwin/eb789e43e317fbb25fbd1f51d0edf9800f8c002a

Git commit 43e3b7db285c48efb25966334262d0e80190fd9e by Thomas Lübking.
Committed on 19/05/2015 at 20:07.
Pushed by luebking into branch 'Plasma/5.3'.

Show Desktop: break state with activation (only)

... not them becoming visible.
Latter doesn't work for most cases (unminimizing)
for obvious reasons (they're not minimized) and
when a new window is mapped, the focus stealing
prevention seems a good filter
(if it's not good enough to gain the focus,
it's not good enough to break the state either)

REVIEW: 123783
Related: bug 346837, bug 346933

M  +12   -16   client.cpp
M  +0    -1    client.h
M  +0    -2    manage.cpp

http://commits.kde.org/kwin/43e3b7db285c48efb25966334262d0e80190fd9e

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