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.

Sign up for GitHub

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

Jump to bottom

[Bug Report] v-carousel nav arrows bad positions with show-arrows-on-hover while RTL is true #8457

Closed
abdumuslim opened this issue Aug 10, 2019 · 0 comments · Fixed by #8458
Closed

[Bug Report] v-carousel nav arrows bad positions with show-arrows-on-hover while RTL is true #8457

abdumuslim opened this issue Aug 10, 2019 · 0 comments · Fixed by #8458
Labels
C: VWindow VWindow rtl S: has PR The issue has a pending Pull Request T: bug Functionality that does not work as intended/expected

Comments

@abdumuslim
Copy link

Screenshot (174) Screenshot (175)

Environment

Vuetify Version: 2.0.5
Vue Version: 2.6.10
Browsers: Chrome 76.0.3809.100
OS: Windows 10

Steps to reproduce

Using v-carousel with show-arrows-on-hover prop while RTL is true

Expected Behavior

Hidden navigation arrows when not hovering on carousel.

Actual Behavior

Navigation arrows don't hide when not hovering on carousel.

Reproduction Link

https://codepen.io/abdumuslim/pen/xvydKr
@ghost ghost added the S: triage label Aug 10, 2019
@jacekkarczmarczyk jacekkarczmarczyk added C: VCarousel VCarousel rtl S: has PR The issue has a pending Pull Request T: bug Functionality that does not work as intended/expected and removed S: triage labels Aug 10, 2019
jacekkarczmarczyk added a commit that referenced this issue Aug 10, 2019
@jacekkarczmarczyk
fix(VCarousel): wrong position of arrows on hover in rtl mode
70efff7 
fixes #8457
jacekkarczmarczyk added a commit that referenced this issue Aug 10, 2019
@jacekkarczmarczyk
fix(VCarousel): wrong position of arrows on hover in rtl mode
399fa13 
fixes #8457
@jacekkarczmarczyk jacekkarczmarczyk mentioned this issue Aug 10, 2019
Merged
9 tasks
@jacekkarczmarczyk jacekkarczmarczyk added C: VWindow VWindow and removed C: VCarousel VCarousel labels Aug 10, 2019
johnleider pushed a commit that referenced this issue Aug 12, 2019
@jacekkarczmarczyk @johnleider
fix(VCarousel): wrong position of arrows on hover in rtl mode (#8458)
8de535f 
fixes #8457
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
C: VWindow VWindow rtl S: has PR The issue has a pending Pull Request T: bug Functionality that does not work as intended/expected
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(VWindow): wrong position of arrows on hover in rtl mode vuetifyjs/vuetify
2 participants
@jacekkarczmarczyk @abdumuslim