ampproject / ampproject/amphtml

[amp-story-player] Audit public API methods for layout state

Open
#34,375 1 comment 0 reactions 1 assignee Claimed by @gmajoulet View on GitHub
Stale Type: Bug WG: stories
Dominant language
JavaScript
Stars
14.9k
Forks
4.1k
PR merge metrics
No merged PRs in 30d

Description

We should audit the public methods of the player for layout state.

In #34360, I found that by calling `play()` under certain circumstances (player hidden, and no user scroll), the player can break. We should investigate other [public methods](https://github.com/ampproject/amphtml/blob/main/spec/amp-story-player.md#programmatic-control) to see if they will work correctly (e.g. `show()`, `add()`, etc..).

/cc @ampproject/wg-stories

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.