<table><tr><td style="">broulik created this revision.<br />broulik added a reviewer: Plasma.<br />Restricted Application added a project: Plasma.<br />Restricted Application added a subscriber: plasma-devel.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D9452" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>In the logout screen we use <tt style="background: #ebebeb; font-size: 13px;">activeFocus</tt> to determin which action will be triggered.<br />
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.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Underline was added by <a href="https://phabricator.kde.org/D9431" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: line-through;" rel="noreferrer">D9431</a> but shouldn't be in logout.<br />
If we want to have an underline we really should only have it with <tt style="background: #ebebeb; font-size: 13px;">focusReason</tt> Tab but that would require QQC2 <tt style="background: #ebebeb; font-size: 13px;">Control</tt> as base instead of <tt style="background: #ebebeb; font-size: 13px;">Item</tt></p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R120 Plasma Workspace</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D9452" rel="noreferrer">https://phabricator.kde.org/D9452</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>lookandfeel/contents/components/ActionButton.qml<br />
lookandfeel/contents/logout/LogoutButton.qml</div></div></div><br /><div><strong>To: </strong>broulik, Plasma<br /><strong>Cc: </strong>plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>