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

[Kai Uwe Broulik]

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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20171221/f822f4b1/attachment.html>