- Issue created by @Grimreaper
- Merge request !216Issue #3484065 by grimreaper: Rework carousel item → (Merged) created by Grimreaper
- 🇫🇷France Grimreaper France 🇫🇷
If you are ok with that I will create a change record.
-
grimreaper →
committed 62c6fe8d on 5.0.x
Issue #3484065 by grimreaper, pdureau: Rework carousel item
-
grimreaper →
committed 62c6fe8d on 5.0.x
Automatically closed - issue fixed for 2 weeks with no activity.
- 🇫🇷France Grimreaper France 🇫🇷
@pdureau,
As discussed on phone, I am re-opening this issue because I think it broke Views usage.
I am ok to revert the change for 5.0.x as I think this would require new mechanism from UIP2.
For views: we need to be able to control the "active" boolean prop with the fact that the element is the first (or nth) result.
For field formatters: we need to be able to control the "active" boolean prop with the fact that the element is the first (or nth) element/delta. - 🇫🇷France Grimreaper France 🇫🇷
Remove active prop.
Custom JS that will check all carousel items and will place the active class on the first one if none have the active class. -
grimreaper →
committed b7953c36 on 5.1.x
Issue #3484065 by grimreaper, pdureau: Fix components after new UIP...
-
grimreaper →
committed b7953c36 on 5.1.x
- Merge request !230Issue #3484746 by grimreaper, pdureau: Use "is sequence" Twig test to prevent... → (Merged) created by Grimreaper
-
grimreaper →
committed cda06b36 on 5.1.x
Issue #3484065 by grimreaper, pdureau: Carousel item: remove active prop...
-
grimreaper →
committed cda06b36 on 5.1.x
- Merge request !231Issue #3484065 by grimreaper, pdureau: Carousel item: remove active prop,... → (Merged) created by Grimreaper
-
grimreaper →
committed 3e30cb3d on 5.0.x
Issue #3484065 by grimreaper, pdureau: Carousel item: remove active prop...
-
grimreaper →
committed 3e30cb3d on 5.0.x
Automatically closed - issue fixed for 2 weeks with no activity.