Bug 448243

Summary:	Scrollable applets' scrollbars have a dead zone on the left side that doesn't accept clicks
Product:	[Frameworks and Libraries] libplasma	Reporter:	Fushan Wen <qydwhotmail>
Component:	components	Assignee:	Marco Martin <notmart>
Status:	RESOLVED FIXED
Severity:	normal	CC:	carl, nate, noahadvs, plasma-bugs-null
Priority:	NOR	Keywords:	regression
Version First Reported In:	5.90.0
Target Milestone:	---
Platform:	openSUSE
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:

Description Fushan Wen 2022-01-11 02:35:02 UTC

SUMMARY
I have to double/triple/whatever times click the scrollbar to drag it in the notification applet. The same bug happens in the Klipper applet.


STEPS TO REPRODUCE
1. Open the notification applet in the system tray.
2. Try to drag the scrollbar

OBSERVED RESULT
The scrollbar can't be dragged immediately until it's clicked multiple times.

EXPECTED RESULT
Normal scrollbar behavior.

SOFTWARE/OS VERSIONS
Information for package plasma5-workspace:
------------------------------------------
Repository     : unstable-kde-frameworks
Name           : plasma5-workspace
Version        : 5.23.80git.20220110T172641~083ef23af-ku.113.1
Arch           : x86_64
Vendor         : obs://build.opensuse.org/KDE:Unstable
Installed Size : 27.0 MiB
Installed      : Yes
Status         : out-of-date (version 5.23.80git.20220108T014216~0d0b71307-ku.111.1 installed)
Source package : plasma5-workspace-5.23.80git.20220110T172641~083ef23af-ku.113.1.src
Summary        : The KDE Plasma Workspace Components

Information for package plasma-framework-components:
----------------------------------------------------
Repository     : unstable-kde-frameworks
Name           : plasma-framework-components
Version        : 5.91.0git.20220110T092209~ecf0cc369-ku.2.1
Arch           : x86_64
Vendor         : obs://build.opensuse.org/KDE:Unstable
Installed Size : 1.2 MiB
Installed      : Yes
Status         : out-of-date (version 5.89.0git.20220108T114247~5f0c55e39-ku.14.1 installed)
Source package : plasma-framework-5.91.0git.20220110T092209~ecf0cc369-ku.2.1.src
Summary        : Plasma QML components

Comment 1 Nate Graham 2022-01-11 16:35:14 UTC

I think I see what the problem is: the right side of the scrollbar accept clicks, but the left side does not. Can you reproduce that?

Comment 2 Noah Davis 2022-01-11 17:28:57 UTC

(In reply to Nate Graham from comment #1)
> I think I see what the problem is: the right side of the scrollbar accept
> clicks, but the left side does not. Can you reproduce that?

I can reproduce that

Comment 3 Nate Graham 2022-01-11 17:34:53 UTC

Cool. Seems like it affects all recently ported applets but not the Widget Explorer. Might be an issue with the margin hint thing?

Comment 4 Bug Janitor Service 2022-01-11 20:51:44 UTC

A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/1347

Comment 5 Carl Schwan 2022-01-11 21:24:30 UTC

Git commit f02adab757df42e5bcaf404b19b1ae197a35c28b by Carl Schwan.
Committed on 11/01/2022 at 21:11.
Pushed by carlschwan into branch 'master'.

Fix scrolling in ScrollViews

M  +3    -4    applets/clipboard/contents/ui/Menu.qml

https://invent.kde.org/plasma/plasma-workspace/commit/f02adab757df42e5bcaf404b19b1ae197a35c28b

Comment 6 Bug Janitor Service 2022-01-11 21:35:43 UTC

A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/1348

Comment 7 Nate Graham 2022-01-11 22:08:22 UTC

Git commit 8d3ec2f89810b55839824cc4d047391a5381cc8e by Nate Graham, on behalf of Carl Schwan.
Committed on 11/01/2022 at 22:08.
Pushed by ngraham into branch 'master'.

[applets]: Fix scrolling in ScrollViews

M  +1    -11   applets/batterymonitor/package/contents/ui/PopupDialog.qml
M  +3    -4    applets/devicenotifier/package/contents/ui/FullRepresentation.qml
M  +4    -5    applets/notifications/package/contents/ui/FullRepresentation.qml

https://invent.kde.org/plasma/plasma-workspace/commit/8d3ec2f89810b55839824cc4d047391a5381cc8e

Comment 8 Bug Janitor Service 2022-01-11 22:21:09 UTC

A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-vault/-/merge_requests/20

Comment 9 Nate Graham 2022-01-11 22:24:18 UTC

Git commit 0c19fa850ac869f8b6bb6edd3ae573c4e1cb434a by Nate Graham, on behalf of Carl Schwan.
Committed on 11/01/2022 at 22:24.
Pushed by ngraham into branch 'master'.

[applet]: Fix scrolling in ScrollViews

M  +3    -4    plasma/package/contents/ui/main.qml

https://invent.kde.org/plasma/plasma-vault/commit/0c19fa850ac869f8b6bb6edd3ae573c4e1cb434a

Comment 10 Nate Graham 2022-01-11 22:31:22 UTC

That should be all of them. Thank you Carl!

Comment 11 Fushan Wen 2022-01-12 01:01:33 UTC

Thank you Carl and Nate very much!