Skip to content

Conversation

@jellydeck
Copy link
Contributor

@jellydeck jellydeck commented Feb 1, 2026

closes #611 resolves #633

  • use accent colors for focus rings in navbar
  • AppLogo component which changes color as per accent color
  • fixes in spacing for visual alignment

Major visual change:
docs bar is below description to accommodate for long package names
image

@vercel
Copy link

vercel bot commented Feb 1, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
npmx.dev Ready Ready Preview, Comment Feb 1, 2026 4:42pm
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
docs.npmx.dev Ignored Ignored Preview Feb 1, 2026 4:42pm
npmx-lunaria Ignored Ignored Feb 1, 2026 4:42pm

Request Review

@jellydeck jellydeck marked this pull request as ready for review February 1, 2026 12:11
@jellydeck jellydeck marked this pull request as draft February 1, 2026 12:12
@jellydeck jellydeck marked this pull request as ready for review February 1, 2026 12:18
@jellydeck jellydeck changed the title feat: better contrast for accent colors feat: better contrast for accent colors & visual changes Feb 1, 2026
@whitep4nth3r
Copy link
Collaborator

These are starting to look better in terms of consistency but I checked out the deployment and the colours chosen for focus outlines do still fail colour contrast. Before I saw this PR I actually opened this issue for reference.

#647

@whitep4nth3r
Copy link
Collaborator

This is looking specifically at the provenance icon in dark mode.

image

default accent-color
otherwise half opacity
@jellydeck
Copy link
Contributor Author

@whitep4nth3r I think accent would work great here, what do you think?

@whitep4nth3r
Copy link
Collaborator

There's actually another PR going on over here for the same thing 😅

#650

@whitep4nth3r
Copy link
Collaborator

Accent colour could work but I'm not sure if all the colours available in the settings have been tested against the background colours.

@jellydeck
Copy link
Contributor Author

Accent colour could work but I'm not sure if all the colours available in the settings have been tested against the background colours.

tweaked colors and opacity of accent colors on both themes, should have no issue by 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.

Download chart tooltip is not visible in light mode Missing focus indicators in banner connect menu [Safari/a11y]

3 participants