<table><tr><td style="">filipf created this revision.<br />Herald added a project: Frameworks.<br />Herald added a subscriber: kde-frameworks-devel.<br />filipf requested review of this revision.
</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/D20977">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Because the shortcut button has text inside it, its height is increased, which leaves the clear button shorter. We want the clear button to follow the shortcut's button height and to then readjust its width so it's a square.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R296 KDeclarative</div></div></div><br /><div><strong>BRANCH</strong><div><div>same-button-height (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D20977">https://phabricator.kde.org/D20977</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/qmlcontrols/kquickcontrols/KeySequenceItem.qml</div></div></div><br /><div><strong>To: </strong>filipf<br /><strong>Cc: </strong>kde-frameworks-devel, michaelh, ngraham, bruns<br /></div>