Skip to content

fix: dark mode contrast#1402

Open
scissorsneedfoodtoo wants to merge 5 commits into
mainfrom
fix/dark-mode-contrast
Open

fix: dark mode contrast#1402
scissorsneedfoodtoo wants to merge 5 commits into
mainfrom
fix/dark-mode-contrast

Conversation

@scissorsneedfoodtoo
Copy link
Copy Markdown
Contributor

Checklist:

This PR includes a handful of fixes for colors in dark mode.

It sets the color for text and links for post cards and pages to the secondary color variable, which is now set to #f5f6f7 to match what we have in the main repo. It also fixes the colors for the under header section on author and tag pages so text and social link buttons are visible, and adds colors to the footer section for better separation, similar to what we have on Learn.

There are some photos below for reference.

  • Current author / tag pages:
image
  • Author / tag pages with updates here:
image
  • Current footer:
image
  • Footer with updates here:
image

@scissorsneedfoodtoo scissorsneedfoodtoo marked this pull request as ready for review June 5, 2026 06:17
@scissorsneedfoodtoo scissorsneedfoodtoo requested a review from a team as a code owner June 5, 2026 06:17
--gray45: #858591;
--gray15: #d0d0d5;
--gray10: #dfdfe2;
--gray05: #eeeef0;
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there a reason we are changing thi?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was going through and noticed that this is different than what we havein the main repo.

But thinking back I do remember you saying that there's a reason for this a while back. I can go ahead and change it back now.

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.

3 participants