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

Authored by ngraham on Jul 18 2019, 6:43 PM.

Details

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).

BUG: 409914
FIXED-IN: 5.61

Test Plan

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.

Diff Detail

Repository
R266 Breeze Icons
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
ngraham created this revision.Jul 18 2019, 6:43 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptJul 18 2019, 6:43 PM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
ngraham requested review of this revision.Jul 18 2019, 6:43 PM
ngraham edited the test plan for this revision. (Show Details)Jul 18 2019, 6:46 PM
ngraham edited the summary of this revision. (Show Details)
ndavis accepted this revision.EditedJul 18 2019, 7:07 PM

Looks good!

This revision is now accepted and ready to land.Jul 18 2019, 7:07 PM
This revision was automatically updated to reflect the committed changes.