Bug 443406 - ScrollView is draggable with a mouse
Summary: ScrollView is draggable with a mouse
Status: RESOLVED FIXED
Alias: None
Product: frameworks-qqc2-desktop-style
Classification: Frameworks and Libraries
Component: general (show other bugs)
Version: 5.86.0
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Marco Martin
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-10-06 17:06 UTC by Derek Christ
Modified: 2023-01-20 22:08 UTC (History)
9 users (show)

See Also:
Latest Commit: https://invent.kde.org/frameworks/kirigami/-/commit/f6ca218607ff7e5d5066eb3224154c3256cb9516
Version Fixed In: 5.90
Sentry Crash Report:

Attachments
Example (142.74 KB, video/mp4)
2021-10-06 17:06 UTC, Derek Christ 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Derek Christ 2021-10-06 17:06:16 UTC 
Created attachment 142213 [details]
Example

SUMMARY
It's possible to drag the items that are displayed in the left bar in every config dialog (Kirigami only) when dragging on text. 

STEPS TO REPRODUCE
1. Open a Kirigami config dialog (for example the settings of the kickoff menu)
2. Click and drag exactly on text in the left settings bar

OBSERVED RESULT
Is flickable

EXPECTED RESULT
Should not be flickable

SOFTWARE/OS VERSIONS
Operating System: openSUSE Tumbleweed 20211001
KDE Plasma Version: 5.22.5
KDE Frameworks Version: 5.86.0
Qt Version: 5.15.2

ADDITIONAL INFORMATION
I'm sorry if this isn't the correct place to put this bug report.
Comment 1 Nate Graham 2021-10-06 19:50:58 UTC 
Actually an issue in AppletConfiguration.qml; nothing to do with Kirigami.
Comment 2 Nate Graham 2021-10-06 19:58:19 UTC 
Actually it's deeper; moving to our ScrollView styling. If it's not possible to fix there, it might require a fix in Qt itself to expose the needed properties on ScrollView itself. This is possible to override when the ScrollView has a Flickable in it, since Flickable exposes these properties. But ScrollView does not.
Comment 3 Nate Graham 2022-01-06 17:16:19 UTC 
This looks to have been fixed with https://invent.kde.org/frameworks/kirigami/-/commit/f6ca218607ff7e5d5066eb3224154c3256cb9516!
Comment 4 Pawel 2023-01-19 22:52:30 UTC 
I can reproduce in Plasma 5.27 beta on Arch Linux.

Operating System: Arch Linux 
KDE Plasma Version: 5.26.90
KDE Frameworks Version: 5.102.0
Qt Version: 5.15.8
Kernel Version: 6.1.7-arch1-1 (64-bit)
Graphics Platform: Wayland

I understand the fix should be included in this release?
Comment 5 Oliver Beard 2023-01-20 05:31:52 UTC 
(In reply to Pawel from comment #4)
> I understand the fix should be included in this release?

Long ago, in Frameworks 5.91 (Currently 5.102). Perhaps it was broken again, or Nate was mistaken. I'm not sure if it would be more appropriate to re-open the bug or create a new bug.

I can reproduce it, but crucially not on every drag. I can't figure out why only some drags would produce the effect.
Comment 6 Naxdy 2023-01-20 07:37:13 UTC 
(In reply to Oliver Beard from comment #5)
> (In reply to Pawel from comment #4)
> > I understand the fix should be included in this release?
> 
> Long ago, in Frameworks 5.91 (Currently 5.102). Perhaps it was broken again,
> or Nate was mistaken. I'm not sure if it would be more appropriate to
> re-open the bug or create a new bug.
> 
> I can reproduce it, but crucially not on every drag. I can't figure out why
> only some drags would produce the effect.

Apparently it's only an issue if you drag from the label. Grab any other part of the element and it won't happen.
Comment 7 Derek Christ 2023-01-20 12:33:29 UTC 
Can reproduce, lets reopen.
Comment 8 Nate Graham 2023-01-20 22:08:50 UTC 
Ah, if it's only on the label that's inappropriately draggable, that's Bug 460052, which is not the same thing. The original bug was fixed so I'm closing this again. Any new bug reports about being able to inappropriately drag from the label should be duped to Bug 460052. Thanks!