search

metafizzy / flickity

:leaves: Touch, responsive, flickable carousels
https://flickity.metafizzy.co
7.53k stars 602 forks source link

Slider breaks with three cells and wrapAround #1086

Closed L3vi closed 3 years ago

L3vi commented 4 years ago

Added styling to make non-selected cells smaller, opaque, and spaced out a bit. During this process ran into this bug that only happens when there is only three cells and wrapAround is true. You can strip down the CSS down to essentially nothing and you'll still see the issue.

Test case: https://codepen.io/Levistum/pen/OJMaPVw

L3vi commented 4 years ago

That didn’t work either :/

On Aug 14, 2020, at 3:43 AM, Hlsg notifications@github.com wrote:

Try this: metafizzy/flickity-fade#1 (comment) https://github.com/metafizzy/flickity-fade/issues/1#issuecomment-486778382 — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/metafizzy/flickity/issues/1086#issuecomment-673990200, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADU3Z3FFBSPNFY3GXKOL3MTSAUBKXANCNFSM4O6RJ2OA.

Hlsgs commented 4 years ago

Yeah, sorry about that. Didn't pay proper attention to your question . That's why I deleted it after.

L3vi commented 4 years ago

That's okay, I just got the email and responded to it.

Thanks for the response though

desandro commented 3 years ago

Looks like this is an issue with wrapAround and not having enough content to fill the container. In which case, I'm tracking a resolution in #809