[Logout screen] Don't underline button when has focus
ClosedPublic

Authored by broulik on Dec 21 2017, 1:13 PM.

Details

Summary

In the logout screen we use activeFocus to determin which action will be triggered.
Here you only have those buttons as controls, tabbing essentially changing it, so having an underline for focus isn't needed and clutters the layout since the opacity already changes depending on your selection.

Test Plan

Underline was added by D9431 but shouldn't be in logout.
If we want to have an underline we really should only have it with focusReason Tab but that would require QQC2 Control as base instead of Item

Diff Detail

Repository
R120 Plasma Workspace
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
broulik created this revision.Dec 21 2017, 1:13 PM
Restricted Application added a project: Plasma. · View Herald TranscriptDec 21 2017, 1:13 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
broulik requested review of this revision.Dec 21 2017, 1:13 PM
davidedmundson accepted this revision.Jan 2 2018, 12:32 PM
This revision is now accepted and ready to land.Jan 2 2018, 12:32 PM
This revision was automatically updated to reflect the committed changes.