Split commit message panel into commit summary and commit description…

… panel

When we use the one panel for the entire commit message, its tricky to have a keybinding both for adding a newline and submitting.
By having two panels: one for the summary line and one for the description, we allow for 'enter' to submit the message when done from the summary panel,
and 'enter' to add a newline when done from the description panel. Alt-enter, for those who can use that key combo, also works for submitting the message
from the description panel. For those who can't use that key combo, and don't want to remap the keybinding, they can hit tab to go back to the summary panel
and then 'enter' to submit the message.

We have some awkwardness in that both contexts (i.e. panels) need to appear and disappear in tandem and we don't have a great way of handling that concept,
so we just push both contexts one after the other, and likewise remove both contexts when we escape.

docs/Config.md

@@ -171,7 +171,6 @@ keybinding:
  diffingMenu-alt: '<c-e>' # deprecated
  copyToClipboard: '<c-o>'
  submitEditorText: '<enter>'
- appendNewline: '<a-enter>'
  extrasMenu: '@'
  toggleWhitespaceInDiffView: '<c-w>'
  increaseContextInDiffView: '}'

pkg/cheatsheet/generate.go

@@ -87,33 +87,34 @@ func writeString(file *os.File, str string) {
 
 func localisedTitle(tr *i18n.TranslationSet, str string) string {
  contextTitleMap := map[string]string{
- "global": tr.GlobalTitle,
- "navigation": tr.NavigationTitle,
- "branches": tr.BranchesTitle,
- "localBranches": tr.LocalBranchesTitle,
- "files": tr.FilesTitle,
- "status": tr.StatusTitle,
- "submodules": tr.SubmodulesTitle,
- "subCommits": tr.SubCommitsTitle,
- "remoteBranches": tr.RemoteBranchesTitle,
- "remotes": tr.RemotesTitle,
- "reflogCommits": tr.ReflogCommitsTitle,
- "tags": tr.TagsTitle,
- "commitFiles": tr.CommitFilesTitle,
- "commitMessage": tr.CommitMessageTitle,
- "commits": tr.CommitsTitle,
- "confirmation": tr.ConfirmationTitle,
- "information": tr.InformationTitle,
- "main": tr.NormalTitle,
- "patchBuilding": tr.PatchBuildingTitle,
- "mergeConflicts": tr.MergingTitle,
- "staging": tr.StagingTitle,
- "menu": tr.MenuTitle,
- "search": tr.SearchTitle,
- "secondary": tr.SecondaryTitle,
- "stash": tr.StashTitle,
- "suggestions": tr.SuggestionsCheatsheetTitle,
- "extras": tr.ExtrasTitle,
+ "global": tr.GlobalTitle,
+ "navigation": tr.NavigationTitle,
+ "branches": tr.BranchesTitle,
+ "localBranches": tr.LocalBranchesTitle,
+ "files": tr.FilesTitle,
+ "status": tr.StatusTitle,
+ "submodules": tr.SubmodulesTitle,
+ "subCommits": tr.SubCommitsTitle,
+ "remoteBranches": tr.RemoteBranchesTitle,
+ "remotes": tr.RemotesTitle,
+ "reflogCommits": tr.ReflogCommitsTitle,
+ "tags": tr.TagsTitle,
+ "commitFiles": tr.CommitFilesTitle,
+ "commitMessage": tr.CommitMessageTitle,
+ "commitDescription": tr.CommitDescriptionTitle,
+ "commits": tr.CommitsTitle,
+ "confirmation": tr.ConfirmationTitle,
+ "information": tr.InformationTitle,
+ "main": tr.NormalTitle,
+ "patchBuilding": tr.PatchBuildingTitle,
+ "mergeConflicts": tr.MergingTitle,
+ "staging": tr.StagingTitle,
+ "menu": tr.MenuTitle,
+ "search": tr.SearchTitle,
+ "secondary": tr.SecondaryTitle,
+ "stash": tr.StashTitle,
+ "suggestions": tr.SuggestionsCheatsheetTitle,
+ "extras": tr.ExtrasTitle,
  }
 
  title, ok := contextTitleMap[str]

pkg/commands/git_commands/commit.go

@@ -8,6 +8,8 @@ import (
  "github.com/jesseduffield/lazygit/pkg/commands/oscommands"
 )
 
+var ErrInvalidCommitIndex = errors.New("invalid commit index")
+
 type CommitCommands struct {
  *GitCommon
 }
@@ -18,11 +20,6 @@ func NewCommitCommands(gitCommon *GitCommon) *CommitCommands {
  }
 }
 
-// RewordLastCommit rewords the topmost commit with the given message
-func (self *CommitCommands) RewordLastCommit(message string) error {
- return self.cmd.New("git commit --allow-empty --amend --only -m " + self.cmd.Quote(message)).Run()
-}
-
 // ResetAuthor resets the author of the topmost commit
 func (self *CommitCommands) ResetAuthor() error {
  return self.cmd.New("git commit --allow-empty --only --no-edit --amend --reset-author").Run()
@@ -45,19 +42,31 @@ func (self *CommitCommands) ResetToCommit(sha string, strength string, envVars [
 }
 
 func (self *CommitCommands) CommitCmdObj(message string) oscommands.ICmdObj {
- splitMessage := strings.Split(message, "\n")
- lineArgs := ""
- for _, line := range splitMessage {
- lineArgs += fmt.Sprintf(" -m %s", self.cmd.Quote(line))
- }
+ messageArgs := self.commitMessageArgs(message)
 
  skipHookPrefix := self.UserConfig.Git.SkipHookPrefix
  noVerifyFlag := ""
  if skipHookPrefix != "" && strings.HasPrefix(message, skipHookPrefix) {
  noVerifyFlag = " --no-verify"
  }
 
- return self.cmd.New(fmt.Sprintf("git commit%s%s%s", noVerifyFlag, self.signoffFlag(), lineArgs))
+ return self.cmd.New(fmt.Sprintf("git commit%s%s%s", noVerifyFlag, self.signoffFlag(), messageArgs))
+}
+
+// RewordLastCommit rewords the topmost commit with the given message
+func (self *CommitCommands) RewordLastCommit(message string) error {
+ messageArgs := self.commitMessageArgs(message)
+ return self.cmd.New(fmt.Sprintf("git commit --allow-empty --amend --only%s", messageArgs)).Run()
+}
+
+func (self *CommitCommands) commitMessageArgs(message string) string {
+ msg, description, _ := strings.Cut(message, "\n")
+ descriptionArgs := ""
+ if description != "" {
+ descriptionArgs = fmt.Sprintf(" -m %s", self.cmd.Quote(description))
+ }
+
+ return fmt.Sprintf(" -m %s%s", self.cmd.Quote(msg), descriptionArgs)
 }
 
 // runs git commit without the -m argument meaning it will invoke the user's editor
@@ -178,3 +187,13 @@ func (self *CommitCommands) RevertMerge(sha string, parentNumber int) error {
 func (self *CommitCommands) CreateFixupCommit(sha string) error {
  return self.cmd.New(fmt.Sprintf("git commit --fixup=%s", sha)).Run()
 }
+
+// a value of 0 means the head commit, 1 is the parent commit, etc
+func (self *CommitCommands) GetCommitMessageFromHistory(value int) (string, error) {
+ hash, _ := self.cmd.New(fmt.Sprintf("git log -1 --skip=%d --pretty=%%H", value)).DontLog().RunWithOutput()
+ formattedHash := strings.TrimSpace(hash)
+ if len(formattedHash) == 0 {
+ return "", ErrInvalidCommitIndex
+ }
+ return self.GetCommitMessage(formattedHash)
+}

pkg/commands/git_commands/commit_test.go

@@ -9,12 +9,32 @@ import (
 )
 
 func TestCommitRewordCommit(t *testing.T) {
- runner := oscommands.NewFakeRunner(t).
- ExpectGitArgs([]string{"commit", "--allow-empty", "--amend", "--only", "-m", "test"}, "", nil)
- instance := buildCommitCommands(commonDeps{runner: runner})
+ type scenario struct {
+ testName string
+ runner *oscommands.FakeCmdObjRunner
+ input string
+ }
+ scenarios := []scenario{
+ {
+ "Single line reword",
+ oscommands.NewFakeRunner(t).ExpectGitArgs([]string{"commit", "--allow-empty", "--amend", "--only", "-m", "test"}, "", nil),
+ "test",
+ },
+ {
+ "Multi line reword",
+ oscommands.NewFakeRunner(t).ExpectGitArgs([]string{"commit", "--allow-empty", "--amend", "--only", "-m", "test", "-m", "line 2\nline 3"}, "", nil),
+ "test\nline 2\nline 3",
+ },
+ }
+ for _, s := range scenarios {
+ s := s
+ t.Run(s.testName, func(t *testing.T) {
+ instance := buildCommitCommands(commonDeps{runner: s.runner})
 
- assert.NoError(t, instance.RewordLastCommit("test"))
- runner.CheckForMissingCalls()
+ assert.NoError(t, instance.RewordLastCommit(s.input))
+ s.runner.CheckForMissingCalls()
+ })
+ }
 }
 
 func TestCommitResetToCommit(t *testing.T) {
@@ -274,3 +294,40 @@ Merge pull request #1750 from mark2185/fix-issue-template
  })
  }
 }
+
+func TestGetCommitMessageFromHistory(t *testing.T) {
+ type scenario struct {
+ testName string
+ runner *oscommands.FakeCmdObjRunner
+ test func(string, error)
+ }
+ scenarios := []scenario{
+ {
+ "Empty message",
+ oscommands.NewFakeRunner(t).Expect("git log -1 --skip=2 --pretty=%H", "", nil).Expect("git rev-list --format=%B --max-count=1 ", "", nil),
+ func(output string, err error) {
+ assert.Error(t, err)
+ },
+ },
+ {
+ "Default case to retrieve a commit in history",
+ oscommands.NewFakeRunner(t).Expect("git log -1 --skip=2 --pretty=%H", "sha3 \n", nil).Expect("git rev-list --format=%B --max-count=1 sha3", `commit sha3
+ use generics to DRY up context code`, nil),
+ func(output string, err error) {
+ assert.NoError(t, err)
+ assert.Equal(t, "use generics to DRY up context code", output)
+ },
+ },
+ }
+
+ for _, s := range scenarios {
+ s := s
+ t.Run(s.testName, func(t *testing.T) {
+ instance := buildCommitCommands(commonDeps{runner: s.runner})
+
+ output, err := instance.GetCommitMessageFromHistory(2)
+
+ s.test(output, err)
+ })
+ }
+}

pkg/config/user_config.go

@@ -165,6 +165,7 @@ type KeybindingUniversalConfig struct {
  Select string `yaml:"select"`
  GoInto string `yaml:"goInto"`
  Confirm string `yaml:"confirm"`
+ ConfirmInEditor string `yaml:"confirmInEditor"`
  Remove string `yaml:"remove"`
  New string `yaml:"new"`
  Edit string `yaml:"edit"`
@@ -193,7 +194,6 @@ type KeybindingUniversalConfig struct {
  CopyToClipboard string `yaml:"copyToClipboard"`
  OpenRecentRepos string `yaml:"openRecentRepos"`
  SubmitEditorText string `yaml:"submitEditorText"`
- AppendNewline string `yaml:"appendNewline"`
  ExtrasMenu string `yaml:"extrasMenu"`
  ToggleWhitespaceInDiffView string `yaml:"toggleWhitespaceInDiffView"`
  IncreaseContextInDiffView string `yaml:"increaseContextInDiffView"`
@@ -492,6 +492,7 @@ func GetDefaultConfig() *UserConfig {
  Select: "<space>",
  GoInto: "<enter>",
  Confirm: "<enter>",
+ ConfirmInEditor: "<a-enter>",
  Remove: "d",
  New: "n",
  Edit: "e",
@@ -520,7 +521,6 @@ func GetDefaultConfig() *UserConfig {
  DiffingMenuAlt: "<c-e>",
  CopyToClipboard: "<c-o>",
  SubmitEditorText: "<enter>",
- AppendNewline: "<a-enter>",
  ExtrasMenu: "@",
  ToggleWhitespaceInDiffView: "<c-w>",
  IncreaseContextInDiffView: "}",

pkg/gui/commit_message_panel.go

@@ -15,7 +15,6 @@ func (gui *Gui) handleCommitMessageFocused() error {
  map[string]string{
  "keyBindClose": keybindings.Label(gui.c.UserConfig.Keybinding.Universal.Return),
  "keyBindConfirm": keybindings.Label(gui.c.UserConfig.Keybinding.Universal.Confirm),
- "keyBindNewLine": keybindings.Label(gui.c.UserConfig.Keybinding.Universal.AppendNewline),
  },
  )
 

pkg/gui/confirmation_panel.go

@@ -78,17 +78,11 @@ func (gui *Gui) getMessageHeight(wrap bool, message string, width int) int {
  return lineCount
 }
 
-func (gui *Gui) getConfirmationPanelDimensions(wrap bool, prompt string) (int, int, int, int) {
- panelWidth := gui.getConfirmationPanelWidth()
- panelHeight := gui.getMessageHeight(wrap, prompt, panelWidth)
- return gui.getConfirmationPanelDimensionsAux(panelWidth, panelHeight)
+func (gui *Gui) getPopupPanelDimensionsForContentHeight(panelWidth, contentHeight int) (int, int, int, int) {
+ return gui.getPopupPanelDimensionsAux(panelWidth, contentHeight)
 }
 
-func (gui *Gui) getConfirmationPanelDimensionsForContentHeight(panelWidth, contentHeight int) (int, int, int, int) {
- return gui.getConfirmationPanelDimensionsAux(panelWidth, contentHeight)
-}
-
-func (gui *Gui) getConfirmationPanelDimensionsAux(panelWidth int, panelHeight int) (int, int, int, int) {
+func (gui *Gui) getPopupPanelDimensionsAux(panelWidth int, panelHeight int) (int, int, int, int) {
  width, height := gui.g.Size()
  if panelHeight > height*3/4 {
  panelHeight = height * 3 / 4
@@ -186,7 +180,7 @@ func (gui *Gui) createPopupPanel(ctx context.Context, opts types.CreatePopupPane
  }
  confirmationView := gui.Views.Confirmation
  confirmationView.Editable = opts.Editable
- confirmationView.Editor = gocui.EditorFunc(gui.defaultEditor)
+ confirmationView.Editor = gocui.EditorFunc(gui.promptEditor)
 
  if opts.Editable {
  textArea := confirmationView.TextArea

pkg/gui/context.go

@@ -9,6 +9,7 @@ import (
  "github.com/jesseduffield/gocui"
  "github.com/jesseduffield/lazygit/pkg/gui/context"
  "github.com/jesseduffield/lazygit/pkg/gui/types"
+ "github.com/samber/lo"
 )
 
 // This file is for the management of contexts. There is a context stack such that
@@ -146,6 +147,36 @@ func (gui *Gui) popContext() error {
  return gui.activateContext(newContext, types.OnFocusOpts{})
 }
 
+func (gui *Gui) removeContexts(contextsToRemove []types.Context) error {
+ gui.State.ContextManager.Lock()
+
+ if len(gui.State.ContextManager.ContextStack) == 1 {
+ gui.State.ContextManager.Unlock()
+ return nil
+ }
+
+ rest := lo.Filter(gui.State.ContextManager.ContextStack, func(context types.Context, _ int) bool {
+ for _, contextToRemove := range contextsToRemove {
+ if context.GetKey() == contextToRemove.GetKey() {
+ return false
+ }
+ }
+ return true
+ })
+ gui.State.ContextManager.ContextStack = rest
+ contextToActivate := rest[len(rest)-1]
+ gui.State.ContextManager.Unlock()
+
+ for _, context := range contextsToRemove {
+ if err := gui.deactivateContext(context, types.OnFocusLostOpts{NewContextKey: contextToActivate.GetKey()}); err != nil {
+ return err
+ }
+ }
+
+ // activate the item at the top of the stack
+ return gui.activateContext(contextToActivate, types.OnFocusOpts{})
+}
+
 func (gui *Gui) deactivateContext(c types.Context, opts types.OnFocusLostOpts) error {
  view, _ := gui.g.View(c.GetViewName())