Skip to content

Conversation

david-a-wheeler
Copy link
Contributor

Switch to OpenSSF brand colors for hyperlink text. The OpenSSF Brand Guidelines are here:
https://openssf.org/about/brand-guidelines/

We changed best.openssf.org to use OpenSSF's fonts long ago, but we never changed the hyperlink colors to adhere to the guidelines. This commit changes hyperlinks to also use OpenSSF brand colors.

I intentionally chose dark colors to provide strong contrast with the white background. This site provides mostly documents, with with text that is smaller than the main OpenSSF website. So ensuring we have good contrast between the text and the background seemed especially important.

For more about changing link colors, see:
https://www.w3schools.com/css/css_link.asp

I suggest that we can argue about changing these colors later. Once this commit is accepted, it's much easier to change the colors again :-).

Switch to OpenSSF brand colors for hyperlink text.
The OpenSSF Brand Guidelines are here:
https://openssf.org/about/brand-guidelines/

We changed best.openssf.org to use OpenSSF's fonts long ago,
but we never changed the hyperlink colors to adhere to the
guidelines. This commit changes hyperlinks to also use
OpenSSF brand colors.

I intentionally chose dark colors to provide *strong* contrast with
the white background.  This site provides mostly documents, with
with text that is smaller than the main OpenSSF website.
So ensuring we have *good* contrast between the text and the background
seemed especially important.

For more about changing link colors, see:
https://www.w3schools.com/css/css_link.asp

I suggest that we can argue about changing these colors later.
Once this commit is accepted, it's much easier to change the colors again :-).

Signed-off-by: David A. Wheeler <[email protected]>
@david-a-wheeler
Copy link
Contributor Author

No comments, so I'll merge. We can always change this later, now that there's a clear place to make the change.

@david-a-wheeler david-a-wheeler merged commit e3677e6 into main Jan 21, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant