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

Make links underlined by default #1684

Closed
wants to merge 3 commits into from
Closed

Make links underlined by default #1684

wants to merge 3 commits into from

Conversation

froboy
Copy link

@froboy froboy commented Oct 2, 2023

As per the change upstream in Bootstrap 5 twbs/bootstrap#30389

Removing the underline from links was in vogue for a while, but WebAIM recommends not doing that.

Browsers underline hypertext links by default. It is possible to remove the underline using Cascading Style Sheets (CSS), but this is a bad idea most of the time.

@google-cla
Copy link

google-cla bot commented Oct 2, 2023

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@froboy
Copy link
Author

froboy commented Oct 2, 2023

Cursor_and_Welcome_to_Docsy___Docsy
Maybe we just do this for the body? Or we leave the default and override for nav, aside

@froboy
Copy link
Author

froboy commented Oct 2, 2023

Ok, sorry for the mess of changes here, and for not opening an issue first, but this is ready for review now. Thanks for your consideration and all of your work.

We're using this in production for the YMCA Website Services documentation at https://ds-docs.y.org/ and it's gotten fantastic feedback.

@chalin chalin added design/style Front-end site design / styling accessibility labels Feb 2, 2024
@chalin chalin added this to the 24Q1 milestone Feb 2, 2024
@chalin
Copy link
Collaborator

chalin commented Feb 2, 2024

Thanks for this PR!

Nice website, you might consider submitting a PR to added it to https://www.docsy.dev/docs/examples/?

I agree that having underlines in the body only (as you've done on your website) is probably best. I'll take a closer look soon.

@chalin
Copy link
Collaborator

chalin commented Feb 2, 2024

Here's the issue to track this:

Also, I think I have an alternative to propose that will contribute to #1655 at the same time.

@chalin
Copy link
Collaborator

chalin commented Feb 2, 2024

Thanks again for this PR, it helped kick this off. A fix has landed, see:

@froboy
Copy link
Author

froboy commented Feb 3, 2024

Thank you! I appreciate your consideration of the issue and I'll get our site submitted to the list of examples. Thanks for your work on Docsy!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accessibility design/style Front-end site design / styling
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants