Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[slider] iOS: cannot drag handle to change value #5016

Closed
sorvell opened this issue Sep 27, 2023 · 2 comments · Fixed by #5005 · May be fixed by X-oss-byte/pigweed#61, X-oss-byte/pigweed#70 or X-oss-byte/pigweed#73
Closed

[slider] iOS: cannot drag handle to change value #5016

sorvell opened this issue Sep 27, 2023 · 2 comments · Fixed by #5005 · May be fixed by X-oss-byte/pigweed#61, X-oss-byte/pigweed#70 or X-oss-byte/pigweed#73
Assignees

Comments

@sorvell
Copy link
Collaborator

sorvell commented Sep 27, 2023

What is affected?

Component

Description

On iOS, dragging the slider handle does not move the handle or change the value.

Reproduction

  1. On iOS, go to https://material-web.dev/components/slider/ and expand the interactive demo.
  2. On one of the sliders, drag the thumb to change the value.

Expected: the thumb moves and the value changes.
Actual: the thumb does not move and the value does not change.

Workaround

N/A

Is this a regression?

No or unsure. This never worked, or I haven't tried before.

Affected versions

1.0

Browser/OS/Node environment

Browser: Safari
OS: iOS 17

@sorvell
Copy link
Collaborator Author

sorvell commented Sep 27, 2023

Here is a fix: https://lit.dev/playground/#gist=8c6324fb869859c7198cb588d05da151

The problem is the transform: scaleX(0) on ::-webkit-slider-thumb. I believe this was done to make it easier to tap near the handle to set a value. The handle is 20px wide and the touch target is 40px wide.

I suggest just removing the scale or maybe making it 0.5 so it's the same size as the handle, but that may make it harder to drag.

@e111077 e111077 self-assigned this Sep 27, 2023
@e111077
Copy link
Contributor

e111077 commented Sep 27, 2023

I would like to note that the blame on that line has your name all over it >:D

copybara-service bot pushed a commit that referenced this issue Sep 27, 2023
copybara-service bot pushed a commit that referenced this issue Sep 27, 2023
copybara-service bot pushed a commit that referenced this issue Sep 27, 2023
copybara-service bot pushed a commit that referenced this issue Sep 28, 2023
copybara-service bot pushed a commit that referenced this issue Oct 4, 2023
copybara-service bot pushed a commit that referenced this issue Oct 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment