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

Provide a vanila js slider instead of slick #441

Open
mariano-dagostino opened this issue Feb 28, 2023 · 7 comments
Open

Provide a vanila js slider instead of slick #441

mariano-dagostino opened this issue Feb 28, 2023 · 7 comments

Comments

@mariano-dagostino
Copy link
Collaborator

As drupal evolves to remove the jquery dependency as much as possible I think is worth to analyze if we can remove jQuery from the carousels

https://github.com/davidhund/awesome-vanilla-js#carousel

@amitaibu
Copy link
Member

About 6 months ago I tried to look for a replacement to Slick, but none was good enough. Yeah, I won't mind changing it

@amitaibu
Copy link
Member

amitaibu commented May 3, 2023

Tried Swiper JS, but I don't think its worth the effort - #491 (comment)

@bmunslow
Copy link
Contributor

As an idea, Splide is looking good (though I haven't tried it myself):

@sonvir249
Copy link
Contributor

@amitaibu @bmunslow I have an Idea of replacing slick slider with tiny slider or glider-js. Please do let me know your thoughts. Thanks!

@amitaibu
Copy link
Member

Unless there's a clear advantage, I believe we can keep the current implementation.

@sonvir249
Copy link
Contributor

sonvir249 commented Nov 13, 2024

@amitaibu As Drupal is removing the jquery dependency we can also do the same as mentioned by @mariano-dagostino for slider. Also I can see jQuery is used in custom server_theme JS libraries. Ideally we should use vanilla JS in favour of removing the jQuery dependency.

@amitaibu
Copy link
Member

Sure, PRs are welcome 😄

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

No branches or pull requests

4 participants