Bug 428147 - "Maximized" window placement mode gets applied to Plasma OSDs
Summary: "Maximized" window placement mode gets applied to Plasma OSDs
Status: RESOLVED FIXED
Alias: None
Product: kwin
Classification: Plasma
Component: wayland-generic (show other bugs)
Version: 5.20.1
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: KWin default assignee
URL:
Keywords: wayland
: 436420 (view as bug list)
Depends on:
Blocks:
 
Reported: 2020-10-23 16:20 UTC by kelnio@yahoo.com
Modified: 2023-08-30 13:48 UTC (History)
5 users (show)

See Also:
Latest Commit: https://invent.kde.org/frameworks/plasma-framework/-/commit/bf8932f293dd7ceb9904a938cdafdc20f33190de
Version Fixed In: 6.0

Attachments
Screenshot_20201027_180314.png (564.65 KB, image/png)
2020-10-27 22:16 UTC, kelnio@yahoo.com 		Details
attachment-20687-0.html (1.75 KB, text/html)
2020-11-02 00:32 UTC, kelnio@yahoo.com 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description kelnio@yahoo.com 2020-10-23 16:20:14 UTC 
SUMMARY
On-screen displays, like the volume rocker, brightness notification, and virtual keyboard indicator take up the entire screen in a Plasma Wayland session on a HiDPI display with the scale factor set to 200%.

STEPS TO REPRODUCE
1. On a HiDPI display, in a Plasma Wayland session, set the scale factor to 200%
2. Logout, then log back in to the Plasma Wayland session
3. Adjust the volume or brightness

OBSERVED RESULT
On-screen displays take up the entire screen

EXPECTED RESULT
On-screen displays are small and unobtrusive

SOFTWARE/OS VERSIONS
KDE neon 5.20 User Edition
Linux/KDE Plasma: 
(available in About System)
KDE Plasma Version: 5.20.1
KDE Frameworks Version: 5.75.0
Qt Version: 5.15.0

ADDITIONAL INFORMATION
Running on a Microsoft Surface Pro 4 with the 5.9.1-surface kernel.
Comment 1 Nate Graham 2020-10-26 15:26:25 UTC 
Can you please attach a screenshot?

Does the problem go away if you delete the SVG cache with `rm ~/.cache/plasma*` and then restart plasma with `plasmashell --replace`?
Comment 2 kelnio@yahoo.com 2020-10-27 22:16:41 UTC 
Created attachment 132813 [details]
Screenshot_20201027_180314.png

 The problem goes away for the very first time I adjust the volume after deleting the cache and restarting the plasmashell. However, it returns immediately for all subsequent volume changes/on-screen display events. A screenshot of the volume on-screen display with the bug is attached.

    On Monday, October 26, 2020, 11:26:27 AM EDT, Nate Graham <bugzilla_noreply@kde.org> wrote:  
 
 https://bugs.kde.org/show_bug.cgi?id=428147

Nate Graham <nate@kde.org> changed:

          What    |Removed                    |Added
----------------------------------------------------------------------------
            Status|REPORTED                    |NEEDSINFO
        Resolution|---                        |WAITINGFORINFO
                CC|                            |nate@kde.org

--- Comment #1 from Nate Graham <nate@kde.org> ---
Can you please attach a screenshot?

Does the problem go away if you delete the SVG cache with `rm ~/.cache/plasma*`
and then restart plasma with `plasmashell --replace`?
Comment 3 Nate Graham 2020-10-29 13:45:05 UTC 
Well that's bizarre.

Are you using any window rules? What plasma theme are you using?
Comment 4 kelnio@yahoo.com 2020-11-02 00:32:11 UTC 
Created attachment 132955 [details]
attachment-20687-0.html

 Sorry, I was in the middle of moving, so I just had a chance to check themes/windows rules. I figured out what is causing the bug! 
Setting "Window placement" to "Maximize" under Window Management ---> Window Behavior causes the on-screen displays to maximize also. I reset it to "Minimal Overlapping," logged out, then logged back in and now the on-screen displays are the regular size. 
    On Thursday, October 29, 2020, 9:45:08 AM EDT, Nate Graham <bugzilla_noreply@kde.org> wrote:  
 
 https://bugs.kde.org/show_bug.cgi?id=428147

--- Comment #3 from Nate Graham <nate@kde.org> ---
Well that's bizarre.

Are you using any window rules? What plasma theme are you using?
Comment 5 Méven Car 2020-11-03 14:41:34 UTC 
That's a (In reply to kelnio@yahoo.com from comment #4)
> Created attachment 132955 [details]
> attachment-20687-0.html
> 
>  Sorry, I was in the middle of moving, so I just had a chance to check
> themes/windows rules. I figured out what is causing the bug! 
> Setting "Window placement" to "Maximize" under Window Management ---> Window
> Behavior causes the on-screen displays to maximize also. I reset it to
> "Minimal Overlapping," logged out, then logged back in and now the on-screen
> displays are the regular size. 

That's a bug : Window Placement settings should not apply to osd.
Comment 6 Nate Graham 2021-04-27 15:32:08 UTC 

*** This bug has been marked as a duplicate of bug 428859 ***
Comment 7 David Redondo 2021-05-05 13:36:05 UTC 
*** Bug 436420 has been marked as a duplicate of this bug. ***
Comment 8 Toadfield 2021-06-13 21:47:11 UTC 
This also happens when I edit the task bar,I assume that this is the same problem.
Comment 9 David Edmundson 2023-08-30 11:44:50 UTC 
Git commit bf8932f293dd7ceb9904a938cdafdc20f33190de by David Edmundson.
Committed on 30/08/2023 at 13:41.
Pushed by davidedmundson into branch 'master'.

Dialog: Split wayland parts into new class

The PlasmaShell wayland parts are split out of Dialog, allowing for
re-use in upcoming replacements.

In addition we move away from KWayland in favour of generated classes
and QtWaylandClientExtension. We also change the lifespan of the
user-facing class to match the Window rather than the plasma_surface.

This allows a caller to call setPanelBehavior once and sending it
multiple times as the underlying shell surface is recreated should
happen automatically.

This also should fix ordering applying initial plasma_surface
properties before the first commit.

Whilst Dialog could now be cleaned up further, this commit does not do
so as it is being phased out.

M  +0    -4    .kde-ci.yml
M  +4    -8    CMakeLists.txt
M  +0    -1    src/plasma/config-plasma.h.cmake
M  +15   -10   src/plasmaquick/CMakeLists.txt
M  +54   -105  src/plasmaquick/dialog.cpp
M  +126  -38   src/plasmaquick/waylandintegration.cpp
M  +39   -24   src/plasmaquick/waylandintegration_p.h

https://invent.kde.org/frameworks/plasma-framework/-/commit/bf8932f293dd7ceb9904a938cdafdc20f33190de