Bug 460310 - 3rd party plasmoids break if size but not implicit size is set in fullRepresentation
Summary: 3rd party plasmoids break if size but not implicit size is set in fullReprese...
Status: RESOLVED INTENTIONAL
Alias: None
Product: plasmashell
Classification: Plasma
Component: general (show other bugs)
Version: 5.25.90
Platform: Other Linux
: NOR normal
Target Milestone: 1.0
Assignee: Plasma Bugs List
URL:
Keywords: regression
: 460338 461171 (view as bug list)
Depends on:
Blocks:
 
Reported: 2022-10-12 14:05 UTC by Christian (Fuchs)
Modified: 2022-11-01 14:06 UTC (History)
4 users (show)

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


Attachments
screenshot of the issue, on the left is 5.26 sizing, on the right is 5.25 (487.22 KB, image/png)
2022-10-12 14:05 UTC, Christian (Fuchs)
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Christian (Fuchs) 2022-10-12 14:05:16 UTC
Created attachment 152749 [details]
screenshot of the issue, on the left is 5.26 sizing, on the right is 5.25

STEPS TO REPRODUCE
1. Get a third party plasmoid that doesn't set implicitWidth and implicitHeight (e.g. weather-applet) 
2.  Update to plasma 5.26
3.  Click to expand

OBSERVED RESULT
Completely wrong sizing (see screenshot)

EXPECTED RESULT
Correct sizing as with 5.25   (implicitWidth = width)

Operating System: Fedora Linux 36
KDE Plasma Version: 5.26.0
KDE Frameworks Version: 5.99.0
Qt Version: 5.15.6
Kernel Version: 5.19.14-200.fc36.x86_64 (64-bit)
Graphics Platform: X11
Processors: 12 × AMD Ryzen 5 5600X 6-Core Processor
Memory: 31.3 GiB of RAM
Graphics Processor: NVIDIA GeForce RTX 3070/PCIe/SSE2
System Version: 1.0

Additional Information: 

the plasmoid doesn't offer resizing as others do with 5.26, but even if manually resized (via the alt+mouseButton shortcut) the custom size is lost after a plasma restart.
Comment 1 Nate Graham 2022-10-12 18:40:35 UTC
Thanks for reporting this. It appears to be caused by the work to make popups resizable; we became more strict about setting correct size properties as a by-product of the work. How sure how we could go back without breaking the resize functionality as currently implemented...
Comment 2 Nate Graham 2022-10-14 18:51:07 UTC
*** Bug 460338 has been marked as a duplicate of this bug. ***
Comment 3 Nate Graham 2022-10-17 20:50:33 UTC
I think unfortunately we have to call this one intentional, as we can't change it without breaking the popup resizability feature. Ideally we would have made such a change for Plasma 6, but that ship has already sailed. I will work on a communication to 3rd-party widget authors.
Comment 4 Nate Graham 2022-11-01 14:06:15 UTC
*** Bug 461171 has been marked as a duplicate of this bug. ***