HalloweenBash icon indicating copy to clipboard operation
HalloweenBash copied to clipboard

Allow color switching intra-PS1

Open nicowilliams opened this issue 12 years ago • 10 comments

Like I do for my own. Just allow the color selections to get dragged into the PS1 box. This makes it easy to color branch and path names (for example) differently, so they stand out.

nicowilliams avatar Aug 14 '13 03:08 nicowilliams

+1

Likeyn avatar Aug 14 '13 08:08 Likeyn

Thanks for providing feedback. This feature would be helpful, but it would be very important to preserve the ease of use in the UI controls.

xta avatar Aug 15 '13 03:08 xta

Okay, what about a two step process then? The current color generator would create a box depending on which {fore,back}ground color you selected, which you could then drag into the configuration box to apply it.

Note that a "color reset" box would be probably needed as well.

Likeyn avatar Aug 15 '13 10:08 Likeyn

@Likeyn That would be one way to do it. I am concerned about having another step in the process and finding the right balance between ease of use and power features. There are many excellent color options that HalloweenBash does not handle at the moment; for many reasons including 1.) color controls are tricky to implement, 2.) a user should not feel overwhelmed with decisions to make and steps to take.

xta avatar Aug 15 '13 11:08 xta

You already handle all other elements the way I propose you handle color: as boxes to drag around, so UI-wise there's no complication. Just make sure to reset the colors automatically at the end of the prompt.

nicowilliams avatar Aug 15 '13 15:08 nicowilliams

@nicowilliams, picking multiple colors is a useful feature, but there are a lot of specific implementation details that remain unanswered.

Should you send a pull request, I'd be happy to review it. Thanks

xta avatar Aug 16 '13 04:08 xta

It's just a suggestion. I already have my own fairly hairy PS1 setup the way I want, including switching to different PS1s depending on whether $PWD is inside a git/hg/fossil workspace, and I'm not going to switch, so no PR from me (sorry). But it is nice to play with other possibilities using your HalloweenBash, and other users may find it very helpful. I'll stop making noise now :)

nicowilliams avatar Aug 16 '13 05:08 nicowilliams

I was also wondering about this. Even without the --no-color flag, everything still comes up in bash as the default foreground color.

For example:

function parse_git_branch {
   git branch 2> /dev/null | sed -e '/^[^*]/d' -e 's/* \(.*\)/(\1)/'
}

export PS1="\w \$(parse_git_branch) \@\n$ "

screen shot 2013-10-04 at 9 57 52 am

kevinSuttle avatar Oct 04 '13 13:10 kevinSuttle

Hi @kevinSuttle,

Are you referring to switching colors between different elements? Your example adds the current git branch.

Thanks for your feedback

xta avatar Oct 31 '13 06:10 xta

@Xta, yes. Switching between elements.

kevinSuttle avatar Nov 01 '13 02:11 kevinSuttle