Skip to content

Conversation

filippo-zorzi
Copy link
Contributor

PR for #3864

Adding the suggestion presented in this PR #3638 by @detlevhfischer, @frankie-wolf and @bruce-usab.

@netlify
Copy link

netlify bot commented Aug 28, 2025

Deploy Preview for wcag2 ready!

Name Link
🔨 Latest commit c933285
🔍 Latest deploy log https://app.netlify.com/projects/wcag2/deploys/68f298b3a2380100083c9cf6
😎 Deploy Preview https://deploy-preview-4589--wcag2.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@patrickhlauke patrickhlauke self-assigned this Aug 29, 2025
@mbgower
Copy link
Contributor

mbgower commented Sep 26, 2025

@filippo-zorzi can you please check for any rationale for why example 3 was removed?

@filippo-zorzi
Copy link
Contributor Author

In the PR #3638 @bruce-usab made a comment on the third esample:

I do not agree that the ability to scroll is justification for small targets. I am open to be swayed. I think hypertext typically is excepted by being part of a sentence. If the target is part of in-page menu, would it not need to meet target size?

Since I wanted to bring over all the comments from the other PR, I had to find a way to highlight and remember the needed revision of the third example.

For sure this method is not the proper way to bring over a discussion given the fact that I forgot about that on last Friday meeting.

That commit is made to be changed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants