-
Notifications
You must be signed in to change notification settings - Fork 310
Restyle doclinks box #4590
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
base: main
Are you sure you want to change the base?
Restyle doclinks box #4590
Conversation
✅ Deploy Preview for wcag2 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Apply proposed styles from #1130 (comment) Closes #1130
44f4fdc
to
2baec87
Compare
@kfranqueiro Could you see if you could find the errant |
Done |
Looks like this has spotted a bug in my TR-vs.-informative check. That failure should be ignored, because this PR only touches TR space. (Update: will be fixed by #4593) |
ah, magic, thanks @kfranqueiro (now realise why I didn't spot it...had my site-wide search in VS code filtered to just look at |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me!
For reviewers, here‘s the link to same bookmark as as in OP.
(Also, I attempted a diff version but it is not of utility as the boxes for Understanding and How To are not visible.
Apply proposed styles from #1130 (comment)
Closes #1130
Live preview: https://deploy-preview-4590--wcag2.netlify.app/guidelines/#text-alternatives