| Summary: | Unnecessary extra padding in Touchpad KCM | ||
|---|---|---|---|
| Product: | [Applications] systemsettings | Reporter: | yellow.dog625 |
| Component: | kcm_touchpad | Assignee: | Plasma Bugs List <plasma-bugs> |
| Status: | RESOLVED FIXED | ||
| Severity: | minor | CC: | 4wy78uwh, natalie_clarius, nate |
| Priority: | NOR | ||
| Version: | 6.1.90 | ||
| Target Milestone: | --- | ||
| Platform: | Other | ||
| OS: | Linux | ||
| Latest Commit: | https://invent.kde.org/plasma/plasma-desktop/-/commit/554018d694075ae454c858d8fa139c1f2e24ab64 | Version Fixed In: | 6.2.1 |
| Sentry Crash Report: | |||
| Attachments: | screenshot | ||
It's actually caused by the whole KCM having an invisible extra set of padding on all sides. You can see further evidence of this in the fast that the scrollbar is not centered in its track, and its separator line doesn't touch the top and bottom of the window. Probably fixable, but I haven't figured out how yet. Aha, found it. Patch incoming. A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-desktop/-/merge_requests/2569 (In reply to Nate Graham from comment #2) > Aha, found it. Patch incoming. Thank you! Git commit af302b741a12b33653162bc2700e866397c2ef12 by Nate Graham. Committed on 11/10/2024 at 04:26. Pushed by ngraham into branch 'master'. kcms/touchpad: remove extra padding from QtWidgets wrapper layout This KCM wraps the QML-provided content in a QtWidgets-provided box layout. Those include 6px padding by default, but we don't want it here since the QML code brings its own. Let's turn it off. FIXED-IN: 6.2.1 M +1 -0 kcms/touchpad/kcm.cpp https://invent.kde.org/plasma/plasma-desktop/-/commit/af302b741a12b33653162bc2700e866397c2ef12 Git commit 554018d694075ae454c858d8fa139c1f2e24ab64 by Nate Graham. Committed on 11/10/2024 at 20:28. Pushed by ngraham into branch 'Plasma/6.2'. kcms/touchpad: remove extra padding from QtWidgets wrapper layout This KCM wraps the QML-provided content in a QtWidgets-provided box layout. Those include 6px padding by default, but we don't want it here since the QML code brings its own. Let's turn it off. FIXED-IN: 6.2.1 (cherry picked from commit af302b741a12b33653162bc2700e866397c2ef12) Co-authored-by: Nate Graham <nate@kde.org> M +1 -0 kcms/touchpad/kcm.cpp https://invent.kde.org/plasma/plasma-desktop/-/commit/554018d694075ae454c858d8fa139c1f2e24ab64 |
Created attachment 174336 [details] screenshot SUMMARY The top padding in the Touchpad KCM is excessively large compared to other KCMs in the KDE System Settings.