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

feat(ReelItem): Add accessibility label #401

Merged
merged 1 commit into from
May 14, 2023

Conversation

absidue
Copy link
Collaborator

@absidue absidue commented May 13, 2023

ReelItems unfortunately only have their duration in one place, the accessibility label in textual form. As parsing that information out is language dependent, it should be left up to library users to do.

An example for how to parse that can be found here: https://github.com/FreeTubeApp/FreeTube/blob/bd6fb416a8b6ba388321dcc20c3d7d89c744f0f1/src/renderer/helpers/api/local.js#L300

Some examples for the accessibility label in English taken from the shorts tab on the LTT channel:

  • These mice keep getting WEIRDER... - 59 seconds - play video
  • How Low Can Our Resolution Go? - 1 minute - play video
  • I just found out about Elon. #SHORTS - 1 minute, 1 second - play video (I suspect this is a rounding error on YouTube's side, as the docs say shorts can only be up to 60 seconds long, https://support.google.com/youtube/answer/10059070?hl=en)

@LuanRT LuanRT merged commit 046103a into LuanRT:main May 14, 2023
@absidue absidue deleted the reel-accessibility-label branch May 14, 2023 07:39
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.

2 participants