Commit 95a8a620 authored by Marco Martin's avatar Marco Martin

fix behavior of scrollable systray popups

Summary:
it's very rare that the icon list in the hidden area
of the systray is long enough to get a scrollbar, but
when it does it had a couple of issues:
sometimes it gets an horizontal scrollbar too and
the wheel events gets eaten:
always disable the horizontal scrollbar and
ignore wheel events over the label.
wheel events over the icon are still managed by
the applet (so still possible to for instance
adjusting the volume on pa applet)

BUG:372050

Test Plan:
hidden all the cions to enable scrollbar: wheel over
the text scrolls the view, over the icon adjusts the volume

Reviewers: #plasma, davidedmundson

Reviewed By: #plasma, davidedmundson

Subscribers: davidedmundson, plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D3263
parent 9ab5f305
......@@ -32,6 +32,7 @@ PlasmaExtras.ScrollArea {
property alias layout: hiddenTasksColumn
//Useful to align stuff to the column of icons, both in expanded and shrink modes
property int iconColumnWidth: root.hiddenItemSize + highlight.marginHints.left + highlight.marginHints.right
horizontalScrollBarPolicy: Qt.ScrollBarAlwaysOff
verticalScrollBarPolicy: activeApplet ? Qt.ScrollBarAlwaysOff : Qt.ScrollBarAsNeeded
Flickable {
......
......@@ -102,7 +102,12 @@ PlasmaCore.ToolTipArea {
abstractItem.hideToolTip()
abstractItem.clicked(mouse)
}
onWheel: abstractItem.wheel(wheel)
onWheel: {
abstractItem.wheel(wheel);
//Don't accept the event in order to make the scrolling by mouse wheel working
//for the parent scrollview this icon is in.
wheel.accepted = false;
}
}
PlasmaComponents.Label {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment