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

Aria 1.1. tests #509

Merged
merged 4 commits into from
Sep 6, 2017
Merged

Aria 1.1. tests #509

merged 4 commits into from
Sep 6, 2017

Conversation

marcysutton
Copy link
Contributor

@marcysutton marcysutton commented Sep 5, 2017

Hopefully I got them all! I didn't see a way to test the owned/context changes for treegrid in https://github.com/dequelabs/axe-core/pull/479/files.

Also, we should really find a way to replace the hard-coded IDs in those integration test files. Couldn't we generate those with JS? Reordering them is such a huge pain.

@nschonni
Copy link
Contributor

nschonni commented Sep 6, 2017

Probably a dumb question, but what IDs these as 1.1 tests vs 1.2?

@marcysutton
Copy link
Contributor Author

@nschonni I assume you mean ARIA 1.1 vs. ARIA 1.2? We don't differentiate ARIA versions really. We could tag them potentially, but I don't know if that would be useful. We simply don't add anything to axe-core that isn't widely supported. See #91 and #350.

@marcysutton marcysutton merged commit 7f3f05d into develop-2x Sep 6, 2017
marcysutton added a commit that referenced this pull request Sep 6, 2017
* feat(ARIA 1.1): Allow row-index, setsize and posinset on more roles

* test(aria): aria-haspopup, aria-modal for 1.1

* style(test/aria): add line breaks for readability

* test(aria): add missing roles and properties
@nschonni
Copy link
Contributor

nschonni commented Sep 7, 2017

@marcysutton thanks for the explanation. I was thinking more back to the Quail days where rulesets were strongly typed

@marcysutton marcysutton deleted the aria-1.1.-tests branch December 7, 2017 18:25
mrtnvh pushed a commit to mrtnvh/axe-core that referenced this pull request Nov 24, 2023
* chore: update webdriverio peerDependency

We believe the issue that caused us to pin to 7.14.1 was this one webdriverio/webdriverio#7994. Testing versions of webdriverio 7.14 - 7.19 no longer causes us any issues. The issue seems to have been resolved and we no longer need to pin to a specific version.

* update version

* regenerate
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.

3 participants