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

A11y implementation for WCAG compliance. #4715

Merged
merged 4 commits into from
Apr 3, 2024

Conversation

cybmiper
Copy link
Contributor

No description provided.

@Mazas
Copy link

Mazas commented Mar 14, 2024

Some context for the PR. We have talked about this trough email before. The current implementation has accessibility support; however, it is not localized. We would like to bring some attention to it and work towards making sure it follows the standards required by the EU.

packages/modelviewer.dev/assets/a11y.json Outdated Show resolved Hide resolved
packages/modelviewer.dev/examples/wcag/index.html Outdated Show resolved Hide resolved
packages/model-viewer/src/features/wcag.ts Outdated Show resolved Hide resolved
packages/modelviewer.dev/examples/wcag/index.html Outdated Show resolved Hide resolved
Copy link
Collaborator

@elalish elalish left a comment

Choose a reason for hiding this comment

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

Looking good, thanks! Just a few details to clear up. Also, this probably isn't a Draft PR anymore, right?

packages/model-viewer/src/features/controls.ts Outdated Show resolved Hide resolved
packages/modelviewer.dev/data/docs.json Outdated Show resolved Hide resolved
@cybmiper cybmiper marked this pull request as ready for review April 2, 2024 14:08
@cybmiper cybmiper requested a review from elalish April 2, 2024 14:09
Copy link
Collaborator

@elalish elalish left a comment

Choose a reason for hiding this comment

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

This looks excellent - thank you for the tests!

@elalish elalish merged commit b8df887 into google:master Apr 3, 2024
3 checks passed
JL-Vidinoti pushed a commit to vidinoti/model-viewer that referenced this pull request Apr 22, 2024
* A11y implementation for WCAG compliance.

* Readd in controls, merge examples into loading and add docs.

* Added a language switch in example page.

* Added tests in control for a11y and minor review fixes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants