Skip to content

Conversation

@ecmasx
Copy link
Contributor

@ecmasx ecmasx commented Aug 28, 2025

Important

Update styles for GitHub install button and theme toggle to use HSL color functions and add transitions for smoother interactions.

  • GitHub Install Button Styles:
    • Updated background, color, and hover styles in styles.module.css to use hsl(var(--variable)) for consistency.
    • Added transition: all 0.2s ease and scale: 1.05 on hover for smoother interaction.
  • Theme Toggle Styles:
    • Updated background-color and color on hover in styles.module.css to use hsl(var(--variable)) for consistency.
    • Updated box-shadow for focus-visible state to use hsl(var(--ring)).
  • Misc:
    • Removed hsl() wrapper from --primary-hover and --primary-dark-hover in custom.css for consistency.

This description was created by Ellipsis for f55b318. You can customize this summary. It will automatically update as commits are pushed.

@vercel
Copy link

vercel bot commented Aug 28, 2025

@ecmasx is attempting to deploy a commit to the Roo Code Team on Vercel.

A member of the Team first needs to authorize it.


/* GitHub install button colors */
--primary-hover: hsl(0 0% 9% / 0.9);
--primary-hover: 0 0% 9% / 0.9;
Copy link
Contributor

Choose a reason for hiding this comment

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

Ensure updated custom properties (e.g. --primary-hover) are wrapped in hsl() where used.


/* GitHub install button colors for dark mode */
--primary-dark-hover: hsl(0 0% 98% / 0.9);
--primary-dark-hover: 0 0% 98% / 0.9;
Copy link
Contributor

Choose a reason for hiding this comment

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

Typographical error: It appears the CSS function 'hsl' was removed from the '--primary-dark-hover' value. The original value 'hsl(0 0% 98% / 0.9)' seems correct. Please restore the 'hsl(' part.

Suggested change
--primary-dark-hover: 0 0% 98% / 0.9;
--primary-dark-hover: hsl(0 0% 98% / 0.9);

@ecmasx
Copy link
Contributor Author

ecmasx commented Aug 28, 2025

image image I changed the colors for the buttons because there were problems with the hover effect.

@ecmasx
Copy link
Contributor Author

ecmasx commented Aug 28, 2025

these are old
image

@daniel-lxs daniel-lxs moved this from Triage to PR [Needs Prelim Review] in Roo Code Roadmap Aug 29, 2025
@vercel
Copy link

vercel bot commented Aug 29, 2025

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

Project Deployment Preview Comments Updated (UTC)
roo-code-docs Ready Ready Preview Comment Aug 29, 2025 10:53pm

Copy link
Member

@daniel-lxs daniel-lxs left a comment

Choose a reason for hiding this comment

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

LGTM

@daniel-lxs daniel-lxs merged commit 175d879 into RooCodeInc:main Aug 29, 2025
3 checks passed
@github-project-automation github-project-automation bot moved this from PR [Needs Prelim Review] to Done in Roo Code Roadmap Aug 29, 2025
@ecmasx ecmasx deleted the fix/navbar-colors branch September 5, 2025 16:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

No open projects
Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants