Add BusyIndicator to the styled controls list
ClosedPublic

Authored by Kanedias on Jan 10 2018, 9:05 PM.

Details

Summary
  • Use breeze view-refresh icon
  • Use appear/fade animation

I'm not sure what can be improved here tbh, any suggestions are welcome.
Qt 5.10 includes PathArc that could make rotation more smooth and
remove breeze icon dependency but for now that's all I managed to do.

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.
Kanedias requested review of this revision.Jan 10 2018, 9:05 PM
Kanedias created this revision.
Kanedias updated this revision to Diff 25125.Jan 10 2018, 9:10 PM

Fix my mail in licensing plate

mart accepted this revision.Jan 11 2018, 9:43 AM

I'm fine with it, and can go in right now, tough I think I would prefer an official busy indicator dedicated icon from the VDG, if possible

This revision is now accepted and ready to land.Jan 11 2018, 9:43 AM
This revision was automatically updated to reflect the committed changes.
Restricted Application added a project: Plasma. · View Herald TranscriptJan 11 2018, 7:10 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript

https://api.kde.org/frameworks/index.html says:

Tier 1 frameworks depend only on Qt (and possibly a small number of other third-party libraries), so can easily be used by any Qt-based project.

Can tier 1 framework depend on other tier 1 framework?

Can tier 1 framework depend on other tier 1 framework?

Not sure, but qqc2-desktop-style used Kirigami units before this change so we depended on it already.