-
Notifications
You must be signed in to change notification settings - Fork 4.1k
Closed
Labels
Component: amp-carouselCustomer: DeveloperP2: SoonType: DevXissues impacting developer experienceissues impacting developer experienceType: Feature RequestWG: components
Milestone
Description
What's the issue?
The amp-carousel of type "slides" automatically updates the event.index when the slides are changed. However, the amp-carousel of type "carousel" does not update the event.index when a majority of a given slide is in view. This makes it difficult to use amp-bin and other bindable actions across different components. The slides type is not sufficient due to the look and feel of the component i.e. not being able to see all the possible slides and only being able to see the one slide.
How do we reproduce the issue?
Your can reproduce this issue here: https://mshishir1.herokuapp.com/shop/amp_carousel_bug.html#development=1
What browsers are affected?
All browsers and all device types should be affected.
Which AMP version is affected?
This was likely always broken. I am currently reproducing this on AMP Version 1525892015019
Reactions are currently unavailable
Metadata
Metadata
Labels
Component: amp-carouselCustomer: DeveloperP2: SoonType: DevXissues impacting developer experienceissues impacting developer experienceType: Feature RequestWG: components