Rotate busy indicator clockwise
ClosedPublic

Authored by GB_2 on Dec 8 2018, 7:33 PM.

Details

Summary

Most loading/refresh icons have a circular arrow pointing clockwise, but the busy indicator is currently rotated counterclockwise. This changes it to rotate clockwise.
How most loading/refresh icons look like:

Test Plan

Use an icon theme with this kind of loading/refresh icon (such as "Papirus", "Adwaita" or "Antü") , open Discover and click on for example "Plasma Addons". At the bottom you can then see the busy indicator.

Diff Detail

Repository
R858 Qt Quick Controls 2: Desktop Style
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
GB_2 created this revision.Dec 8 2018, 7:33 PM
Restricted Application added a project: Plasma. · View Herald TranscriptDec 8 2018, 7:33 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
GB_2 requested review of this revision.Dec 8 2018, 7:33 PM
ndavis added a subscriber: ndavis.Dec 8 2018, 8:05 PM

Makes sense to me. Time moves clockwise on a clock.

broulik added a subscriber: broulik.Dec 8 2018, 8:35 PM

Did we ever have a busy indicator that rotated counter-clockwise? +1

davidedmundson accepted this revision.Dec 8 2018, 10:55 PM
davidedmundson added a subscriber: davidedmundson.

Plasma components also goes that way

This revision is now accepted and ready to land.Dec 8 2018, 10:55 PM
ndavis accepted this revision.Dec 9 2018, 1:18 AM
ngraham added a subscriber: ngraham.Dec 9 2018, 5:36 PM

Yep, the PC one already does go clockwise

This revision was automatically updated to reflect the committed changes.