-
Notifications
You must be signed in to change notification settings - Fork 25
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
Make semver color output more readable #74
Conversation
Signed-off-by: Micheal Mand <micheal@kmdwebdesigns.com>
I just had a thought, maybe I should leave |
Signed-off-by: Micheal Mand <micheal@kmdwebdesigns.com>
Signed-off-by: Micheal Mand <micheal@kmdwebdesigns.com>
Thanks for the PR 🎉 . Just a small question though: as the label is only colored now (I mean without background), what about separating it from the phrase before with a dash (ie Could you post a screen with your latest changes in colors to see what it looks like? 🙏 |
I can add a dash if you want me to, however, I don't think it is necessarily required. It might just clutter up the output. That's just my opinion, though. I will change whatever you want me to change. 👍 Sure, no problem. Here's what they look like now - again my theme on top: Edit: By the way, all these screenshots are coming from flopping between |
By the way, I don't know if it matters to you or not but these colors have just enough difference to still be useful for someone with some kind of color blindness. I use a Chrome extension to check my websites regularly, so it's kind of a habit that I check this too. |
That's perfect if you also checked this accessibility issue 🙌 |
Let's merge and release it! Thanks @mikemand 🎉 |
Woohoo! Thanks @pyrech! 🚀 |
just a small question: why not keeping the same idea of colors? I understand the need of change related to your issue and my original code, but the color swap seems reducing the primary idea of change :) thx |
Hi @noniagriconomie, I tried the yellow text on a white background but it was just as bad as white text on a yellow background. The green text was alright on the white background, but as you can see from my screenshot of a light theme (the default light theme that comes with macOS and probably iTerm and PuTTY as well) it gets kind of washed out. It is still legible, but it is a little bit hard to read. As a programmer, the last thing you need is anything that makes your eyes work harder since you are reading text all day long. If @pyrech wants the colors changed back, one of us can PR the change. Or maybe we could make it configurable somehow? I'm pretty sure a Composer plugin should have access to
{
...
"extra": {
"composer-changelogs": {
"colors": {
"major": "red",
"minor": "yellow",
"patch": "green"
}
}
}
...
} I don't know if this would even work or not, but it might be something to look into if the colors are a problem? |
For the record this plugin already proposes a system to handle configuration in composer.json (local or global). See https://github.com/pyrech/composer-changelogs/blob/main/doc/configuration.md About the colors I must admit that I don't really know what the best choice here. It's mainly about standard colors VS readability. If we can't agree here, maybe the configuration option is the way to go? |
I've required v1.8.0 on my projetcs, fg/bg colors suits me well :) Thank you |
As per @pyrech here: #73 (comment)
I went with magenta for
major
, blue forminor
, and cyan forpatch
. I don't know what other colors we have available. Most terminals should be at least 16 colors these days, but I figure sticking with the typical 8 colors would be safer.The blue might be a bit too dark. Let me know what other colors you would like me to try.
Here's what my terminal looks like, followed by a typical light theme and a typical dark theme: