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

link-in-text-block needs to take in to account other ways to underline links within text. #4602

Open
1 task done
curtbellew opened this issue Oct 7, 2024 · 0 comments
Open
1 task done
Labels
ungroomed Ticket needs a maintainer to prioritize and label

Comments

@curtbellew
Copy link

Product

axe-core

Product Version

4.10.0

Latest Version

  • I have tested the issue with the latest version of the product

Issue Description

Expectation

If I remove text decoration and use box shadow to underline link text within a paragraph of other text I should not the "Links must be distinguishable without relying on color" error from "link-in-text-block".

Actual

when I remove text decoration from a link that is other wise default and visually create an underline using box shadow I do get the error "Links must be distinguishable without relying on color" back from axe-core

How to Reproduce

Use this code to reproduce the issue --

<p>
  This is some text in a paragraph before a link. And this is
  <a href="#" style="box-shadow: rgb(0, 104, 140) 0px -1px 0px 0px inset;
	text-decoration-line: none;">a link with class="oj-link-embedded"</a>
  in a sentence. This is some text in a paragraph after a link
</p>

Additional context

Sometimes we need to do some animation effects so we aren't able to rely on text decoration for the underline. We have to do something more custom using box shadow. In the end the effect is the same visually in that that link appears as it would by default, complete with an underline. I think axe-core must be looking for text decoration among other variables to determine if the link looks different than the surrounding text. Could it also look for a box shadow effect?

@curtbellew curtbellew added the ungroomed Ticket needs a maintainer to prioritize and label label Oct 7, 2024
@curtbellew curtbellew changed the title Need to take in to account other ways to underline links within text. link-in-text-block needs to take in to account other ways to underline links within text. Oct 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ungroomed Ticket needs a maintainer to prioritize and label
Projects
None yet
Development

No branches or pull requests

1 participant