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

Changed visited-link and link colors in default theme to be more accessible #16413

Merged
merged 2 commits into from
Mar 4, 2025

Conversation

Godmartinz
Copy link
Collaborator

@Godmartinz Godmartinz commented Mar 4, 2025

Changes the visited-linkand link colors to have better contrast against the grey background color as well as be distinguishable between eachother.

Before:
image

After:
image
image
[sc-28469]

Copy link

what-the-diff bot commented Mar 4, 2025

PR Summary

  • Updating Color of Links
    The color for links displayed on the website has been updated from a darker shade of blue to a new color (No multilanguage support for slack messages #3366). This change is expected to improve the visibility and accessibility of the links.

  • Modifying Visited Link Color
    The color of previously visited links has been adjusted from a lighter shade of blue to a lighter version of the new link color (No multilanguage support for slack messages #3366, lightened by 20%). This should make it easier for users to distinguish between the links they have already clicked on and the ones they haven't.

@Godmartinz Godmartinz requested a review from marcusmoore March 4, 2025 18:16
@Godmartinz Godmartinz changed the title changes visited and link colors Changed visited-link and link colors in default theme to be more accessible Mar 4, 2025
@snipe snipe merged commit f25b837 into snipe:develop Mar 4, 2025
9 checks passed
@snipe
Copy link
Owner

snipe commented Mar 5, 2025

This change overall looks good, but it's also changed the color of other links, like the breadcrumbs, and the blue is a lot colder. The "wrong" blue, versus the more grayish blue we use for the headerl.

Screenshot 2025-03-05 at 11 42 15 AM

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.

2 participants