On Sun, Jan 17, 2010 at 03:31:07PM +1100, Michael Wookey wrote: > > There is an existing highlight when the user is not on any branch. > > Enhance this functionality to always provide the name of the current > > branch in color. > > > > Signed-off-by: Michael Wookey <michaelwookey@xxxxxxxxx> > > --- > > wt-status.c | 10 ++++++---- > > wt-status.h | 3 ++- > > 2 files changed, 8 insertions(+), 5 deletions(-) > > Any thoughts on this? I don't have an objection to the concept, but for it to be acceptable for inclusion in mainstream git, the color selection needs to be configurable (see builtin-commit.c:parse_status_slot) and documented (see Documentation/config.txt, color.status.<slot>). And then we can decide what the _default_ color should be. Personally, I find it a bit distracting to change the color for something so mundane, and would be in favor of leaving it the same as the header color. But I will not argue too hard either way; I have no problem with setting my own config to make it invisible if others disagree. :) -Peff -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html