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

[BUG]: a_target_warning should pick up text alternatives to indicate the new window #1951

Open
tombrunet opened this issue Jul 23, 2024 · 1 comment
Labels
Bug Something isn't working engine Issues in the accessibility-checker-engine component T62

Comments

@tombrunet
Copy link
Member

tombrunet commented Jul 23, 2024

Project

accessibility-checker-engine

Browser

Chrome

Operating system

MacOS

Description

The a_target_warning is looking for indications of a new window in the text or title. If there's an icon with alt text to indicate the new window, that is not detected, so it files a false positive. Note that this is a recommendation, but still should be a straight forward fix.

Steps to reproduce

<a href="#" target="_blank">Some link<img alt="opens in new window" src=""></a>

Note that this should also not fire if the alt text is in an svg (title, aria-label, etc). See Carbon link with icon for example of that structure

@MHoov MHoov added engine Issues in the accessibility-checker-engine component Bug Something isn't working labels Jul 23, 2024
@shunguoy shunguoy added the T62 label Jul 28, 2024
@shunguoy
Copy link
Contributor

Triage: the NR shouldn't be reported if the link has an alt, or it's an icon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working engine Issues in the accessibility-checker-engine component T62
Projects
None yet
Development

No branches or pull requests

3 participants