Skip to content

✨ [amp story captions] get and set captions state from viewer - #38009

Merged
processprocess merged 3 commits into
ampproject:mainfrom
processprocess:captions-messaging
Apr 18, 2022
Merged

processprocess merged 3 commits into
ampproject:mainfrom
processprocess:captions-messaging

Conversation

@processprocess

@processprocess processprocess commented Apr 4, 2022 •

Copy link
Copy Markdown
Contributor

Functionality and tests to get and set captions state from viewer.

Closes #37898

@processprocess processprocess changed the title ✨ [amp story player] get and set captions state from viewer ✨ [amp story captions] get and set captions state from viewer Apr 4, 2022
@processprocess
processprocess marked this pull request as ready for review April 18, 2022 14:57
@amp-owners-bot

Copy link
Copy Markdown

Hey @gmajoulet, @newmuis! These files were changed:

extensions/amp-story/1.0/amp-story-viewer-messaging-handler.js
extensions/amp-story/1.0/test/test-amp-story-viewer-messaging-handler.js

@gmajoulet

Copy link
Copy Markdown
Contributor

Bundle-size shows +1kB, can you rebase your PR and make sure everything is ok?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Amp story captions] Allow Players to get/set the captions state

4 participants