metafizzy / flickity

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

Adaptive height #11

Closed myconcretelab closed 8 years ago

myconcretelab commented 9 years ago

Is it planned to have a adaptive height or we just build it with event?

Thank you for this great slider that is really sweeter than others

desandro commented 9 years ago

Thanks for this request! Currently, no it's not in the plans. If others would like to see this feature, please +1 this ticket.

Adding adaptive height can be done with a bit of code. I'll follow up once I'm able to produce an example.


Thanks you for this feedback. If you like stickers, please email me yo@metafizzy so I can send some your way.

myconcretelab commented 9 years ago

Thanks, i has searching for event but not directly find it.

lunelson commented 9 years ago

+1 either for building-in the feature, or just callback/events API and classes so we react on changes and reference properties of prev/current/next slides

lunelson commented 9 years ago

But it's only logical to do this if the slide width is 100%, otherwise you can still see other content on the slides which is either shorter or higher

myconcretelab commented 9 years ago

The slider would take the height of the highest item in the viewport as reference.

stephenmcghee commented 9 years ago

If by adaptive height, you mean auto-height, +1. Would like to see this feature similar to slick/flexslider.

gergelypap commented 9 years ago

+1, would be neat to have.

stefancova commented 9 years ago

+1, it would be nice to no specify height for slider items

72quadrat commented 9 years ago

+1 really an extremely useful feature

dumplingsol commented 9 years ago

+1 really need this when creating sliders that have dynamic height, such as sliders where the content doesn't consist of images. Take the height of the highest element of the slides and apply to the container.

Does anyone know how to to this before the feature is release, if it will be released? I'm struggling with this problem atm.

desandro commented 9 years ago

@jonathanolsen

Take the height of the highest element of the slides and apply to the container.

Flickity already supports this: http://codepen.io/desandro/pen/ZYvemV

joelriveradev commented 9 years ago

+1 for auto height. I am using flickity for a project right now that needs this feature, and not having this option built in is a real bummer.

dom-turnbull commented 9 years ago

+1 for this.

zslabs commented 9 years ago

+1

Sliders are used for more than just image galleries -- I've even used them for a tabbed interface.

jacmaes commented 9 years ago

Another +1

notflip commented 9 years ago

+1! Would be amazing

richgcook commented 9 years ago

+1 🍕

impsart commented 9 years ago

+1 ... in need of this feature.

macbruker commented 9 years ago

+1

komunyako commented 9 years ago

93 — That's mine attempt to implement this feature.

ksherman commented 9 years ago

+1!

appliculture commented 9 years ago

+1

estrattonbailey commented 9 years ago

+1

languitar commented 9 years ago

+1

anthony0579 commented 9 years ago

+1

ErikFontanel commented 9 years ago

flickity is my slider of choice, so +1 as well!

ianxd commented 9 years ago

+1 !!

paulmasek commented 9 years ago

+1

kimlarocca commented 9 years ago

+1

circa1983 commented 9 years ago

+1 for me as well

CumpsD commented 9 years ago

+1 Would love this

imaginamundo commented 9 years ago

+1

kaisermann commented 9 years ago

+1 pretty please

animaux commented 9 years ago

+1

chappydboz commented 9 years ago

+1

scottcarver commented 9 years ago

+1

kyleoliveiro commented 9 years ago

+1, Flickity is my de-facto carousel plugin, except when adaptive height is a requirement. Would be awesome to see this feature added.

matthiashonert commented 9 years ago

+1

christophermiles commented 9 years ago

+1

raywongjr commented 9 years ago

+1

brandexponents commented 9 years ago

+1

maxariss commented 9 years ago

+100 (best slider out there)

andreylysenko commented 9 years ago

+1

jpkempf commented 9 years ago

+1!

idiazroncero commented 9 years ago

+1 I was a bxSlider user until iI came across Flickity. The API, touch events, physic animations, the "asNavFor", the overall performance... there is no competition here: flickity is better. However, I am missing bxSlider's "adaptiveHeight" feature. For a recent project, I had to use bxSlider instead of flickity because of this. I would definitively LOVE flickity to add this feature.

laurelstreng commented 9 years ago

+1!

vision34 commented 9 years ago

+1

mikebikeboy commented 9 years ago

Super easy:

slider.on( 'cellSelect', function() { var selectedHeight = slider.find('.is-selected').height(); slider.find('.flickity-viewport').height(selectedHeight); });

vision34 commented 9 years ago

your code doesnt worked for me, but you gave me an idea, so i ve done it my way.

  $slider.on( 'cellSelect', function() {
            $newTab = $(this).find('.is-selected'); 
    $slider.find('.flickity-viewport').height($newTab.height());        

  });

and if you want the animation:

 //     $slider.find('.flickity-viewport').height($newTab.height());    

 $slider.find('.flickity-viewport').delay(111).animate({height: $newTab.height()} );        

but does somebody knows why animation sometimes in chrome slowdown? In firefox it works well.

kevinPire commented 9 years ago

+1