Skip to content

Conversation

@JonasBa
Copy link
Member

@JonasBa JonasBa commented Dec 18, 2025

The ColorOrAlias type is problematic since it creates a coupling between how the theme is structured and which colors are available. Along the same lines, organizing our theme with a flat key list isn't a scalable long term approach.


Developers should use variants to encapsulate meanings or access colors directly on the theme if they need to.

@JonasBa JonasBa requested a review from a team as a code owner December 18, 2025 23:03
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Dec 18, 2025
@JonasBa JonasBa requested a review from Jesse-Box December 18, 2025 23:21
@JonasBa
Copy link
Member Author

JonasBa commented Dec 19, 2025

I'm going to hold off on merging as there might be some changes to the warning variant color set

@JonasBa JonasBa added the Do Not Merge Don't merge label Dec 19, 2025
@getsantry
Copy link
Contributor

getsantry bot commented Jan 10, 2026

This issue has gone three weeks without activity. In another week, I will close it.

But! If you comment or otherwise update it, I will reset the clock, and if you remove the label Waiting for: Community, I will leave it alone ... forever!


"A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀

@getsantry getsantry bot added the Stale label Jan 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Do Not Merge Don't merge Scope: Frontend Automatically applied to PRs that change frontend components Stale

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants