Skip to content
This repository was archived by the owner on Jul 27, 2025. It is now read-only.

Conversation

jespr
Copy link
Contributor

@jespr jespr commented Jan 2, 2024

@Shpigford is this what you had in mind for #3?

I've kept the naming from the graphic you added, so we're using primary-50 instead of cyan-50, primary-300 instead of cyan-300 and so on - but the others are still named using the color names: blue, orange, pink, grape, indigo and green.

I've also nulled out the standard tailwindcss colors - but we can change that of course.

@jespr
Copy link
Contributor Author

jespr commented Jan 2, 2024

I just noticed that Primary Cyan 50 and Primary Cyan 300 has the same hex code in the graphic?

@Shpigford
Copy link
Member

@jespr Thank you! Let's stick with the actual color names (instead of those "primary/error/etc"). Those colors will still be used outside of the context of errors/warnings.

@jespr
Copy link
Contributor Author

jespr commented Jan 2, 2024

@Shpigford changes pushed

Copy link
Member

@Shpigford Shpigford left a comment

Choose a reason for hiding this comment

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

Perfect! Thank you so much!

@Shpigford Shpigford merged commit 88bbcf9 into maybe-finance:main Jan 2, 2024
@Shpigford
Copy link
Member

Resolves #3

@jespr jespr deleted the jespr/tailwind-colors branch January 2, 2024 19:39
vinteo referenced this pull request in vinteo/maybe Jul 27, 2025
* Turkish language files added

Co-authored-by: Copilot <[email protected]>
Signed-off-by: Juan José Mata <[email protected]>

---------

Signed-off-by: Juan José Mata <[email protected]>
Co-authored-by: Juan José Mata <[email protected]>
Co-authored-by: Copilot <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants