Bug 419764

Summary: On X11, OSD to set multi-monitor mode inappropriately respects tiling shortcuts
Product: [Plasma] KScreen Reporter: Patrick Silva <bugseforuns>
Component: Plasma AppletAssignee: David Edmundson <kde>
Status: RESOLVED FIXED    
Severity: minor CC: andresbs2000, nate, plasma-bugs
Priority: NOR    
Version: 5.25.90   
Target Milestone: 1.0   
Platform: Arch Linux   
OS: Linux   
Latest Commit: Version Fixed In: 5.26.1
Sentry Crash Report:
Attachments: screen recording
video recorded on X11

Description Patrick Silva 2020-04-06 21:26:44 UTC
Created attachment 127338 [details]
screen recording

SUMMARY
As we can see in the attached screen recording, currently the OSD to set multi-monitor mode is tiled when we press mete+any arrow. This OSD should ignore all tiling shortcuts.

STEPS TO REPRODUCE
1. press meta+p
2. press meta+any arrow
3. 

SOFTWARE/OS VERSIONS
Operating System: KDE neon Unstable Edition
KDE Plasma Version: 5.18.80
KDE Frameworks Version: 5.69.0
Qt Version: 5.14.1

ADDITIONAL INFORMATION
Also reproducible with Plasma 5.18.4 on Arch Linux.
Comment 1 Nate Graham 2020-04-15 03:02:32 UTC
All of these plasma dialogs should, really.
Comment 2 Patrick Silva 2020-09-26 16:08:30 UTC
*** Bug 426941 has been marked as a duplicate of this bug. ***
Comment 3 Nate Graham 2021-09-24 21:01:38 UTC

*** This bug has been marked as a duplicate of bug 411462 ***
Comment 4 Patrick Silva 2022-09-25 09:58:10 UTC
This bug persists only on X11. Please watch the screen recording attached to this comment and observe what happens when I press the tiling shortcuts.

Operating System: KDE neon Unstable Edition
KDE Plasma Version: 5.26.80
KDE Frameworks Version: 5.99.0
Qt Version: 5.15.6
Graphics Platform: X11
Comment 5 Patrick Silva 2022-09-25 09:59:37 UTC
Created attachment 152405 [details]
video recorded on X11
Comment 6 Nate Graham 2022-09-26 17:07:08 UTC
Do you still see the issue for other Plasma dialogs? That is to say, should we re-open Bug 411462? Or are you only seeing the problem for the KScreen OSD?
Comment 7 Patrick Silva 2022-09-26 17:14:37 UTC
Only the KScreen OSD is affected.
Comment 8 Nate Graham 2022-09-27 14:27:49 UTC
Gotcha, thanks. Can reproduce on X11.
Comment 9 Bug Janitor Service 2022-10-14 10:49:59 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/kscreen/-/merge_requests/144
Comment 10 Marco Martin 2022-10-14 11:04:05 UTC
Git commit 5d3dbac4bbcdb47ff1d1292ca516363c55cbf57b by Marco Martin.
Committed on 14/10/2022 at 11:03.
Pushed by mart into branch 'master'.

Use the osd window type

using the proper osd window ttpe forbids it to minimize, maximize or tile

M  +1    -0    osd/osd.cpp

https://invent.kde.org/plasma/kscreen/commit/5d3dbac4bbcdb47ff1d1292ca516363c55cbf57b
Comment 11 Nate Graham 2022-10-14 14:11:46 UTC
Git commit e72a8d984c724748cf9034ce0df3a0b93468a975 by Nate Graham, on behalf of Marco Martin.
Committed on 14/10/2022 at 14:11.
Pushed by ngraham into branch 'Plasma/5.26'.

Use the osd window type

using the proper osd window ttpe forbids it to minimize, maximize or tile


(cherry picked from commit 5d3dbac4bbcdb47ff1d1292ca516363c55cbf57b)

M  +1    -0    osd/osd.cpp

https://invent.kde.org/plasma/kscreen/commit/e72a8d984c724748cf9034ce0df3a0b93468a975