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

[Bug Report] Chip group slider in touch mode in RTL has the wrong direction #8116

Closed
smortexa opened this issue Jul 28, 2019 · 1 comment · Fixed by #7928
Closed

[Bug Report] Chip group slider in touch mode in RTL has the wrong direction #8116

smortexa opened this issue Jul 28, 2019 · 1 comment · Fixed by #7928
Labels
C: VSlideGroup rtl S: has PR The issue has a pending Pull Request T: bug Functionality that does not work as intended/expected

Comments

@smortexa
Copy link

Environment

Vuetify Version: 2.0.1
Vue Version: 2.6.10
Browsers: Chrome 75.0.3770.142
OS: Windows 10

Steps to reproduce

Fix the direction of the slider in RTL mode

Expected Behavior

When swiping to the right, the slider does not back to the left.

Actual Behavior

When swiping to the right, the slider will back to the left.

Reproduction Link

https://codepen.io/seyed_me/pen/XvNONv?&editable=true&editors=101

@ghost ghost added the S: triage label Jul 28, 2019
@jacekkarczmarczyk jacekkarczmarczyk added rtl C: VSlideGroup T: bug Functionality that does not work as intended/expected S: has PR The issue has a pending Pull Request and removed S: triage labels Jul 28, 2019
johnleider pushed a commit that referenced this issue Jul 31, 2019
* fix: rtl VTabs onTouchEnd

* fix lint

fixes #8116
@MOAB7698

This comment has been minimized.

@vuetifyjs vuetifyjs locked as resolved and limited conversation to collaborators Dec 15, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
C: VSlideGroup rtl S: has PR The issue has a pending Pull Request T: bug Functionality that does not work as intended/expected
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants