Add Git action styles

[sorin-ionescu]

Right now, it is not possible to style Git actions am, cherry-pick, merge, rebase, etc. using any word, letter or unicode pictogram because the identifiers are hard coded in the _git-action function.

Should I add zstyles to remedy this?

[ColinHebert]

I don't think a lot of people use the git actions feature, while improving it could be nice, I don't think you should bother with that (unless someone actually wants this feature)

[sorin-ionescu]

I disagree. Everybody does merges. A few do rebases. I do interactive rebases all the time. Furthermore, everyone who trades patches via mailing lists uses git-am.

On Aug 24, 2012, at 5:25, Colin Hebert [email protected] wrote:

I don't think a lot of people use the git actions feature, while improving it could be nice, I don't think you should bother with that (unless someone actually wants this feature)

—
Reply to this email directly or view it on GitHub.

[ColinHebert]

I think I wasn't really clear; I didn't mean to say that nobody uses am, cherry-pick, merge, rebase, etc. I meant that I don't think that many people use the git action "status" in the prompt.

AFAIK, only two themes actually use git-info, because most themes consider that vcs_info gives enough information, and for the few which really use git-info I'm not sure there is a need to customise how the action status is displayed.

I could be wrong, but I think that it would preferable to work on that when someone asks for this feature specifically.

[sorin-ionescu]

I suppose, it would look something like the following.

zstyle ':prezto:module:git' action 'action:%s'
zstyle ':prezto:module:git:action' am 'am'
zstyle ':prezto:module:git:action' bisect 'bisect'
zstyle ':prezto:module:git:action' cherry-pick 'cherry'
zstyle ':prezto:module:git:action' cherry-pick-sequence 'cherry-seq'
zstyle ':prezto:module:git:action' merge 'merge'
zstyle ':prezto:module:git:action' rebase 'rebase'
zstyle ':prezto:module:git:action' rebase-interactive 'rebase-i'
zstyle ':prezto:module:git:action' rebase-merge 'rebase-m'

[ColinHebert]

Ah, this wasn't what I expected at all. I thought you wanted to create zstyle ':prezto:module:git' am, etc. to replace action (that was silly of me actually...).

This is a good idea (now that I understand what it is exactly)