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] Touch functionality works improperly in RTL #6696

Closed
FunnyPhantom opened this issue Mar 7, 2019 · 3 comments
Closed

[Bug Report] Touch functionality works improperly in RTL #6696

FunnyPhantom opened this issue Mar 7, 2019 · 3 comments
Assignees
Labels
T: bug Functionality that does not work as intended/expected
Milestone

Comments

@FunnyPhantom
Copy link

Versions and Environment

Vuetify: 1.5.2
Vue: 2.6.6
Browsers: Chrome 72.0.3626.119
OS: Windows 10

Steps to reproduce

  • change direction
  • use a touchscreen to swipe

Expected Behavior

swiping to left shows the picture to the right of the current image in the view and vice versa.

Actual Behavior

swiping to left results to show the picture to the left,
and swiping right results to show the picture to the right.

Reproduction Link

https://codepen.io/funnyphantom/pen/PLpMRE

Other comments

Although it is very nice to see that the setting $vuetify.rtl to true is reflected in direction of the items in the carousel, but it seems like to have an unnecessary impact on the touch control of the items.
(this bug is also present in touch functionality of VTabs and presumably all other component which mixined touch)

@johnleider johnleider self-assigned this Apr 5, 2019
@johnleider johnleider added the T: bug Functionality that does not work as intended/expected label Apr 5, 2019
@johnleider johnleider added this to the v2.0.0 milestone Apr 5, 2019
@dsseng
Copy link
Contributor

dsseng commented Jun 11, 2019

@johnleider that's not fixed yet

@FunnyPhantom
Copy link
Author

@sh7dm the current workaround is to swap left and right default action, the default action can be accessed in touch util.

@jacekkarczmarczyk jacekkarczmarczyk added S: has PR The issue has a pending Pull Request and removed S: has PR The issue has a pending Pull Request labels Jun 17, 2019
@johnleider
Copy link
Member

This unfortunately will not be resolved in time for v2.0. Will resolve in a patch later on. Thank you for your patience!

If you have any additional questions, please reach out to us in our Discord community.

@johnleider johnleider modified the milestones: v2.0.0, v2.x.x Jun 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T: bug Functionality that does not work as intended/expected
Projects
None yet
Development

No branches or pull requests

4 participants