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

Implementation of the slidesPerGroup option #59

Open
shossk opened this issue Jan 24, 2021 · 4 comments
Open

Implementation of the slidesPerGroup option #59

shossk opened this issue Jan 24, 2021 · 4 comments
Labels
discuss enhancement New feature or request

Comments

@shossk
Copy link

shossk commented Jan 24, 2021

Is your feature request related to a problem? Please describe.
Hello, joe.
I found this library because I was tired of Swiper's complex and large number of dependencies.
It is exactly the best library I could have asked for.
However, I noticed that there is one option missing.
It is slidesPerGroup.
I will be very happy to stop using Swiper, which I hate to do.
Have you considered implementing this? If not, can you consider it as a pull request?

Describe the solution you'd like
Implementation of the slidesPerGroup option

Describe alternatives you've considered
We know that if we implement this, we will get closer and closer to Swiper.
I would like to see this implemented, even as a plugin.

thanks!

@joe223
Copy link
Owner

joe223 commented Jan 24, 2021

@shossk Really appreciate.

Implementation of the slidesPerGroup option

That is what I'm concerned with.

The better solution is provide a group of APIs that give us the ability to modify render/translate effect. We can implement slidesPerGroup with such APIs in an external plugin. But I have not figure out what this APIs should be yet.

Here is what I'm doing(the API is kinda of babbling)

options.injections.translate(state, env, options, timing)

How do you think?

@joe223 joe223 added discuss enhancement New feature or request labels Jan 24, 2021
@shossk
Copy link
Author

shossk commented Jan 28, 2021

@joe223
That's good!
Thanks for taking the time to do this!

@joe223
Copy link
Owner

joe223 commented Jan 30, 2021

@shossk Just creat a PR if you got other idea😃

@kmergen
Copy link

kmergen commented Apr 14, 2021

I think that should be a native option. We have the option "slidesPerView" then an option "slidesPerSlide" would make sense

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discuss enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants