Bug 499154 - `plasmawindowed` does not pad window decorations, for me.
Summary: `plasmawindowed` does not pad window decorations, for me.
Status: RESOLVED FIXED
Alias: None
Product: Plasma SDK
Classification: Plasma
Component: plasmoidviewer (other bugs)
Version First Reported In: 6.2.5
Platform: Fedora RPMs Linux
: NOR minor
Target Milestone: ---
Assignee: Plasma Bugs List
URL: https://discuss.kde.org/t/this-week-i...
Keywords:
Depends on:
Blocks:
 
Reported: 2025-01-26 11:15 UTC by Roke Julian Lockhart Beedell
Modified: 2025-02-12 11:44 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
Screenshot of mine in 6.2.5 (for `#c0`). (19.73 KB, image/png)
2025-01-26 11:15 UTC, Roke Julian Lockhart Beedell
Details
Screenshot of another's in 6.3.80 (for `#c0`). (22.50 KB, image/png)
2025-01-26 11:16 UTC, Roke Julian Lockhart Beedell
Details
An updateable report to replace `#c0`. (1.60 KB, text/markdown)
2025-01-26 11:21 UTC, Roke Julian Lockhart Beedell
Details
Screenshot of mine in 6.3.0 (for `#c6`). (22.60 KB, image/png)
2025-02-12 10:59 UTC, Roke Julian Lockhart Beedell
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Roke Julian Lockhart Beedell 2025-01-26 11:15:43 UTC
Created attachment 177689 [details]
Screenshot of mine in 6.2.5 (for `#c0`).

SUMMARY
-------

For me, `plasmawindowed` does not pad window decorations. Or, rather, it renders the content of the window beneath them, since the scrollbar is partially cut-off. You can see this in undermentioned screenshots.

This does not occur to another user with the same version of relevant libraries.

STEPS TO REPRODUCE
------------------

~~~sh
#!/usr/bin/env sh
plasmawindowed org.kde.plasma.volume
~~~

OBSERVED RESULT
---------------

It's not padded, as [this screenshot of mine](https://discuss-cdn.kde.org/uploads/default/original/3X/a/e/aef1e54d7593878f1d350218229b072d268b85d7.png) depicts.

EXPECTED RESULT
---------------

It should be padded, as [this screenshot from another user](https://discuss-cdn.kde.org/uploads/default/original/3X/3/4/347154b29d4ca8cde175037aaffa42c3b540f9fb.png) depicts.

SOFTWARE/OS VERSIONS
--------------------

1.	~~~sh
	#!/usr/bin/env sh
	kinfo
	~~~

2.	> ~~~YAML
	> Operating System: Fedora Linux 41
	> KDE Plasma Version: 6.2.5
	> KDE Frameworks Version: 6.10.0
	> Qt Version: 6.8.1
	> Kernel Version: 6.12.9-200.fc41.x86_64 (64-bit)
	> Graphics Platform: Wayland
	> Processors: 12 ร— AMD Ryzen 5 7600X 6-Core Processor
	> Memory: 30.4 GiB of RAM
	> Graphics Processor: AMD Radeon RX 5700
	> ~~~

ADDITIONAL INFORMATION
----------------------

Another user explains at https://discuss.kde.org/t/this-week-in-plasma-fancy-time-zone-picker-kde-blogs/28753/20?u=rokejulianlockhart that their installation does pad the windows correctly, and has provided visual evidence of it.

I shall attach it after this has been filed, since I want to attach the erroneous one first.
Comment 1 Roke Julian Lockhart Beedell 2025-01-26 11:16:40 UTC
Created attachment 177690 [details]
Screenshot of another's in 6.3.80 (for `#c0`).

(In reply to Roke Julian Lockhart Beedell from comment #0)  
> Another user explains at https://discuss.kde.org/t/this-week-in-plasma-fancy-time-zone-picker-kde-blogs/28753/20?u=rokejulianlockhart that their installation does pad the windows correctly, and has provided visual evidence of it.
> 
> I shall attach it after this has been filed, since I want to attach the erroneous one first.
Comment 2 Roke Julian Lockhart Beedell 2025-01-26 11:19:18 UTC
(In reply to Roke Julian Lockhart Beedell from comment #0)
> OBSERVED RESULT
> ---------------
> 
> It's not padded, as [this screenshot of mine](https://discuss-cdn.kde.org/uploads/default/original/3X/a/e/aef1e54d7593878f1d350218229b072d268b85d7.png) depicts.
> 
> EXPECTED RESULT
> ---------------
> 
> It should be padded, as [this screenshot from another user](https://discuss-cdn.kde.org/uploads/default/original/3X/3/4/347154b29d4ca8cde175037aaffa42c3b540f9fb.png) depicts.

Apologies, for I forget that format doesn't work well in Bugzilla. I should have used substitution syntax:

~~~Markdown
OBSERVED RESULT
---------------

It's not padded, as [this screenshot of mine][1] depicts.

[1]: https://discuss-cdn.kde.org/uploads/default/original/3X/a/e/aef1e54d7593878f1d350218229b072d268b85d7.png

EXPECTED RESULT
---------------

It should be padded, as [this screenshot from another user][2] depicts.

[2]: https://discuss-cdn.kde.org/uploads/default/original/3X/3/4/347154b29d4ca8cde175037aaffa42c3b540f9fb.png
~~~
Comment 3 Roke Julian Lockhart Beedell 2025-01-26 11:21:29 UTC
Created attachment 177691 [details]
An updateable report to replace `#c0`.

(In reply to Roke Julian Lockhart Beedell from comment #2)  
I'll just update a file. No need to publish updates here. Collapse them, if desirable.
Comment 4 cwo 2025-01-28 08:58:48 UTC
I can reproduce this on stock Fedora 41 and stock Tuxedo OS. I can't reproduce this on master (as the screenshot shows).

I suspect that this is an issue that was fixed somewhere along the way. I don't see anything obvious in the plasmoidviewer commits that could be related to this, so possibly something in a library. Qt version is the same, so that shouldn't be it.

Plasma 6.3 is right around the corner, so we can test the distro versions then. I'm setting this to NEEDSINFO until then; the 14-day reminder email should come right around the 6.3 release so exactly when we can check it.
Comment 5 Bug Janitor Service 2025-02-12 03:46:45 UTC
๐Ÿ›๐Ÿงน โš ๏ธ This bug has been in NEEDSINFO status with no change for at least 15 days. Please provide the requested information, then set the bug status to REPORTED. If there is no change for at least 30 days, it will be automatically closed as RESOLVED WORKSFORME.

For more information about our bug triaging procedures, please read https://community.kde.org/Guidelines_and_HOWTOs/Bug_triaging.

Thank you for helping us make KDE software even better for everyone!
Comment 6 Roke Julian Lockhart Beedell 2025-02-12 10:59:35 UTC
Created attachment 178175 [details]
Screenshot of mine in 6.3.0 (for `#c6`).

(In reply to Bug Janitor Service from comment #5)  
Good timing! I got the 6.3 update a few minutes ago. It's indeed remediated in 6.3.
Comment 7 cwo 2025-02-12 11:02:06 UTC
I was just about to go test this, thanks for taking care of it! Marking as resolved.
Comment 8 Roke Julian Lockhart Beedell 2025-02-12 11:11:01 UTC
(In reply to cwo from comment #7)  
Though, does it look to you like the toolbar buttons are correctly padded? To me, it looks like the right-most one overlaps with the window border.
Comment 9 cwo 2025-02-12 11:24:54 UTC
(In reply to Roke Julian Lockhart Beedell from comment #8)
> To me, it looks like the right-most one overlaps with the window border.

It goes right up to the window border. It probably shouldn't do that and have a smallSpacing instead. But I guess that's a different issue.
Comment 10 Roke Julian Lockhart Beedell 2025-02-12 11:33:17 UTC
(In reply to cwo from comment #9)  
...I'll file *that*, too, then. Thanks.