Fix mouse events being withheld from the scrollbar.
ClosedPublic
Actions

Authored by hein on Nov 19 2018, 6:43 AM.

Details

Reviewers

ngraham
davidedmundson

Commits

Summary

Fixes regression introduced in 48b32259e73f.

Thanks to ngraham for pointing out the bug.

Repository

R119 Plasma Desktop

Lint

Automatic diff as part of commit; lint not applicable.

Unit

Automatic diff as part of commit; unit tests not applicable.

Restricted Application added a subscriber: plasma-devel. · View Herald Transcript

hein requested review of this revision.Nov 19 2018, 6:43 AM

Thanks Eike! Can confirm that this fixes the issue, and the code change looks eminently sane.

This revision is now accepted and ready to land.Nov 19 2018, 3:18 PM

If you run in arabic/RTL the scrollbars will be on the left.

If you add:
anchors.left: parent.left

It should get automatically flipped to the right anchors and continue to work. If you don't specify anything, it'll stay on the left.

This revision now requires changes to proceed.Nov 19 2018, 6:01 PM

Fix RTL bug

This revision is now accepted and ready to land.Nov 21 2018, 1:30 PM

This revision was automatically updated to reflect the committed changes.

			Path	Packages
M			applets/kickoff/package/contents/ui/KickoffListView.qml (5 lines)

Diff	ID	Base	Description	Created	Lint	Unit
Base			Base
Diff 1	45777	94e9d5e		Nov 19 2018, 6:43 AM	★	★
Diff 2	45864	7caafc8	Fix RTL bug	Nov 20 2018, 6:42 AM	★	★
Diff 3	45948	dfce500	R119:b664aa9c055b15ee6b3fa6b9877ac702e94c5dc0	Nov 20 2018, 7:03 AM	★	★