broulik created this revision.
broulik added a reviewer: Plasma.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.

REVISION 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 https://phabricator.kde.org/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`

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D9452

AFFECTED FILES
  lookandfeel/contents/components/ActionButton.qml
  lookandfeel/contents/logout/LogoutButton.qml

To: broulik, #plasma
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, 
abetts, sebas, apol, mart

Reply via email to