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.

Sign up for GitHub

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

Jump to bottom

Merge legacy NodeWalker to DomWalker #1971

Closed
Tracked by #1970
shunguoy opened this issue Aug 6, 2024 · 0 comments
Closed
Tracked by #1970

Merge legacy NodeWalker to DomWalker #1971

shunguoy opened this issue Aug 6, 2024 · 0 comments
Assignees
@shunguoy
Labels
engine Issues in the accessibility-checker-engine component enhancement New feature or request

Comments

@shunguoy
Copy link
Contributor

shunguoy commented Aug 6, 2024

The DomWalker constructor can take a Node and can do what NodeWalker does. The functions from NodeWalker need to be merged to DomWalker, and NodeWalker needs to be removed.
@shunguoy shunguoy added enhancement New feature or request engine Issues in the accessibility-checker-engine component labels Aug 6, 2024
@shunguoy shunguoy self-assigned this Aug 6, 2024
@MHoov MHoov mentioned this issue Aug 9, 2024
Closed
10 tasks
@philljenkins philljenkins mentioned this issue Aug 14, 2024
Closed
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@shunguoy shunguoy

Labels
engine Issues in the accessibility-checker-engine component enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@shunguoy