https://bugs.kde.org/show_bug.cgi?id=388807

Kai Uwe Broulik <k...@privat.broulik.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|                            |https://invent.kde.org/fram
                   |                            |eworks/kiconthemes/commit/e
                   |                            |d30c6c5bad397444f78a70f89b2
                   |                            |f4aec152466c
         Resolution|---                         |FIXED
             Status|ASSIGNED                    |RESOLVED

--- Comment #6 from Kai Uwe Broulik <k...@privat.broulik.de> ---
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+)]

https://invent.kde.org/frameworks/kiconthemes/commit/ed30c6c5bad397444f78a70f89b2f4aec152466c

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to