Bug 460310

Summary: 3rd party plasmoids break if size but not implicit size is set in fullRepresentation
Product: [Plasma] plasmashell Reporter: Christian (Fuchs) <kde>
Component: generalAssignee: Plasma Bugs List <plasma-bugs>
Status: RESOLVED INTENTIONAL    
Severity: normal CC: kde, nate, nucleo, saunders
Priority: NOR Keywords: regression
Version: 5.25.90   
Target Milestone: 1.0   
Platform: Other   
OS: Linux   
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

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. ***