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

Feature request | arrow prompt ❯ color #303

Closed
fazlearefin opened this issue Mar 13, 2017 · 13 comments · Fixed by #472
Closed

Feature request | arrow prompt ❯ color #303

fazlearefin opened this issue Mar 13, 2017 · 13 comments · Fixed by #472

Comments

@fazlearefin
Copy link

fazlearefin commented Mar 13, 2017

The arrow prompt color for normal ($? -eq 0) and not normal ($? -ne 0) is not very obvious. I suggest you use grey or green for the arrow prompt when normal ($? -eq 0).

@fazlearefin
Copy link
Author

For this to happen, the only change required is to replace magenta with green in pure.zsh

- PROMPT='%(?.%F{magenta}.%F{red})${PURE_PROMPT_SYMBOL:-❯}%f '
+ PROMPT='%(?.%F{green}.%F{red})${PURE_PROMPT_SYMBOL:-❯}%f '

Magenta and red are not good combinations as on some color schemes it is a bit difficult to tell them apart. Green and red should be easy to spot.

@sindresorhus
Copy link
Owner

I don't want a green prompt character. That looks boring. How about we instead change the error color to yellow?

@fazlearefin
Copy link
Author

Changing the error color to yellow is fine with me.

@mikeerickson
Copy link
Sponsor

@sindresorhus @fazlearefin Yes, I agree as well on yellow. That is a biggest enough difference imo

@fazlearefin
Copy link
Author

I can make the change and raise a pull request if that helps get this change to the repo quicker.

@mafredri
Copy link
Collaborator

I'm not a fan to switching the color to yellow, too close to green (== OK). And it does look quite out of place.

Thoughts on #306?

@mafredri
Copy link
Collaborator

To demonstrate what it looks like for me:

screenshot 2017-03-23 18 34 13

@anton-johansson
Copy link

What about using options for the two colors in .zshrc? They can default to the same values that are currently being used.

PURE_PROMPT_NORMAL_COLOR=green
PURE_PROMPT_ERROR_COLOR=red

@mafredri
Copy link
Collaborator

@will-stone this is being considered, I think both me and @sindresorhus agree that we don't want new environment variables for configuring this, though. So using zstyle as proposed in #306 seems like the most likely candidate, I just haven't gotten around to it yet 😅.

This feature has been quite low-prio for me personally, so if anyone is interested to help, a proposal / PR is definitely welcomed. To start with, I think it makes sense to limit this to colorizing the prompt character only, in the future more colors customizations could be introduced, but let's see how it works first.

@fazlearefin
Copy link
Author

I'll raise a PR if I get some time and manage to get it to work as proposed in #306 .

@alexozer
Copy link

How's this going?

@craftyguy
Copy link

I hope it's still going. It's very hard for my eyes to differentiate between the default color (magenta-ish) and the error color (red), considering the prompt character is really thin/small.

@disrupted
Copy link

@craftyguy +1 same here

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 a pull request may close this issue.

8 participants