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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: bright colors variants #55

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

MHase
Copy link

@MHase MHase commented Aug 15, 2023

Bright colors variants as mentioned in #47 馃槃
Also added few empty lines separating colors group for easier navigating through the code

@MHase MHase force-pushed the feat/bright-colors-variants branch from 658a2ae to 92e88c2 Compare August 15, 2023 17:03
@MrFoxPro
Copy link

Is it possible to add orange color?

@MHase
Copy link
Author

MHase commented Aug 30, 2023

@MrFoxPro absolutely I can contribute to that :) but this PR focus only on adding corresponding/existing colors in bright variants. if that will/would be accepted then I can proceed in adding something more 馃憤馃徎

@WaldoJeffers
Copy link

Thanks for your work @MHase
Any chance of seeing this merged?
I feel it would really help people who are migrating from chalk -> picocolors

Or are there any issues with the PR, or the impact it would have on the overall size?

@MHase
Copy link
Author

MHase commented Dec 13, 2023

@WaldoJeffers I would love to see this merged, but we need someone to approve/merge this PR for us
Only those with write access to this repository can merge pull requests.

Any idea who we can ping to move this forward?

@MrFoxPro
Copy link

@alexeyraspopov

@jaclas
Copy link

jaclas commented Feb 2, 2024

Still not approved? What is it about? I also need "bright" colors.
@marco-carvalho @borodean @alexeyraspopov ?

@MHase
Copy link
Author

MHase commented Feb 2, 2024

based on responses it looks like this repo was abandoned...

@jaclas
Copy link

jaclas commented Feb 2, 2024

15 million projects use this module, maybe someone will take it further...

@alexeyraspopov
Copy link
Owner

The library is mostly used indirectly, by other more significant projects. I appreciate the PRs and will get back to them as soon as I got time, but I can鈥檛 just prioritize PRs that start bloating the size for little to no benefit.

Thanks

@jaclas
Copy link

jaclas commented Feb 4, 2024

@alexeyraspopov Thanks for showing up!
This PR increases the size by just a few hundred bytes, in return doubles the number of available colors. And most importantly, it fulfills the promise of easy migration from Chalk, where *Bright colors are present.

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.

None yet

5 participants