Created attachment 109794 [details]
choose icon dialog
See the attached screenshot: The distance between the widgets in the top group box is too high. At first glance it is hard to recognise these widgets as belonging together.
Only when you make the window really really wide though, right?
(In reply to Nate Graham from comment #1)
> Only when you make the window really really wide though, right?
Yes. I stumbled over it like this: 1) Open the icon chooser, 2) make it wider to see more icons, 3) realized that there are options at the top to show other icons, 4) being confused why the distance between corresponding elements is so high.
This is sort of a hard one, because if we make the comboboxes stay with the text when the window is resized, it'll look really weird to have a ton of whitespace between them and the right edge of the window.
As it is now, you have the amount of whitespace in between, and in the middle a heading over this whitespace. :)
Why not right align the contents of top groupbox and maybe also do not expand up to the right side? Or put everything in one row.
A possibly relevant merge request was started @ https://invent.kde.org/frameworks/kiconthemes/-/merge_requests/34
Git commit ed30c6c5bad397444f78a70f89b2f4aec152466c by Kai Uwe Broulik.
Committed on 05/07/2021 at 11:43.
Pushed by broulik into branch 'master'.
[KIconDialog] Revamp UI
This modernizes the UI of KIconDialog that hasn't changed since childhood.
The code is overall tidied up and layout moved into a `.ui` file.
The default size is changed from 4x4 to 6x3 to accommodate more widescreen
monitors and the grid allows for three lines of text since many icon names
are longer than just one line.
Furthermore, the stark separation between "system" and "user" icons is lifted,
and instead the "Browse..." button is always accessible and "Other" is just
another item in the "context" combo box.
Also, a "No icons" placeholder is shown when there are no icons in the
given category or because of the search term input.
Finally, the icons are now loaded on demand as the view scrolls along rather
than loading all thousands of them at once with a slow progress bar.
Related: bug 401408
M +4 -0 src/CMakeLists.txt
M +340 -362 src/kicondialog.cpp
M +1 -9 src/kicondialog.h
A +88 -0 src/kicondialog.ui
M +27 -109 src/kicondialog_p.h
A +55 -0 src/kicondialogmodel_p.h [License: LGPL(v2.0+)]