Bug 419764 - On X11, OSD to set multi-monitor mode inappropriately respects tiling shortcuts
Summary: On X11, OSD to set multi-monitor mode inappropriately respects tiling shortcuts
Status: RESOLVED FIXED
Alias: None
Product: KScreen
Classification: Plasma
Component: Plasma Applet (show other bugs)
Version: 5.25.90
Platform: Arch Linux Linux
: NOR minor
Target Milestone: 1.0
Assignee: David Edmundson
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-04-06 21:26 UTC by Patrick Silva
Modified: 2022-10-14 14:12 UTC (History)
3 users (show)

See Also:
Latest Commit: https://invent.kde.org/plasma/kscreen/commit/e72a8d984c724748cf9034ce0df3a0b93468a975
Version Fixed In: 5.26.1

Attachments
screen recording (2.02 MB, video/webm)
2020-04-06 21:26 UTC, Patrick Silva 		Details
video recorded on X11 (1.21 MB, video/webm)
2022-09-25 09:59 UTC, Patrick Silva 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
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