Skip to content

Comments

style: update footer sitemap link colors#491

Open
mi-vaishnavi wants to merge 2 commits intoaccordproject:mainfrom
mi-vaishnavi:footer-discord-hover
Open

style: update footer sitemap link colors#491
mi-vaishnavi wants to merge 2 commits intoaccordproject:mainfrom
mi-vaishnavi:footer-discord-hover

Conversation

@mi-vaishnavi
Copy link

@mi-vaishnavi mi-vaishnavi commented Feb 3, 2026

What this PR does

  • Updates the sitemap link colors in the footer for better visibility and consistency with the site theme.

Changes made

  • Modified website/core/Footer.js to adjust link styles.
  • Updated website/static/css/main.css to reflect the new hover and default colors.

Testing

  • Verified footer links visually in local development server.
  • Checked hover states and color contrast for readability.

Signed-off-by: Vaishnavi majojuvaishnavi@gmail.com

Signed-off-by: Vaishnavi <majojuvaishnavi@gmail.com>
@mi-vaishnavi
Copy link
Author

Hi team! I have updated the footer sitemap link colors to ensure they are consistent with the theme colors and have verified the hover states.

@mi-vaishnavi
Copy link
Author

Hi @mttrbrts @dselman

I’ve opened this PR to improve footer sitemap link visibility and align the hover styles with the site theme. The changes are tested locally.

This is one of my early contributions here, so I’d really appreciate your review and any feedback or suggestions.

Thanks for your time!

Copy link
Member

@mttrbrts mttrbrts left a comment

Choose a reason for hiding this comment

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

Does this meet accessibility guidelines for the necessary contrast ratio?

@mi-vaishnavi
Copy link
Author

mi-vaishnavi commented Feb 16, 2026

Hey @mttrbrts,
Thanks for pointing this out. I’ve updated the footer link color to ensure it meets accessibility contrast requirements while staying aligned with the site theme. Please let me know if any further adjustments are needed.

Signed-off-by: Vaishnavi <majojuvaishnavi@gmail.com>
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.

2 participants