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

Nate Graham <n...@kde.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Version Fixed In|                            |5.61
         Resolution|---                         |FIXED
      Latest Commit|                            |https://commits.kde.org/bre
                   |                            |eze-icons/74c4155ff3d703e29
                   |                            |9a3d86e7d3d030d2c6151e1
             Status|ASSIGNED                    |RESOLVED

--- Comment #6 from Nate Graham <n...@kde.org> ---
Git commit 74c4155ff3d703e299a3d86e7d3d030d2c6151e1 by Nate Graham.
Committed on 18/07/2019 at 19:12.
Pushed by ngraham into branch 'master'.

Use clockwise arrows for *refresh* and update-* icons

Summary:
Right now we are inconsistent with the arrow directionality for our circular
icons:
some use clockwise arrows, and some use counterclockwise.

This patch standardizes everything on the clockwise direction. This not only
makes
everything nice and consistent, but it also fixes the busy indictor spinner
looking
weird (it rotates clockwise, like all opther busy spinners we've ever seen in
any UI
toolkit).
FIXED-IN: 5.61

Test Plan:
{F7046657}

{F7046658}

{F7046660}

{F7046661}

{F7046668}

{F7046669}

{F7046670}

The eagle-eyed reviewer will note that the different icons also have different
styles of arrows and their rotational offset is also different, but if that's a
pre-existing issue that can be done in another patch, if it's also something we
want to unify.

Reviewers: #vdg, ndavis

Reviewed By: #vdg, ndavis

Subscribers: kde-frameworks-devel

Tags: #frameworks

Differential Revision: https://phabricator.kde.org/D22534

M  +6    -11   icons-dark/actions/16/view-refresh.svg
M  +6    -15   icons-dark/actions/22/view-refresh.svg
M  +6    -11   icons-dark/actions/24/view-refresh.svg
M  +6    -11   icons-dark/actions/32/view-refresh.svg
M  +42   -1    icons-dark/preferences/32/yast-update.svg
M  +7    -16   icons-dark/status/22/update-high.svg
M  +7    -21   icons-dark/status/22/update-low.svg
M  +7    -22   icons-dark/status/22/update-medium.svg
M  +6    -11   icons-dark/status/22/update-none.svg
M  +6    -11   icons/actions/16/view-refresh.svg
M  +6    -15   icons/actions/22/view-refresh.svg
M  +6    -11   icons/actions/24/view-refresh.svg
M  +6    -11   icons/actions/32/view-refresh.svg
M  +42   -1    icons/preferences/32/yast-update.svg
M  +7    -16   icons/status/22/update-high.svg
M  +7    -21   icons/status/22/update-low.svg
M  +7    -22   icons/status/22/update-medium.svg
M  +6    -11   icons/status/22/update-none.svg

https://commits.kde.org/breeze-icons/74c4155ff3d703e299a3d86e7d3d030d2c6151e1

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

Reply via email to