Summary: | Buttons in ExpandableListIem's expanded list can overlap with highlight depending on font size | ||
---|---|---|---|
Product: | [Frameworks and Libraries] libplasma | Reporter: | toni_rocha <antonioni.rocha> |
Component: | components | Assignee: | Marco Martin <notmart> |
Status: | RESOLVED FIXED | ||
Severity: | minor | CC: | antonioni.rocha, nate, plasma-bugs |
Priority: | NOR | ||
Version: | 5.87.0 | ||
Target Milestone: | --- | ||
Platform: | Neon | ||
OS: | Linux | ||
Latest Commit: | https://invent.kde.org/frameworks/plasma-framework/commit/a61587a08cb400eddd1c036ecbf21b94381883f7 | Version Fixed In: | 5.88 |
Sentry Crash Report: | |||
Attachments: |
Wrong panel design.
Another example. |
Created attachment 142456 [details]
Another example.
(Wrong position = overlapping label and panel.) Cannot reproduce with Noto Sans 10pt or 11pt. What font size are you using? Also are you *sure* you've got Frameworks 5.87? I ask because I fixed an issue like this recently and the fix was supposed to be released in Frameworks 5.87. So it would make sense if you actually had 5.86 and not 5.87. . (In reply to Nate Graham from comment #3) > Cannot reproduce with Noto Sans 10pt or 11pt. What font size are you using? > Also are you *sure* you've got Frameworks 5.87? I ask because I fixed an > issue like this recently and the fix was supposed to be released in > Frameworks 5.87. So it would make sense if you actually had 5.86 and not > 5.87. The version of KDE Framework is 5.87.0, and the link below is a print of my fonts (I like little fonts): https://i.ibb.co/3FZCYhr/Screenshot-20211018-001525.png I now have discovered that the widget "Disk & Devices" has the same problem here. Print below, should I report a new bug? https://i.ibb.co/dgG3yT6/overlapping.jpg Can reproduce with 8pt DejaVu Sans, thanks. I'll look into it. No need to submit a new bug report; the issue in a common component used by multiple applets. (In reply to Nate Graham from comment #8) > No need to submit a new bug report; the issue in a common component used by > multiple applets. Ok, thanks. Git commit a61587a08cb400eddd1c036ecbf21b94381883f7 by Nate Graham. Committed on 18/10/2021 at 19:09. Pushed by ngraham into branch 'master'. ExpandableListItem: Finally fix expanded height calculation permanently After many different approaches to determine the height of the expanded view popup, the simplest and most robust is to do nothing at all and let the implicit size bubble up, which works just fine in this case because the items own implicit heights will automatically take into consideration the font size, and any invisible disabled items won't be added to the implicit height, so the final height calculation will always be correct with any set of items and font size. FIXED-IN: 5.88 M +0 -3 src/declarativeimports/plasmaextracomponents/qml/ExpandableListItem.qml https://invent.kde.org/frameworks/plasma-framework/commit/a61587a08cb400eddd1c036ecbf21b94381883f7 |
Created attachment 142453 [details] Wrong panel design. SUMMARY Error in design of the vault panel in "Configure vault" option when it is focused. STEPS TO REPRODUCE 1. Open the vault panel. 2. Select and expand an existent vault. 3. Focus the "Configure vault" option. The line border is in a wrong position. OBSERVED RESULT When the "Configure vault" option is focused this line border is in a wrong position. EXPECTED RESULT A correct position of the line border when the "Configure vault" option is focused. SOFTWARE/OS VERSIONS Linux/KDE Plasma: (available in About System) KDE Plasma Version: 5.23.0 KDE Frameworks Version: 5.87.0 Qt Version: 5.15.3