<table><tr><td style="">broulik created this revision.<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/D4959" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>I often end up accidentally opening the folder in a file manager because especially when FolderView is placed on a desktop, the hit area for the label spans the entire width and a significantly taller area than the label's text height actually is.</p>
<p>There's also no visual cue that clicking the heading would do anything. Turn it into a "hyperlink" by underlining it on hover and using the PointingHandCursor.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Tested with normal labels as well as super long ones (navigating through lots of sub folders) and the MouseArea position always covers just the label's text. It also underlines just fine on hover.</p>
<p><a href="https://phabricator.kde.org/F2749878" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;" rel="noreferrer">F2749878: Screenshot_20170306_143636.png</a></p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D4959" rel="noreferrer">https://phabricator.kde.org/D4959</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>containments/desktop/package/contents/ui/FolderViewLayer.qml</div></div></div><br /><div><strong>To: </strong>broulik, Plasma, hein<br /><strong>Cc: </strong>plasma-devel, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol<br /></div>