[applets/systemtray] Remove unused imports
ClosedPublic

Authored by filipf on Oct 13 2019, 5:15 PM.

Diff Detail

Repository
R120 Plasma Workspace
Branch
systray-remove-unused-imports (branched from master)
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 17657
Build 17675: arc lint + arc unit
filipf created this revision.Oct 13 2019, 5:15 PM
Restricted Application added a project: Plasma. · View Herald TranscriptOct 13 2019, 5:15 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
filipf requested review of this revision.Oct 13 2019, 5:15 PM
filipf updated this revision to Diff 67857.Oct 13 2019, 5:19 PM

remove some more

ngraham accepted this revision.EditedOct 13 2019, 6:23 PM
ngraham added a subscriber: ngraham.

Could also port entirely away from PlasmaComponents and PlasmaExtras without too much effort. I suppose that's best done in a separate patch though.

This revision is now accepted and ready to land.Oct 13 2019, 6:23 PM
This revision was automatically updated to reflect the committed changes.

Could also port entirely away from PlasmaComponents and PlasmaExtras without too much effort. I suppose that's best done in a separate patch though.

We would use QQC2 for labels and Kirigami for headings?

broulik added inline comments.
applets/systemtray/package/contents/ui/PlasmoidPopupsContainer.qml
31

units effectively comes from PlasmaCore

filipf marked an inline comment as done.Oct 14 2019, 12:50 PM
filipf added inline comments.
applets/systemtray/package/contents/ui/PlasmoidPopupsContainer.qml
31

Could also port entirely away from PlasmaComponents and PlasmaExtras without too much effort. I suppose that's best done in a separate patch though.

We would use QQC2 for labels and Kirigami for headings?

Yep.