search

cgkineo / adapt-scrollSnap

An extension which hides the scrollbar and snaps to blocks for large devices. Navigation based on swipe gestures, specific keyboard navigation, the mouse wheel, and an optional scroll button. Content is set to fill the viewport height.
GNU General Public License v3.0
1 stars 1 forks source link

Text cannot be selected #33

Open chris-steele opened 2 years ago

chris-steele commented 2 years ago

Within the page content (excluding notify popups) the user is unable to perform text operations such as select when scrollsnap is active.

danielghost commented 2 years ago

This will be due to the Hammer.js library used to handle swipe gestures. If gestures are required, delete Hammer.defaults.cssProps.userSelect; could be added to maintain text selection, but the UX may suffer - see http://hammerjs.github.io/tips/.

swashbuck commented 2 years ago

I’ve set userSelect to "auto" in the Hammer.js library. We will now do some further testing in our project.

The main issue I've found on desktop is that, in some cases, all text in the current block will be selected. This only happens on desktop when clicking with your mouse and then swiping up or down (similar to touching the screen and swiping). If you simply swipe up or down with a touchpad, no text is selected. Same for clicking the "next" button in the ScrollSnap bar... no issues.

On iPad (tested iPad Pro 5th gen), no text is selected when touching / swiping. An extended touch / press is required to begin selecting text. So, I did not see this problem there.