[Help]: Missing destroyHeight from AutoHeight() options
#915
-
SummaryHi @davidjerleke, hope you're doing well. I update dependencies routinely in some of my projects (Hawa in this case) and it seems the latest version of I couldn't find any mention in the changelog/release descriptions. Was it replaced with something else?
Thank you for the continuous work you do for this wonderful package 🌹 If applicable, which variants of Embla Carousel are relevant to this question?
|
Beta Was this translation helpful? Give feedback.
Replies: 1 comment 3 replies
-
|
Ok after a little digging I found the moment it was removed 👇 8eca652#diff-9b492d4efffa04663108927b0281a3403b6426079070f8bb9732cc274abdba41 |
Beta Was this translation helpful? Give feedback.
-
|
Hi @zaaakher, Nice to hear from you again 🙂. You're right, it was on my todo for
No need to set Best, |
Beta Was this translation helpful? Give feedback.
-
|
Hey brother 👋, Sounds excellent, well done as always 👍! Don't worry about it, I kinda wanted to put this discussion out there so that other people can find the answer if they ever had the same question. If you think it's not enough and other devs might create issues regarding the missing Note As of Let me know if you'd like that and I will create a PR for it :) However, feel free to @ me anytime you need assistance in anything and I will try to do my best. 👌 |
Beta Was this translation helpful? Give feedback.
-
@zaaakher that's great and very helpful 👍.
Good idea!. Let's see if this discussion is enough or if it will cause a lot of trouble.
Thanks for supporting this project! Will do 🙂. |
Beta Was this translation helpful? Give feedback.
Hi @zaaakher,
Nice to hear from you again 🙂. You're right, it was on my todo for
v8.0.0but somehow I forgot to remove it. Sorry for the inconvenience. The new behavior is much better and works like this:styleproperty from the container.No need to set
destroyHeightanymore. A breaking change not related to a major version doesn't happen often I promise.Best,
David