Bug 457320 - Panel resize box breaks with large textedit svgs
Summary: Panel resize box breaks with large textedit svgs
Status: RESOLVED FIXED
Alias: None
Product: frameworks-ksvg
Classification: Frameworks and Libraries
Component: General (show other bugs)
Version: unspecified
Platform: Neon Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-07-30 21:58 UTC by doncbugs
Modified: 2023-12-13 11:42 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
broken panel height (18.73 KB, image/png)
2022-07-30 21:58 UTC, doncbugs
Details
A plasma theme with large margins for text edit svg (91.06 KB, application/zip)
2022-08-02 00:59 UTC, doncbugs
Details

Note You need to log in before you can comment on or make changes to this bug.
Description doncbugs 2022-07-30 21:58:50 UTC
Created attachment 151013 [details]
broken panel height

SUMMARY
After creating a new theme, I observed that the textedit box that shows the panel height had broken and now only appeared under the "-" minus button. It appears to be due to the theme having very large margins at ~10px.


STEPS TO REPRODUCE
1. Set the plasma style to lightly-plasma https://github.com/doncsugar/lightly-plasma
2. Right-click the panel and enter edit mode

OBSERVED RESULT
The panel height box only shows the textedit.svg under the minus sign

EXPECTED RESULT
The panel height box should cover both the minus, actual text field, and plus buttons.

Operating System: KDE neon Testing Edition
KDE Plasma Version: 5.25.3
KDE Frameworks Version: 5.97.0
Qt Version: 5.15.5
Kernel Version: 5.15.0-41-generic (64-bit)
Graphics Platform: X11
Comment 1 Nate Graham 2022-08-01 19:44:22 UTC
Can you attach the Plasma theme you're developing that can be used to reproduce the issue?
Comment 2 doncbugs 2022-08-02 00:59:17 UTC
Created attachment 151063 [details]
A plasma theme with large margins for text edit svg

Sure
Comment 3 ratijas 2022-08-02 05:42:57 UTC
(on a side note, that's some cool looking Minus button!)
Comment 4 doncbugs 2022-08-03 23:51:17 UTC
(In reply to ratijas from comment #3)
> (on a side note, that's some cool looking Minus button!)

Ha ha, yeah, it was such an elegant way of failing that I didn't catch it until I asked myself why there was only one button
Comment 5 Marco Martin 2023-12-13 11:42:20 UTC
Git commit 13e2215cc7e51429640b7c9b84c475a3a3c9827e by Marco Martin.
Committed on 13/12/2023 at 12:40.
Pushed by mart into branch 'master'.

Prevent background loader to resize itself

the loader will try  to resize itself to the implicit size of
contronts on load. work around that with anchors

M  +9    -0    src/declarativeimports/plasmacomponents3/SpinBox.qml

https://invent.kde.org/plasma/libplasma/-/commit/13e2215cc7e51429640b7c9b84c475a3a3c9827e