Bug 476484 - Touch swipe scrolling in ScrollablePage does not work over SelectableLabel text
Summary: Touch swipe scrolling in ScrollablePage does not work over SelectableLabel text
Status: RESOLVED FIXED
Alias: None
Product: frameworks-kirigami
Classification: Frameworks and Libraries
Component: general (other bugs)
Version First Reported In: 5.109.0
Platform: Flatpak Linux
: NOR normal
Target Milestone: Not decided
Assignee: kdelibs bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-11-02 21:28 UTC by Tammes Burghard
Modified: 2024-01-31 14:45 UTC (History)
4 users (show)

See Also:
Latest Commit:
Version Fixed/Implemented In: 6.0
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Tammes Burghard 2023-11-02 21:28:36 UTC
SUMMARY
***
I use Kasts on my surface tablet running fedora silverblue and have several touch issues. This bug is for the first one: 

When I try to scroll the main ScrollablePage via touch, it works great as long as the ScrollablePage contains a GridView or ListView. But when it contains text, swiping over it selects the text instead of scrolling which is weird.
***


STEPS TO REPRODUCE
1. Use Kasts on a tablet with pen
2. Open the Episodes Page with your finger
3. Open a single episode that has a page filling (long enough) description
4. Try to scroll by swiping over the text

OBSERVED RESULT
Text is selected

EXPECTED RESULT
The page gets scrolled

SOFTWARE/OS VERSIONS
Linux: Fedora Silverblue
Kasts: 23.04.3 as a Flatpak
KDE Frameworks Version: 5.109
Qt Version: 5.15.10
Tablet: Microsoft Surface Go 2
Pen: Microsoft Surface Pen
Comment 1 Tammes Burghard 2023-11-02 21:36:54 UTC
The same applies for swiping with the pen, but there, selecting text is fine with me.
Comment 2 Nate Graham 2023-11-03 21:45:40 UTC
Can still confirm in Plasma 6 Wayland.
Comment 3 bart 2024-01-30 09:09:35 UTC
Couldn't this be due to qtquick/kirigami not switching to mobile/touch/tablet mode?  I.e. treating the pen actions as a regular mouse.

Could you perhaps try and start up Kasts with "QT_QUICK_CONTROLS_MOBILE=1 kasts"?  If this works, then it's probably due to that.

As background info: the pages you are refering to, have explicitly set the "text selectable by mouse" on them.  But, for me, it doesn't interfere with touch swiping when in mobile mode.  Unfortunately, I don't have a pen to test with.  (In fact, I don't even know whether a pen in qtquick apps should be considered a mouse or a touch or ...)
Comment 4 Nate Graham 2024-01-30 20:29:49 UTC
This is actually fixed for me now in the SelectableLabel component itself with current git master; before I could reproduce the issue in Discover, and now I can't. I seem to recall Fushan fixing it sometime in the last few months.

If the issue persists in Kasts, I might suggest looking into the implementation code in that app.
Comment 5 Tammes Burghard 2024-01-31 14:45:20 UTC
Yes, It is fixed for me as well. Nice :)