D10098: Fix drag cursor
Kai Uwe Broulik
noreply at phabricator.kde.org
Thu Jan 25 16:29:38 UTC 2018
broulik created this revision.
broulik added reviewers: Plasma, drosca, VDG.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
broulik requested review of this revision.
REVISION SUMMARY
The `MouseArea` for allowing middle click to mute overrules the `cursorShape` of the draggable area.
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.
TEST PLAN
- Verified hand cursor does not show up when it cannot be dragged (device list or only one device in application list)
- Verified that left pressing mouse changes cursor to closed hand on drag area
- Verified middle pressing mouse does not change cursor on drag area
- Verified drag and drop to send applications to other audio devices still works
- Verified that middle click on the list item still mutes/unmutes
- Verified that middle click on the icon still mutes/unmutes
- Verified that layout is correct with layout mirroring
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.
Note the little dead area
F5674568: Screenshot_20180125_172806.png <https://phabricator.kde.org/F5674568>
REPOSITORY
R115 Plasma Audio Volume Applet
REVISION DETAIL
https://phabricator.kde.org/D10098
AFFECTED FILES
applet/contents/ui/ListItemBase.qml
To: broulik, #plasma, drosca, #vdg
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/20180125/600c769c/attachment.html>
More information about the Plasma-devel
mailing list