Skip to content

Change the spec for amp-audio[autoplay]#7574

Merged
Gregable merged 2 commits intoampproject:masterfrom
Gregable:autoplay
Feb 15, 2017
Merged

Change the spec for amp-audio[autoplay]#7574
Gregable merged 2 commits intoampproject:masterfrom
Gregable:autoplay

Conversation

@Gregable
Copy link
Copy Markdown
Member

Change the spec for amp-audio[autoplay] to correctly indicate that this is a boolean attribute, rather than the enum mentioned today.

this is a boolean attribute, rather than the enum mentioned today.
@Gregable Gregable merged commit ae46731 into ampproject:master Feb 15, 2017
@Gregable Gregable deleted the autoplay branch February 15, 2017 17:34
The `autoplay` attribute allows the author to specify when - if ever - the animated image will autoplay.

The presence of the attribute alone implies that the animated image will always autoplay. The author may specify values to limit when the animations will autoplay. Allowable values are `desktop`, `tablet`, or `mobile`, with multiple values separated by a space. The runtime makes a best-guess approximation to the device type to apply this value.
If present, the attribute implies that the audio start playing as soon as it is
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

change "start" to "will start"

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is already merged, so I created #7583

mrjoro pushed a commit to mrjoro/amphtml that referenced this pull request Apr 28, 2017
* Change the spec for `amp-audio[autoplay]` to correctly indicate that
this is a boolean attribute, rather than the enum mentioned today.

* Change amp-audio language to reference 'audio' rather than 'image'.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants