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

[Feature Request] v-slider thumb is tiny for mobile use. #4356

Closed
amitgur opened this issue Jun 20, 2018 · 7 comments
Closed

[Feature Request] v-slider thumb is tiny for mobile use. #4356

amitgur opened this issue Jun 20, 2018 · 7 comments
Assignees
Labels
C: VSlider VSlider T: bug Functionality that does not work as intended/expected
Milestone

Comments

@amitgur
Copy link

amitgur commented Jun 20, 2018

Problem to solve

I can't drag the v-slider on mobile. The v-slider thumb is too small.

Proposed solution

Changing the size of the thumb for friendly UX on mobile. Like this thumb https://www.w3schools.com/howto/tryit.asp?filename=tryhow_css_rangeslider

@johnleider
Copy link
Member

The v-slider thumb slider was built to match Material Design 1 spec. The newer version has some style updates that we will be implementing in the future as part of our MD2 update. Because of that, this issue is not required as all components will be updated to match.

@KaelWD
Copy link
Member

KaelWD commented Jun 22, 2018

I asked him to open this. It's fine visually, but the touch target is so small it's almost impossible to use.

@KaelWD KaelWD reopened this Jun 22, 2018
@KaelWD KaelWD added the T: enhancement Functionality that enhances existing features label Jun 22, 2018
@MajesticPotatoe MajesticPotatoe added the C: VSlider VSlider label Apr 10, 2019
@KaelWD KaelWD added T: bug Functionality that does not work as intended/expected and removed T: enhancement Functionality that enhances existing features labels Jun 23, 2019
@pianistprogrammer

This comment has been minimized.

@Zyles

This comment has been minimized.

@DEberhardBrelag

This comment has been minimized.

@MentalGear

This comment has been minimized.

@MentalGear
Copy link

Quickly fixed this by enlarging the slider touch target through a none-destructive CSS addition.
Specifically, I added a :after (new pseudo element) and set it invisible. Now the slider is a joy to use (as the rest of the frame work) as it's easily manipulatable. The fix can be made simply by adding this to your general / default css file.

.v-slider__thumb:after { transition: 0.3s cubic-bezier(0.25, 0.8, 0.5, 1); content: ''; color: inherit; width: 400%; height: 400%; border-radius: 50%; background: transparent; position: absolute; left: -150%; top: -150%; }

Of course vuetify is welcome to add this as well. @johnleider
Also, I would suggest modifying :before to make the touch effect adoptable to different v-slider__thumb sizes.

.v-slider__thumb:before { transition: 0.3s cubic-bezier(0.25, 0.8, 0.5, 1); content: ''; color: inherit; width: 200%; height: 200%; border-radius: 50%; background: currentColor; opacity: 0.3; position: absolute; left: -50%; top: -50%; transform: scale(0.1); pointer-events: none; }

Example: (green border to visualize new touch target)

Screenshot 2020-11-15 at 20 40 19

@KaelWD KaelWD self-assigned this Jul 20, 2021
@KaelWD KaelWD added this to the v2.5.x milestone Jul 20, 2021
@KaelWD KaelWD closed this as completed in 9c6a4a6 Jul 20, 2021
mcdmaster pushed a commit to mcdmaster/vuetify that referenced this issue Oct 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C: VSlider VSlider T: bug Functionality that does not work as intended/expected
Projects
None yet
Development

No branches or pull requests

8 participants