Skip to content

[amp-carousel] type=carousel does not update event index #15181

@shishirm

Description

@shishirm

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

Metadata

Metadata

Type

No type

Projects

No projects

Relationships

None yet

Development

No branches or pull requests

Issue actions