Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add pink theme, convert hard-coded color names to variables #35

Closed
wants to merge 1 commit into from

Conversation

rio-codes
Copy link
Contributor

I created a pink palette and mode for risotto. I also modified logo.css and typography.css to use variables instead of hard-coded colors from the existing palettes, making it easier to add new palettes and modes in the future. Finally, I changed the existing dark and light mode files to use the original colors.

@rio-codes
Copy link
Contributor Author

I know this hasn't been approved yet but do you mind if I start working on one of the existing issues?

@joeroe
Copy link
Owner

joeroe commented Aug 18, 2022

Please go ahead. I'll review this as soon as I can :)

@rio-codes
Copy link
Contributor Author

Cool. Is there an existing issue that needs some love? I could maybe tackle #6 ?

@rio-codes
Copy link
Contributor Author

Hi, I was wondering if you mind if I start another theme for this? I've been working on this Catppuccin palette project and I'd love to make a risotto theme for it. If you don't like this pink one you can cancel it, but it would be nice to have the changes to the variables in it, maybe a separate PR?

@rio-codes
Copy link
Contributor Author

Breaking variable changes off into new PR.

@rio-codes rio-codes closed this Sep 20, 2022
@joeroe
Copy link
Owner

joeroe commented Sep 26, 2022

@rio-codes Sorry for the slow response. Both the pink and Catppuccin palettes are very welcome, if you still want to contribute them?

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.

2 participants