Bug 424182 - Small icons in the sidebar of the System Settings after update
Summary: Small icons in the sidebar of the System Settings after update
Status: RESOLVED INTENTIONAL
Alias: None
Product: systemsettings
Classification: Applications
Component: sidebarview (show other bugs)
Version: 5.19.3
Platform: Other Linux
: NOR minor
Target Milestone: ---
Assignee: Marco Martin
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-07-14 08:25 UTC by popov895
Modified: 2020-07-15 23:58 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
Sidebar (88.41 KB, image/jpeg)
2020-07-14 08:25 UTC, popov895
Details

Note You need to log in before you can comment on or make changes to this bug.
Description popov895 2020-07-14 08:25:20 UTC
Created attachment 130106 [details]
Sidebar

SUMMARY

After one of the latest updates the icons in the sidebar of the System Settings have become small and the search field has stopped displaying a blue frame when it has input focus. It seems that it appeared after updating to KDE Frameworks 5.72 but I'm not sure. Also I'm not sure if this is a bug or feature, but with larger icons it looks better for me: now the height of the item is less than the height of the header and it's become more difficult to select the desired item with your finger from the touch screen.

SOFTWARE/OS VERSIONS

Linux/KDE Plasma: KDE neon 5.19 User Edition
KDE Plasma Version: 5.19.3
KDE Frameworks Version: 5.72.0
Qt Version: 5.14.2
Comment 1 Nate Graham 2020-07-14 15:55:58 UTC
Yeah, this is intentional. We lived with the large icons for a while but in the end decided they were just too big and the sidebar list was too tall as a result. It might have been different if there were fewer items in the sidebar, but in its current state, we just felt like it made the list too tall which caused excessive scrolling.

It was a compromise position and I do understand your viewpoint: taller items are indeed more touch-friendly, and also taller items means that the icons can be displayed at their native size of 32px which means no blurry downscaling to 22px, which we're doing now.

We can continue discussing it in the VDG and are open to suggestions and proposed changes; feel free to participate!

https://community.kde.org/Get_Involved/design
Comment 2 Nate Graham 2020-07-14 15:56:32 UTC
See also https://phabricator.kde.org/T12789
Comment 3 popov895 2020-07-14 18:29:31 UTC
Thanks for the explanation. And what about the missing focus frame in the search field?
Comment 4 Nate Graham 2020-07-14 18:34:04 UTC
Oh right, I forgot about that. This is why it's good to report only one thing per bug report. :) I can't reproduce the issue. Can you file a separate bug report about it and we'll continue the investigation there?
Comment 6 Nate Graham 2020-07-15 20:26:08 UTC
Weird, if that broke it, then I guess the text field is doing something strange internally since that was supposed to be a buxfix.

Can you tell me:
- What is the resolution of your screen?
- Whether or not you have set a scale factor in the KSccreen KCM, and if so, what it is?
- Whether or not you have manually changed the Force Fonts DPI setting in the Fonts KCM?
Comment 7 popov895 2020-07-15 20:50:41 UTC
This commit also makes icons in the sidebar small (it affects System Settings, KInfoCenter and Discover). I guess that the problem is that I use "Force font DPI" only (from "System Settings" > "Fonts" > "Fonts"). If I switch to use global DPI scaling (from "System Settings" > "Display and Monitor" > "Display Configuration"), the issue no longer exists. But I don't like current state of DPI scaling because it makes icons blurry and widgets ugly.
Comment 8 Nate Graham 2020-07-15 23:58:18 UTC
I had a feeling. :)

You probably already know this, but that's not a supported way of doing full-screen scaling, for precisely this reason: it makes various things break in weird ways.

The scaling slider in the KScreen KCM is the only officially supported method of doing significant scaling. To scale things by very small amounts, you should just adjust the font size instead.

If either of these supported methods produce visual bugs, please feel free to file bug reports, as we're quite interested in making them better so people aren't hesitant to use them instead of the Force Fonts DPI spinbox. :)