Skip to content

Conversation

@foolip
Copy link
Collaborator

@foolip foolip commented May 20, 2024

No description provided.

@@ -0,0 +1,8 @@
name: WebVTT cue alignment
description: The WebVTT cue alignment settings control which part of the cue is aligned with the given line and position. Cue alignment is set using `line` and `position` settings in WebVTT files or the `lineAlign` and `positionAlign` properties of `VTTCue` using JavaScript.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Optional suggestion here: What do you think of something like this? I'm trying to avoid (my initial) confusion that this affects overall placement of the cue/region with respect to the video viewport.

Suggested change
description: The WebVTT cue alignment settings control which part of the cue is aligned with the given line and position. Cue alignment is set using `line` and `position` settings in WebVTT files or the `lineAlign` and `positionAlign` properties of `VTTCue` using JavaScript.
description: WebVTT cue alignment settings control how text is inset within a cue box, relative to the writing direction of the cue. Cue alignment is set using `line` and `position` settings in WebVTT files or the `lineAlign` and `positionAlign` properties of `VTTCue` using JavaScript.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Maybe I have misunderstood how this all is supposed to work.

My understanding was that line and position (the percentages before the comma in the file format) define a point in the video viewport, and by default it's the top of the cue box and a horizontal coordinate that depends on align that is lined up with that point.

And then lineAlign and positionAlign (they keywords after the comma in the file format) change which point of the cue box is lined up with that point in the video viewport.

Copy link
Collaborator

Choose a reason for hiding this comment

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

I think it's genuinely possible I've misunderstood it, but I can't be sure. But I think it's not worth it to spend a time understanding this to write a slightly better description. But if it's wrong then I suspect that it won't be long before someone who cares about and understands WebVTT will run across this and demand a correction.

Which is to say, let's merge it as is.

@foolip foolip force-pushed the webvtt-cue-alignment branch from c9e16c7 to 1395c9f Compare June 5, 2024 13:45
@@ -0,0 +1,8 @@
name: WebVTT cue alignment
description: The WebVTT cue alignment settings control which part of the cue is aligned with the given line and position. Cue alignment is set using `line` and `position` settings in WebVTT files or the `lineAlign` and `positionAlign` properties of `VTTCue` using JavaScript.
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think it's genuinely possible I've misunderstood it, but I can't be sure. But I think it's not worth it to spend a time understanding this to write a slightly better description. But if it's wrong then I suspect that it won't be long before someone who cares about and understands WebVTT will run across this and demand a correction.

Which is to say, let's merge it as is.

@ddbeck ddbeck merged commit ec4214a into web-platform-dx:main Jun 5, 2024
@foolip foolip deleted the webvtt-cue-alignment branch June 5, 2024 21:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature definition Creating or defining new features or groups of features.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants