<table><tr><td style="">broulik created this revision.<br />broulik added reviewers: Plasma, drosca, VDG.<br />Restricted Application added a project: Plasma.<br />Restricted Application added a subscriber: plasma-devel.<br />broulik 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/D10098" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>The <tt style="background: #ebebeb; font-size: 13px;">MouseArea</tt> for allowing middle click to mute overrules the <tt style="background: #ebebeb; font-size: 13px;">cursorShape</tt> of the draggable area.<br />
By moving the middle click area to the right of the icon, we avoid this and can change the cursor to an open hand when hovering the icon to indicate something draggable. The drag area can still be middle clicked to mute.</p></div></div><br /><div><strong>TEST PLAN</strong><div><ul class="remarkup-list">
<li class="remarkup-list-item">Verified hand cursor does not show up when it cannot be dragged (device list or only one device in application list)</li>
<li class="remarkup-list-item">Verified that left pressing mouse changes cursor to closed hand on drag area</li>
<li class="remarkup-list-item">Verified middle pressing mouse does not change cursor on drag area</li>
<li class="remarkup-list-item">Verified drag and drop to send applications to other audio devices still works</li>
<li class="remarkup-list-item">Verified that middle click on the list item still mutes/unmutes</li>
<li class="remarkup-list-item">Verified that middle click on the icon still mutes/unmutes</li>
<li class="remarkup-list-item">Verified that layout is correct with layout mirroring</li>
</ul>

<p>There's now a little gap above and below the icon where you can no longer middle click to mute/unmute but that's negligible imho as there's nothing visually there.</p>

<p>Note the little dead area<br />
<a href="https://phabricator.kde.org/F5674568" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">F5674568: Screenshot_20180125_172806.png</a></p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R115 Plasma Audio Volume Applet</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D10098" rel="noreferrer">https://phabricator.kde.org/D10098</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>applet/contents/ui/ListItemBase.qml</div></div></div><br /><div><strong>To: </strong>broulik, Plasma, drosca, VDG<br /><strong>Cc: </strong>plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>