[VikasKSingh05]

Resolves #4126

Changes

Accessibility color updates

• Renamed p5-contrast-pink to p5-contrast-light-pink (#FFA9D9)
• Added a new p5-contrast-dark-pink color (#B20046)
• Updated existing references to use p5-contrast-light-pink

Theme-specific link colors

• Light theme links now use p5-contrast-dark-pink
• Dark theme links now use p5-contrast-light-pink
• Contrast theme links continue to use p5-contrast-light-pink

Link styling updates

• Updated shared link styling to use the theme-specific link-text-color variable instead of hardcoded color values
• Updated About page contact/footer links to use the accessibility-compliant theme link color
• Preserved existing hover and underline behavior

Why

This change improves link accessibility by ensuring link colors meet WCAG contrast requirements for small text while maintaining consistent behavior across themes.

QA Steps

1. Start the web editor locally.
2. Switch to Light Mode.
   • Verify links appear in dark pink (#B20046).
3. Switch to Dark Mode.
   • Verify links appear in light pink (#FFA9D9).
4. Switch to Contrast Mode.
   • Verify links appear in light pink (#FFA9D9).
5. Open the Preferences dialog and verify links use the correct theme color.
6. Open the About page and verify contact/footer links use the correct theme color.
7. Confirm links remain underlined and visually distinguishable from surrounding text in all themes.

Screenshots

Light Mode

Dark Mode

[runningonsideprojects]

✅ Light mode link color #b20046

✅ Dark mode link color #ffa9d9

✅ Contrast mode link color #ffa9d9

[runningonsideprojects]

hey @VikasKSingh05 incredible sorry for this belated review. Please take a look at my comments! Eager to get your PR merged soon. Thanks for the great work on this

[runningonsideprojects]

@VikasKSingh05, gently checking in!

per p5js policy, we may close this PR if we do not receive a timely follow up on your status!

[VikasKSingh05]

Hi @runningonsideprojects, thanks for checking in! I'm still interested in getting this PR merged. I’ve completed the changes requested for #4159 and the PR currently reflects the three theme-specific link colors you confirmed. I don’t see any unresolved review comments on the PR at the moment. Please let me know if there’s anything else you’d like me to address, and I’ll take care of it.

[runningonsideprojects]

Hi @runningonsideprojects, thanks for checking in! I'm still interested in getting this PR merged. I’ve completed the changes requested for #4159 and the PR currently reflects the three theme-specific link colors you confirmed. I don’t see any unresolved review comments on the PR at the moment. Please let me know if there’s anything else you’d like me to address, and I’ll take care of it.

Thanks! Are you unable to see these comments in this PR?

[VikasKSingh05]

@runningonsideprojects, I've addressed all three review comments: moved link-text-color into alphabetical order for all three themes, removed the extra space after the comma, and updated the dark theme error-color to $p5-contrast-light-pink. I've committed the changes to the PR. Please take another look when you get a chance!

[runningonsideprojects]

looks great! thank you for your amazing work on this :) @VikasKSingh05

[runningonsideprojects]

oh one more thing @VikasKSingh05 can u your branch is rebased with the master branch properly?

[VikasKSingh05]

I've rebased the PR branch onto the latest develop and pushed the updated changes. Please take another look when you get a chance @runningonsideprojects !