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

test: setup accessibility tests with axe #844

Draft
wants to merge 2 commits into
base: develop
Choose a base branch
from
Draft

Conversation

barmac
Copy link
Member

@barmac barmac commented Apr 4, 2024

This will ensure that we don't violate basic principles in the initial view.

@bpmn-io-tasks bpmn-io-tasks bot added the in progress Currently worked on label Apr 4, 2024
@barmac
Copy link
Member Author

barmac commented Apr 16, 2024

Right now the test fails due to empty heading for the index column. In DMN, this heading is reserved for the hit policy which we moved to the top a couple of years ago. Hit policy is unrelated to the index though.

@barmac
Copy link
Member Author

barmac commented Apr 16, 2024

From the spec:

image

@barmac barmac force-pushed the setup-axe-tests branch 2 times, most recently from a1174bc to 0eee8af Compare May 24, 2024 14:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in progress Currently worked on
Development

Successfully merging this pull request may close these issues.

None yet

1 participant