David, On Tue, Sep 3, 2019 at 10:52 AM Bert Wesarg <bert.wesarg@xxxxxxxxxxxxxx> wrote: > > Birger, > > On Mon, Sep 2, 2019 at 9:56 PM Birger Skogeng Pedersen > <birger.sp@xxxxxxxxx> wrote: > > > > Selecting whether to do a "New Commit" or "Amend Last Commit" does not have > > a hotkey. > > > > With this patch, the user may toggle between the two options with > > CTRL/CMD+e. > > David A. (in Cc from git-cola) suggested, that we try to re-use > existing key bindings in Git GUIs. Here is Git-Cola hotkeys: > > http://git-cola.github.io/share/doc/git-cola/hotkeys.html > > and this indicates, that it already uses CTRL+m for "amend". Though > I'm unsure if this is a toggle like in git-gui or a one-shot. David > A., can you please clarify? git-gui already assigns CTRL+m to "Local merge…" sorry. I think trying to synchronize the keybindings seems impossible. Bert > > > > > Signed-off-by: Birger Skogeng Pedersen <birger.sp@xxxxxxxxx> > > --- > > git-gui.sh | 11 +++++++++++ > > 1 file changed, 11 insertions(+) > > > > diff --git a/git-gui.sh b/git-gui.sh > > index 5bc21b8..14be1e0 100755 > > --- a/git-gui.sh > > +++ b/git-gui.sh > > @@ -2640,6 +2640,16 @@ proc show_less_context {} { > > } > > } > > > > +proc toggle_commit_type {} { > > + global selected_commit_type > > + if {[string match amend* $selected_commit_type]} { > > + set selected_commit_type new > > + } else { > > + set selected_commit_type amend > > + } > > + do_select_commit_type > > +} > > + > > ###################################################################### > > ## > > ## ui construction > > @@ -3843,6 +3853,7 @@ bind . <$M1B-Key-equal> {show_more_context;break} > > bind . <$M1B-Key-plus> {show_more_context;break} > > bind . <$M1B-Key-KP_Add> {show_more_context;break} > > bind . <$M1B-Key-Return> do_commit > > +bind . <$M1B-Key-e> toggle_commit_type > > The commit type has also a two toggle menu entries (under "Commit") > they should now also indicate the key binding. > disable > > Though how to express a toggle keybinding in the menu? I don't know if > you can assign the same keybinding to the same menu entry. Maybe we > need to add/remove the keybinding depending on the current mode. > > Bert > > > foreach i [list $ui_index $ui_workdir] { > > bind $i <Button-1> { toggle_or_diff click %W %x %y; break } > > bind $i <$M1B-Button-1> { add_one_to_selection %W %x %y; break } > > -- > > 2.21.0.windows.1 > >
