Differential D24158

use KBusyIndicatorWidget for all busy indication uses
ClosedPublic
Actions

Authored by sitter on Sep 23 2019, 1:26 PM.

Details

Reviewers

broulik

Group Reviewers

Plasma

Commits

R871:871ed0a975a5: use KBusyIndicatorWidget for all busy indication uses

Summary

unlike the pixmapsequence this scales properly and is actually theme color
sensitive (contrast on dark themes) and looks nicely integrated into plasma

Test Plan

triggered all possible busy indicators and they look dandy

Diff Detail

Repository

R871 DrKonqi

Branch

Plasma/5.17

Lint

No Linters Available

Unit

No Unit Test Coverage

Build Status

Buildable 16908
Build 16926: arc lint + arc unit

sitter created this revision.Sep 23 2019, 1:26 PM

Restricted Application added a project: Plasma. · View Herald TranscriptSep 23 2019, 1:26 PM

Restricted Application added a subscriber: plasma-devel. · View Herald Transcript

sitter requested review of this revision.Sep 23 2019, 1:26 PM

Harbormaster completed remote builds in B16908: Diff 66661.Sep 23 2019, 1:26 PM

broulik accepted this revision.Sep 25 2019, 10:33 AM

This revision is now accepted and ready to land.Sep 25 2019, 10:33 AM

Closed by commit R871:871ed0a975a5: use KBusyIndicatorWidget for all busy indication uses (authored by sitter). · Explain WhySep 25 2019, 11:25 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

			Path	Packages
M			CMakeLists.txt (2 lines)
M			src/bugzillaintegration/ui/assistantpage_bugzilla_version.ui (22 lines)
M			src/statuswidget.h (4 lines)
M			src/statuswidget.cpp (6 lines)