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

Rtl should affect programmatical navigation #243

Open
Yahohoo opened this issue May 5, 2022 · 1 comment · May be fixed by #277
Open

Rtl should affect programmatical navigation #243

Yahohoo opened this issue May 5, 2022 · 1 comment · May be fixed by #277

Comments

@Yahohoo
Copy link

Yahohoo commented May 5, 2022

When I use the rtl option of the carousel its slides reorder to match the rtl usability priciples. This package will work well if I use default or slot arrows just because these arrow positions are at the sides of the carousel. But if I use the custom controls outside of the carousel and prev and next methods like this.$refs.carousel.next it will work like in the ltr mode. For users of rtl language "next" button should move the carousel to the next left slide. That’s why I need to choose methods based on my app current language direction to make it work right and thus mess up code.

It also will apply to the indexes of slides and events like beforeChange if I use custom pagination outside of the carousel.

I know it’s a breaking change, maybe it would be enough to add an option like the rtlNavigation.

@Iran-110
Copy link

Iran-110 commented Sep 20, 2023

Hi, react-slick and vue-slick-carousel did not truly implemented the RTL feature. For supporting the RTL feature, I changed some functions and created a new fork based on vue-slick-carousel. You can use it from vue-slick-carousel-rtl.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants