Bug 486551 - Layout broken on small screen with 200% scaling
Summary: Layout broken on small screen with 200% scaling
Status: RESOLVED MOVED
Alias: None
Product: systemsettings
Classification: Applications
Component: kcm_sddm (show other bugs)
Version: 6.0.4
Platform: Fedora RPMs Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2024-05-04 09:21 UTC by Plata
Modified: 2024-05-13 19:23 UTC (History)
3 users (show)

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


Attachments
screenshot (133.49 KB, image/png)
2024-05-04 09:21 UTC, Plata
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Plata 2024-05-04 09:21:53 UTC
Created attachment 169179 [details]
screenshot

SUMMARY
The preview pictures are rather small and the buttons do not fit on them.

STEPS TO REPRODUCE
1. Use device with small screen.
2. Set scaling to 200%.
3. Open SDDM KCM.

OBSERVED RESULT
see attachment

EXPECTED RESULT
Use bigger pictures such that all buttons fit, e.g. like in the color settings.

SOFTWARE/OS VERSIONS
Linux: Fedora 40 (Kinoite)
KDE Plasma Version: 6.0.4
KDE Frameworks Version: 6.1.0
Qt Version: 6.7.0

ADDITIONAL INFORMATION
There are multiple KCMs with a similar structure which should use the same design. For example, colors, application theme, Plasma theme, system sounds... Some of them only do not face the same issue as the SDDM KCM because they show only 1-2 buttons.
Comment 1 Nate Graham 2024-05-05 02:31:33 UTC
There appears to be a problem with your widget style; the buttons are not supposed to be that wide. Are you by any chance using qqc2-breeze-style on a tablet-style device or something?
Comment 2 Plata 2024-05-05 09:40:00 UTC
That's the reason indeed!

After switching to org.kde.desktop, the buttons look fine. There's still a minor issue because the selection highlight is cut off at the top but that's not important here.

While I can of course now change the style, I'm wondering what the correct/sustainable solution should be.
- Is it possible to change qqc2-breeze-style in a way that it works as expected on tablet (this KCM is the only place where I've noticed a problem so far)? My understanding is that qqc2-breeze-style is intended to be faster, so using desktop style on low power tablet hardware might be problematic.
- If not (and the intention is that Plasma Mobile on tablets uses the desktop style), how can this be more user friendly? I don't think that we can expect users to know and change this on their own after installing a Plasma Mobile package. Should there be a first start wizard like "It seems that you use Plasma Mobile on a tablet, do you want to change the style to ..."? Or should distributions have meta packages for phone and tablet which set the style accordingly?
Comment 3 Nate Graham 2024-05-06 20:36:56 UTC
Regardless of how we untangle the existence of two styles, this is clearly an issue in qqc2-breeze-style. It unfortunately does not seem to have a bugzilla product yet, but it seems to be https://invent.kde.org/plasma/qqc2-breeze-style/-/issues/17.
Comment 4 Bug Janitor Service 2024-05-09 10:30:22 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/qqc2-breeze-style/-/merge_requests/96
Comment 5 Jack Hill 2024-05-13 19:23:47 UTC
Git commit 8a22b7fe6d7286f6b3e06011c026d038b1ffd326 by Jack Hill.
Committed on 09/05/2024 at 10:29.
Pushed by jackh into branch 'master'.

Fix IconOnly Button width with text

Syncs behaviour with qqc2-desktop-style

Some apps may assign a value to text but still have the button as
IconOnly. In this case we do not want to pad the minimum width of the
button.

M  +1    -1    style/qtquickcontrols/Button.qml

https://invent.kde.org/plasma/qqc2-breeze-style/-/commit/8a22b7fe6d7286f6b3e06011c026d038b1ffd326