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

Incorrect element stack for flex and position absolute #4041

Closed
straker opened this issue May 25, 2023 · 1 comment · Fixed by #4086
Closed

Incorrect element stack for flex and position absolute #4041

straker opened this issue May 25, 2023 · 1 comment · Fixed by #4086
Assignees
Labels
fix Bug fixes pr A pr has been created for the issue support
Milestone

Comments

@straker
Copy link
Contributor

straker commented May 25, 2023

For the page https://www.sony.com/en/SonyInfo/csr/, axe.commons.dom.getElementStack gives an incorrect stack for the links in the navbar (e.g. "Contact Us" a[href$="SupportAll/"]) and reports them as needs review. I'll have to debug further as to the reason, but at a quick glance it looks like theres another header on the page that is claiming to be on top of the stack. The headers are using flex and some position: absolute.

@straker straker added fix Bug fixes support labels May 25, 2023
@straker straker changed the title Incorrect element stack for Incorrect element stack for flex and position absolute May 25, 2023
@WilcoFiers WilcoFiers modified the milestones: Axe-core 4.9, Axe-core 4.8 Jun 27, 2023
@straker straker self-assigned this Jun 29, 2023
@straker straker added pr A pr has been created for the issue and removed pr A pr has been created for the issue labels Jun 29, 2023
@straker straker added the pr A pr has been created for the issue label Jul 11, 2023
@straker
Copy link
Contributor Author

straker commented Jul 13, 2023

@padmavemulapati the fix did not resolve the needs review issue, so not much will have changed while running the page. However axe-core is reporting it correctly as there is indeed another element on top of the links that if it had a background would hide everything underneath if it had a background color. Going to say this is a dev task.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix Bug fixes pr A pr has been created for the issue support
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants