This repository has been archived by the owner on Nov 26, 2021. It is now read-only.
Releases: Financial-Times/o-video
Releases · Financial-Times/o-video
patch: Don't nest caption link within play button - CI-788
Don't nest caption link within play button (#210) Previously the guidance element which contains a link was nested within the play button. Interactive elements shouldn't be nested within each other. Co-authored-by: Jake Champion <[email protected]>
patch: Don't nest caption link within play button (v6) - CI-788
Don't nest caption link within play button (#211) Previously the guidance element which contains a link was nested within the play button. Interactive elements shouldn't be nested within each other.
patch: Update test instructions in README - add link to origami build tools
v6.1.3 Update test instructions in README - add link to origami build tools
patch: Fix ads script loading - store error to reject on subsequent calls
The previous code never resolved or rejected after the ads script tag failed to load once
patch: Update issues link in README
v6.1.1 Update issues link in README
minor: Track `waiting` event
closes #124
patch: add extension to relative imports to make the code work directly in a browser
v6.0.11 fix extension in import
patch: Add general usage to readme
Links to instructions on how to include Origami components. https://github.com/Financial-Times/origami/issues/59
patch: Remove h4 heading from the optional o-video title
1. We cannot assume an `h4` is the correct heading level in context. 2. The video title likely doesn't make sense as a heading in the document context. 3. The heading is removed when the video is played. Just because a video is played, the structure of the document shouldn't change. Given point 2 and 3, we can set the element to a `span`. However as a `span` the title of the video is read out as text by a screen reader in the middle of a page without context. The title is then repeated as part of the play button. To address that set `aria-label="Video Player”` to the root `o-video` element which is read `“Video Player, group“` by VoiceOver; giving useful context when stepping through page content. https://github.com/Financial-Times/o-video/issues/170
patch: update support details to be content-innovation team
v6.0.8 add correct email to support details