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

refactor aria-utils #29

Closed
cookiecrook opened this issue Apr 22, 2023 · 1 comment · Fixed by web-platform-tests/wpt#39724
Closed

refactor aria-utils #29

cookiecrook opened this issue Apr 22, 2023 · 1 comment · Fixed by web-platform-tests/wpt#39724
Assignees
Labels

Comments

@cookiecrook
Copy link
Collaborator

cookiecrook commented Apr 22, 2023

After web-platform-tests/wpt#39604 lands, refactor aria-utils.js to avoid assigning an ID if it doesn't have to... I thought I had tried that initially, but assumed it didn't work without an ID once the scope was passed to the webdriver context...

In hindsight, I must've just had a bug, because it clearly works without an ID in @nolanlawson's patch for web-platform-tests/wpt#36541

This will likely remove the ID check and assignment loops in all three of the methods inside aria-utils.js, and then pass in the element ref directly to the async getter.

@cookiecrook
Copy link
Collaborator Author

Agenda+ to get an r+

cookiecrook added a commit to web-platform-tests/wpt that referenced this issue Jun 6, 2023
moz-v2v-gh pushed a commit to mozilla/gecko-dev that referenced this issue Jun 13, 2023
…y ID usage, a=testonly

Automatic update from web-platform-tests
refactor aria-utils to remove unnecessary ID usage (#39724)

Resolves web-platform-tests/interop-accessibility#29

Refactors aria-utils to remove unnecessary ID usage.
--

wpt-commits: 886b8496dcfb6f9489f57867a1767eee244414f8
wpt-pr: 39724
ErichDonGubler pushed a commit to erichdongubler-mozilla/firefox that referenced this issue Jun 14, 2023
…y ID usage, a=testonly

Automatic update from web-platform-tests
refactor aria-utils to remove unnecessary ID usage (#39724)

Resolves web-platform-tests/interop-accessibility#29

Refactors aria-utils to remove unnecessary ID usage.
--

wpt-commits: 886b8496dcfb6f9489f57867a1767eee244414f8
wpt-pr: 39724
gecko-dev-updater pushed a commit to marco-c/gecko-dev-comments-removed that referenced this issue Jun 16, 2023
…y ID usage, a=testonly

Automatic update from web-platform-tests
refactor aria-utils to remove unnecessary ID usage (#39724)

Resolves web-platform-tests/interop-accessibility#29

Refactors aria-utils to remove unnecessary ID usage.
--

wpt-commits: 886b8496dcfb6f9489f57867a1767eee244414f8
wpt-pr: 39724

UltraBlame original commit: 2e5571db57132b0acd6a65e8baf83a2fc4d4ad5b
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified-and-comments-removed that referenced this issue Jun 16, 2023
…y ID usage, a=testonly

Automatic update from web-platform-tests
refactor aria-utils to remove unnecessary ID usage (#39724)

Resolves web-platform-tests/interop-accessibility#29

Refactors aria-utils to remove unnecessary ID usage.
--

wpt-commits: 886b8496dcfb6f9489f57867a1767eee244414f8
wpt-pr: 39724

UltraBlame original commit: 2e5571db57132b0acd6a65e8baf83a2fc4d4ad5b
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified that referenced this issue Jun 16, 2023
…y ID usage, a=testonly

Automatic update from web-platform-tests
refactor aria-utils to remove unnecessary ID usage (#39724)

Resolves web-platform-tests/interop-accessibility#29

Refactors aria-utils to remove unnecessary ID usage.
--

wpt-commits: 886b8496dcfb6f9489f57867a1767eee244414f8
wpt-pr: 39724

UltraBlame original commit: 2e5571db57132b0acd6a65e8baf83a2fc4d4ad5b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants
@cookiecrook and others