[kwin] [Bug 406101] Window "Keep above" overrides "Show Desktop"

2023-06-02 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=406101

Nate Graham  changed:

   What|Removed |Added

   Version Fixed In|5.16.0  |6.0

-- 
You are receiving this mail because:
You are watching all bug changes.

[kwin] [Bug 406101] Window "Keep above" overrides "Show Desktop"

2023-06-01 Thread Vlad Zahorodnii
https://bugs.kde.org/show_bug.cgi?id=406101

Vlad Zahorodnii  changed:

   What|Removed |Added

 Status|REOPENED|RESOLVED
  Latest Commit|https://commits.kde.org/kwi |https://invent.kde.org/plas
   |n/1e2a0028c3151cb38f252baa7 |ma/kwin/-/commit/a7f41f26bb
   |8a5f22b8cd7c7e6 |ed92c153d2b47fb4ad295713f2f
   ||8e6
 Resolution|--- |FIXED

--- Comment #7 from Vlad Zahorodnii  ---
Git commit a7f41f26bbed92c153d2b47fb4ad295713f2f8e6 by Vlad Zahorodnii.
Committed on 01/06/2023 at 08:59.
Pushed by vladz into branch 'master'.

Make show desktop mode hide windows

After porting the desktop background window to the layer-shell protocol,
the desktop window is not raised up in the stack anymore when the
showing desktop window is activated. The main reason to avoid this is to
avoid fighting and overriding layer shell logic.

As another way to implement the show desktop mode, this change makes the
workspace hide windows that don't belong to desktop.

It's a better solution for a couple of reasons: "keep above" and other
overlay windows will be properly hidden and it lets us avoid touching
the stacking order.
Related: bug 387593, bug 469827

M  +5-0src/activation.cpp
M  +1-0src/effects.cpp
M  +1-0src/effects.h
M  +2-2src/input.cpp
M  +1-1src/internalwindow.cpp
M  +1-1src/libkwineffects/kwinanimationeffect.cpp
M  +6-0src/libkwineffects/kwineffects.h
M  +23   -80   src/plugins/eyeonscreen/package/contents/code/main.js
M  +2-11   src/plugins/windowaperture/package/contents/code/main.js
M  +2-1src/scene/windowitem.cpp
M  +1-1src/scripting/workspace_wrapper.cpp
M  +1-1src/waylandwindow.cpp
M  +19   -12   src/window.cpp
M  +6-1src/window.h
M  +14   -26   src/workspace.cpp
M  +16   -15   src/x11window.cpp
M  +2-1src/x11window.h
M  +0-3src/xdgshellwindow.cpp

https://invent.kde.org/plasma/kwin/-/commit/a7f41f26bbed92c153d2b47fb4ad295713f2f8e6

-- 
You are receiving this mail because:
You are watching all bug changes.

[kwin] [Bug 406101] Window "Keep above" overrides "Show Desktop"

2019-04-29 Thread Vlad Zagorodniy
https://bugs.kde.org/show_bug.cgi?id=406101

Vlad Zagorodniy  changed:

   What|Removed |Added

 Status|RESOLVED|REOPENED
 Resolution|INTENTIONAL |---

--- Comment #6 from Vlad Zagorodniy  ---
Well, no. The bug is still there.

-- 
You are receiving this mail because:
You are watching all bug changes.

[kwin] [Bug 406101] Window "Keep above" overrides "Show Desktop"

2019-04-28 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=406101

Nate Graham  changed:

   What|Removed |Added

 Resolution|FIXED   |INTENTIONAL

--- Comment #4 from Nate Graham  ---
The fix was reverted because it broke other things, unfortunately. So it looks
like this behavior is here to stay and is not considered to be "as designed."
:/

-- 
You are receiving this mail because:
You are watching all bug changes.

[kwin] [Bug 406101] Window "Keep above" overrides "Show Desktop"

2019-04-28 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=406101

--- Comment #5 from Nate Graham  ---
I mean, *now* considered as designed. :)

-- 
You are receiving this mail because:
You are watching all bug changes.

[kwin] [Bug 406101] Window "Keep above" overrides "Show Desktop"

2019-04-27 Thread Patrick Silva
https://bugs.kde.org/show_bug.cgi?id=406101

Patrick Silva  changed:

   What|Removed |Added

 CC||bugsefor...@gmx.com

--- Comment #3 from Patrick Silva  ---
The behavior described here persists on neon dev unstable.

Operating System: KDE neon Unstable Edition
KDE Plasma Version: 5.15.80
KDE Frameworks Version: 5.58.0
Qt Version: 5.12.0

-- 
You are receiving this mail because:
You are watching all bug changes.

[kwin] [Bug 406101] Window "Keep above" overrides "Show Desktop"

2019-04-16 Thread Vlad Zagorodniy
https://bugs.kde.org/show_bug.cgi?id=406101

--- Comment #2 from Vlad Zagorodniy  ---
Git commit 8010e076ee2dbcaaa39f777b2ea6dc96a5495d6f by Vlad Zagorodniy.
Committed on 16/04/2019 at 08:10.
Pushed by vladz into branch 'master'.

Revert "Move keep-above clients to the Normal layer when showing desktop"

This reverts commit 1e2a0028c3151cb38f252baa78a5f22b8cd7c7e6.

Unfortunately, we can't move clients from the above layer to the normal
layer because some of those clients have to be visible when showing
desktop, one such client for example is krunner.

M  +1-1abstract_client.cpp
M  +1-3workspace.cpp

https://commits.kde.org/kwin/8010e076ee2dbcaaa39f777b2ea6dc96a5495d6f

-- 
You are receiving this mail because:
You are watching all bug changes.

[kwin] [Bug 406101] Window "Keep above" overrides "Show Desktop"

2019-04-03 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=406101

Nate Graham  changed:

   What|Removed |Added

   Version Fixed In||5.16.0

-- 
You are receiving this mail because:
You are watching all bug changes.

[kwin] [Bug 406101] Window "Keep above" overrides "Show Desktop"

2019-04-02 Thread Vlad Zagorodniy
https://bugs.kde.org/show_bug.cgi?id=406101

Vlad Zagorodniy  changed:

   What|Removed |Added

  Latest Commit||https://commits.kde.org/kwi
   ||n/1e2a0028c3151cb38f252baa7
   ||8a5f22b8cd7c7e6
 Resolution|--- |FIXED
 Status|CONFIRMED   |RESOLVED

--- Comment #1 from Vlad Zagorodniy  ---
Git commit 1e2a0028c3151cb38f252baa78a5f22b8cd7c7e6 by Vlad Zagorodniy.
Committed on 02/04/2019 at 20:44.
Pushed by vladz into branch 'master'.

Move keep-above clients to the Normal layer when showing desktop

Summary:
Implementation of the Show Desktop feature moves desktop windows to
the Above layer, but it doesn't take into account existing clients
that belong to the Above layer. If there are any, we have to move
them to a layer below (e.g. normal), otherwise those clients will be
visible when showing the desktop.

Reviewers: #kwin, graesslin

Reviewed By: #kwin, graesslin

Subscribers: graesslin, kwin

Tags: #kwin

Differential Revision: https://phabricator.kde.org/D20153

M  +1-1abstract_client.cpp
M  +3-1workspace.cpp

https://commits.kde.org/kwin/1e2a0028c3151cb38f252baa78a5f22b8cd7c7e6

-- 
You are receiving this mail because:
You are watching all bug changes.

[kwin] [Bug 406101] Window "Keep above" overrides "Show Desktop"

2019-04-01 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=406101

Nate Graham  changed:

   What|Removed |Added

 CC||n...@kde.org

-- 
You are receiving this mail because:
You are watching all bug changes.

[kwin] [Bug 406101] Window "Keep above" overrides "Show Desktop"

2019-04-01 Thread Vlad Zagorodniy
https://bugs.kde.org/show_bug.cgi?id=406101

Vlad Zagorodniy  changed:

   What|Removed |Added

URL||https://phabricator.kde.org
   ||/D20153
  Flags||ReviewRequest+
 Ever confirmed|0   |1
 Status|REPORTED|CONFIRMED

-- 
You are receiving this mail because:
You are watching all bug changes.