Skip to content

Fix: Multiline Text though clipped can be scrolled into view#2584

Closed
tienifr wants to merge 1 commit intonecolas:masterfrom
tienifr:fix/clipped-text-can-be-scrolled-into-view
Closed

Fix: Multiline Text though clipped can be scrolled into view#2584
tienifr wants to merge 1 commit intonecolas:masterfrom
tienifr:fix/clipped-text-can-be-scrolled-into-view

Conversation

@tienifr
Copy link
Contributor

@tienifr tienifr commented Sep 25, 2023

Details

Fixes #2582.

On mobile Web, multiline Text's clipped part can be scrolled into view by dragging selection cursor. This PR blocks scrolling entirely, through any mechanism, over overflown text.

Tests

  1. Create a long Text with numberOfLines > 1
  2. On mWeb, long press the text to toggle text selection cursor
  3. Drag the selection cursor vertically
  4. Observe that the clipped text cannot be scrolled into view

Screenshots/Videos

Screenrecording_20230925_183054.mp4

@codesandbox-ci
Copy link

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 9224e34:

Sandbox Source
react-native-web-examples Configuration

@neil-marcellini
Copy link

@necolas could you please take a look at this or get someone to review it for us please?

@quantizor
Copy link

Released under the new fork https://github.com/quantizor/react-native-web-fork/releases/tag/v0.20.2

@necolas necolas closed this in 47f0144 Apr 3, 2025
necolas pushed a commit that referenced this pull request Apr 3, 2025
necolas pushed a commit that referenced this pull request Apr 3, 2025
@TheKohan
Copy link

I'm having an issue with this fix on older chromium versions - seem's like overflow:clip is not supported on chromium versions <89

A lot of Tizen/LG WebOS web engine powered TV's are running old chromium versions (even tv's from like 2020-2022). Any way to conditionally apply this change only to webengines that support that ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

mWeb - Multiline Text's clipped part is scrolled into view by dragging selection cursor

4 participants