D16905: Add new generic sorting icons and rename existing alphabetic sorting icons to -name scheme
TrickyRicky
noreply at phabricator.kde.org
Fri Nov 16 17:32:04 GMT 2018
trickyricky26 planned changes to this revision.
trickyricky26 added a comment.
@emateli
1. You're absolutely right about that, don't know how I missed this. With the correct naming, should the arrows still point to the end with longer lines?
2. They are sorted, but some lines were shortened to make room for the arrow so it could stay in the bottom right. Earlier I had a design with the arrows moved to the top in the ascending version (I have edited the comment to have correct naming):
In D16905#360495 <https://phabricator.kde.org/D16905#360495>, @trickyricky26 wrote:
> This is how the set looks with the arrows above the lines in the ascending versions:
>
> | size | `view-sort` | `view-sort-descending` | `view-sort-ascending` |
> | ----------------- | ----------- | ---------------------- | --------------------- |
> | `16px` | F6427148: view-sort-16.svg.png <https://phabricator.kde.org/F6427148> | F6427145: view-sort-16-ascending.svg.png <https://phabricator.kde.org/F6427145> | F6427205: view-sort-16-descending-alt.svg.png <https://phabricator.kde.org/F6427205> |
> | `22px` and `24px` | F6427153: view-sort-22.svg.png <https://phabricator.kde.org/F6427153> | F6427155: view-sort-22-ascending.svg.png <https://phabricator.kde.org/F6427155> | F6427208: view-sort-22-descending-alt.svg.png <https://phabricator.kde.org/F6427208> |
> | `32px` | F6427177: view-sort-32.svg.png <https://phabricator.kde.org/F6427177> | F6427162: view-sort-32-ascending.svg.png <https://phabricator.kde.org/F6427162> | F6427210: view-sort-32-descending-alt.svg.png <https://phabricator.kde.org/F6427210> |
> |
Do you feel these are less confusing?
REPOSITORY
R266 Breeze Icons
REVISION DETAIL
https://phabricator.kde.org/D16905
To: trickyricky26, #vdg, #breeze, ndavis
Cc: emateli, ndavis, ngraham, kde-frameworks-devel, michaelh, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20181116/dab00fd2/attachment-0001.html>
More information about the Kde-frameworks-devel
mailing list