fix-pr-tooltip-contrast#710
Conversation
|
@tharunika-19 is attempting to deploy a commit to the PRIYANSHU DOSHI's projects Team on Vercel. A member of the Team first needs to authorize it. |
GSSoC Label Checklist 🏷️@Priyanshu-byte-coder — please apply the appropriate labels before merging: Difficulty (pick one):
Quality (optional):
Validation (required to score):
|
Priyanshu-byte-coder
left a comment
There was a problem hiding this comment.
Two issues:
-
Tooltip CSS vars broken: The tooltip was previously using
getCSSVariable('--tooltip')andgetCSSVariable('--tooltip-foreground')— the new code replaces these with hardcoded hex values (#111827,#374151,#ffffff). This breaks dark/light mode switching and our CSS variable system. Please restore the CSS var usage (or usegetCSSVariable('--card')/getCSSVariable('--foreground')if --tooltip isn't defined). -
Scope: The custom
/auth/signinpage andauth.tspages config are a separate feature from the tooltip contrast fix. If you want to add the custom signin page, please open a separate PR for it.
Related Issue
Closes #605
Changes Made
Before
After