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

Fix/411/sign language interpretation not available for video content wcag 22 level aaa #3119

Conversation

alessioventuriniAND
Copy link
Contributor

@alessioventuriniAND alessioventuriniAND commented Apr 3, 2024

What is the context of this PR?

Fixes: https://github.com/ONSdigital/ons-service-manual-website/issues/398
Fixes: https://github.com/ONSdigital/ons-service-manual-website/issues/411

I have changed the video that we currently use in our tab-details component and video component. The old video failed two accessibility criteria as per the aforementioned issues.

This new video has sign language and doesn't need audio description.

How to review this PR

Confirm that the new video displays correctly.

Checklist

This needs to be completed by the person raising the PR.

  • I have selected the correct Assignee
  • I have linked the correct Issue

adi-unni and others added 10 commits February 28, 2024 14:46
* Page template for length

* Add macro

* Add component foundations

* Build stylesheet

* Add additional functionality

* Improved functionality

* Fix border and layout

* Fix resize behaviour

* Changes to CSS

* Add more tests

* Update VR Tests

* Change to full width

---------

Co-authored-by: Alessio Venturini <112873190+alessioventuriniAND@users.noreply.github.com>
* initial change

* update test

* update test

* update comment

* update comment
@alessioventuriniAND alessioventuriniAND added the Accessibility Issues discovered through accessibility testing label Apr 3, 2024
@alessioventuriniAND alessioventuriniAND requested a review from a team April 3, 2024 09:32
@alessioventuriniAND alessioventuriniAND self-assigned this Apr 3, 2024
Copy link

netlify bot commented Apr 3, 2024

Deploy Preview for ons-design-system-preview ready!

Name Link
🔨 Latest commit 4656a20
🔍 Latest deploy log https://app.netlify.com/sites/ons-design-system-preview/deploys/660d27264a1401000807d6f9
😎 Deploy Preview https://deploy-preview-3119--ons-design-system-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@alessioventuriniAND alessioventuriniAND merged commit ade7695 into main Apr 4, 2024
10 checks passed
@alessioventuriniAND alessioventuriniAND deleted the fix/411/sign-language-interpretation-not-available-for-video-content-wcag-22-level-aaa branch April 4, 2024 08:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Accessibility Issues discovered through accessibility testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants