Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WebVTT: Support font-size CSS property #8964

Closed
icbaker opened this issue May 19, 2021 · 0 comments
Closed

WebVTT: Support font-size CSS property #8964

icbaker opened this issue May 19, 2021 · 0 comments
Assignees

Comments

@icbaker
Copy link
Collaborator

icbaker commented May 19, 2021

The ::cue pseudo-element can contain "the properties corresponding to the font shorthand" (spec). This includes font-size.

Lack of support for this was flagged in #8958

Implementing this probably involves editing the WebVTT CssParser. WebvttCssStyle already has font size support (though it doesn't seem to be used, and I'm not sure it ever has been), so it should be possible to re-use that.

@icbaker icbaker self-assigned this May 19, 2021
@icbaker icbaker assigned andpodob and unassigned icbaker Jul 16, 2021
andrewlewis pushed a commit that referenced this issue Aug 3, 2021
This CL addresses the github issue [#8946](#8964). That issue requests support for `font-size` CSS property in WebVTT subtitle format. This CL:
* Adds support for `font-size` property by extending capabilities of WebVTT `CssParser`. Implementation of `font-size` property value parsing is based on the one in `TtmlDecoder`.
* Adds unit test along with test file containing WebVTT subtitles with all currently supported `font-size` units.

#minor-release

PiperOrigin-RevId: 388423859
@icbaker icbaker closed this as completed Aug 5, 2021
christosts pushed a commit that referenced this issue Aug 11, 2021
This CL addresses the github issue [#8946](#8964). That issue requests support for `font-size` CSS property in WebVTT subtitle format. This CL:
* Adds support for `font-size` property by extending capabilities of WebVTT `CssParser`. Implementation of `font-size` property value parsing is based on the one in `TtmlDecoder`.
* Adds unit test along with test file containing WebVTT subtitles with all currently supported `font-size` units.

#minor-release

PiperOrigin-RevId: 388423859
@google google locked and limited conversation to collaborators Oct 5, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants