Skip to content

Conversation

homotechsual
Copy link
Contributor

Add default underline to links for better non-colour dependant compliance with WCAG 1.4.1 (rule id: link-in-text-block)

Add default underline to links for better non-colour dependant compliance with WCAG 1.4.1 (rule id: link-in-text-block)
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Sep 6, 2023
@netlify
Copy link

netlify bot commented Sep 6, 2023

Deploy Preview for infima ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 09a99e2
🔍 Latest deploy log https://app.netlify.com/sites/infima/deploys/64f888fabf38940008523603
😎 Deploy Preview https://deploy-preview-290--infima.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 site configuration.

@Josh-Cena
Copy link
Contributor

I think this can be opt-in, but I doubt most users are going to welcome this change.

@clicktodev
Copy link
Contributor

I agree it should be off by default. Users can easily override the css variable.

It doesn't look good in the navbar

image

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

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants