- Issue created by @mabho
- Merge request !14Issue #3476245 by mabho: Update the carousel macro to deliver a dynamic amount of slides. → (Merged) created by mabho
- 🇧🇷Brazil mabho Rio de Janeiro, RJ
Here is a static patch file out of the changes delivered in commit
2a5f14b79fd68d6c2d92c96eaffad63728558d0a
. - First commit to issue fork.
-
aaronchristian →
committed edb9ebcb on 1.x authored by
mabho →
Issue #3476245: Carousel template is throwing error Deprecated function...
-
aaronchristian →
committed edb9ebcb on 1.x authored by
mabho →
- 🇨🇦Canada AaronChristian Kelowna, BC
Yep just ran into this issue myself today.
I like the addition of having the dynamic slide count as well. I never really use the macro outside of the UI kit but i think its a welcomed addition. :)
Thanks for submitting the patch Marcos!
- 🇧🇷Brazil mabho Rio de Janeiro, RJ
Sure thing, @aaronchristian, I am happy you agree with the solution, and that I could contribute back to the module's codebase :-)
Automatically closed - issue fixed for 2 weeks with no activity.