1528420 - [webvtt] Cue's positionAlign should be `auto` by default

Status: RESOLVED FIXED

(Core :: Audio/Video: Playback, enhancement, P2)

Description

[Alastor Wu [:alwu]]

According to spec [1], By default, the position alignment is set to auto

[1] https://www.w3.org/TR/webvtt1/#webvtt-cue-position-automatic-alignment

Comment 1

[Alastor Wu [:alwu]]

Attachment: Bug 1528420 - part1 : set positionAlign to `auto` by default.

According to spec [1], By default, the position alignment is set to auto

[1] https://www.w3.org/TR/webvtt1/#webvtt-cue-position-automatic-alignment

Comment 2

[Alastor Wu [:alwu]]

Attachment: Bug 1528420 - part2 : enable testing/web-platform/tests/webvtt/api/VTTCue/constructor.html.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/6d27c85a2ec3
part1 : set positionAlign to `auto` by default. r=jya
https://hg.mozilla.org/integration/autoland/rev/ba8b99b72a31
part2 : enable testing/web-platform/tests/webvtt/api/VTTCue/constructor.html. r=jya

Comment 4

[Bogdan Tara[:bogdan_tara | bogdant]]

https://hg.mozilla.org/mozilla-central/rev/6d27c85a2ec3
https://hg.mozilla.org/mozilla-central/rev/ba8b99b72a31

Comment 5

[Chris Mills [:cmills]]

Note to MDN writers — for the moment, I've just added a note to the Fx 67 rel notes about this:
https://developer.mozilla.org/en-US/docs/Mozilla/Firefox/Releases/67#APIs

We don't have proper docs for the WebVTT API at the moment, so we can't do anything more for now.

I've submitted an issue to our sprints board so that we can write a plan on what needs to be done to document this properly: https://github.com/mdn/sprints/issues/1333

Feel free to add to it if you have thoughts on what we need to include.

Comment 6

[Kohei Yoshino [:kohei]]

Posted site compatibility note: https://www.fxsitecompat.com/en-CA/docs/2019/vttcue-s-default-position-alignment-will-be-auto/

Comment 7

[Eric Shepherd [:sheppy]]

Documentation updates:

• Added sentence mentioning the default to the positionAlign entry in the Properties section on the VTTCue page
• Cleaned up and improved VTTCue() page
• Rewrote the note in Firefox 67 for developers to be more detailed, and moved it from the DOM section to Media, Web Audio, and WebRTC