Skip to content

Conversation

@ace-of-aces
Copy link
Contributor

πŸ”— Linked issue

Resolves #1544

❓ Type of change

  • πŸ“– Documentation (updates to the documentation, readme or JSdoc annotations)
  • 🐞 Bug fix (a non-breaking change that fixes an issue)
  • πŸ‘Œ Enhancement (improving an existing functionality like performance)
  • ✨ New feature (a non-breaking change that adds functionality)
  • 🧹 Chore (updates to the build process or auxiliary tools and libraries)
  • ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

πŸ“š Description

This brings the default green theme back to the green color palette instead of the lime palette that breaks with the rest of the branding and green colors used for charts.

Just for a more consistent look ❀️

πŸ“Έ Screenshots (if appropriate)

Before:

Screenshot 2025-11-16 at 11 52 58 Screenshot 2025-11-16 at 11 53 15

After:

Screenshot 2025-11-16 at 11 51 09 Screenshot 2025-11-16 at 11 51 24

πŸ“ Checklist

  • I have linked an issue or discussion.
  • I have updated the documentation accordingly.

@zernonia
Copy link
Member

Thanks for the PR @ace-of-aces , I was thinking of replacing the color as well, but that would cause the color to deviate from the original. Im thinking we can add it as emerald color, and keep the current green as is? WDYT? 😁

@ace-of-aces
Copy link
Contributor Author

@zernonia Good point, I only noticed just now that shadcn/ui has the green theme.. (thought this was a shadcn-vue only thingπŸ˜…)

I'll revert my changes in the green theme then, but also apply the lime palette to the chart- variables to have it consistent with the original, if that's okay.

Regarding the extra theme, would you use the tailwind emerald palette then, or only use this name for the theme?
If you meant the latter, I think we shouldn't create confusion with the tailwind names, so maybe something different like mint?

And should we then set this new theme as the default for users visiting the website? :)

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.

Green theme is visually inconsistent

2 participants