Merge pull request #2694 from stefanhaller/conflict-handling-menu

Show menu instead of prompt when there are conflicts in a rebase or merge

pkg/gui/controllers/helpers/merge_and_rebase_helper.go

@@ -138,15 +138,26 @@ func (self *MergeAndRebaseHelper) CheckMergeOrRebase(result error) error {
 		// assume in this case that we're already done
 		return nil
 	} else if isMergeConflictErr(result.Error()) {
-		return self.c.Confirm(types.ConfirmOpts{
+		mode := self.workingTreeStateNoun()
-			Title:  self.c.Tr.FoundConflictsTitle,
+		return self.c.Menu(types.CreateMenuOptions{
-			Prompt: self.c.Tr.FoundConflicts,
+			Title: self.c.Tr.FoundConflictsTitle,
-			HandleConfirm: func() error {
+			Items: []*types.MenuItem{
-				return self.c.PushContext(self.c.Contexts().Files)
+				{
-			},
+					Label: self.c.Tr.ViewConflictsMenuItem,
-			HandleClose: func() error {
+					OnPress: func() error {
-				return self.genericMergeCommand(REBASE_OPTION_ABORT)
+						return self.c.PushContext(self.c.Contexts().Files)
+					},
+					Key: 'v',
+				},
+				{
+					Label: fmt.Sprintf(self.c.Tr.AbortMenuItem, mode),
+					OnPress: func() error {
+						return self.genericMergeCommand(REBASE_OPTION_ABORT)
+					},
+					Key: 'a',
+				},
 			},
+			HideCancel: true,
 		})
 	} else {
 		return self.c.ErrorMsg(result.Error())

pkg/i18n/chinese.go

@@ -175,7 +175,6 @@ func chineseTranslationSet() TranslationSet {
 		ReturnToFilesPanel:                  `返回文件面板`,
 		FastForward:                         `从上游快进此分支`,
 		Fetching:                            "抓取并快进 {{.from}} -> {{.to}} ...",
-		FoundConflicts:                      "冲突！需要中止操作请按‘esc’, 否则按‘enter’",
 		FoundConflictsTitle:                 "自动合并失败",
 		ViewMergeRebaseOptions:              "查看 合并/变基 选项",
 		NotMergingOrRebasing:                "您目前既不进行变基也不进行合并",

pkg/i18n/dutch.go

@@ -140,8 +140,7 @@ func dutchTranslationSet() TranslationSet {
 		ReturnToFilesPanel:                  `Ga terug naar het bestanden paneel`,
 		FastForward:                         `Fast-forward deze branch vanaf zijn upstream`,
 		Fetching:                            "Fetching en fast-forwarding {{.from}} -> {{.to}} ...",
-		FoundConflicts:                      "Conflicten!, Om af te breken druk 'esc', anders druk op 'enter'",
+		FoundConflictsTitle:                 "Conflicten!",
-		FoundConflictsTitle:                 "Auto-merge mislukt",
 		ViewMergeRebaseOptions:              "Bekijk merge/rebase opties",
 		NotMergingOrRebasing:                "Je bent momenteel niet aan het rebasen of mergen",
 		RecentRepos:                         "Recente repositories",

pkg/i18n/english.go

@@ -181,8 +181,9 @@ type TranslationSet struct {
 	ReturnToFilesPanel                  string
 	FastForward                         string
 	Fetching                            string
-	FoundConflicts                      string
 	FoundConflictsTitle                 string
+	ViewConflictsMenuItem               string
+	AbortMenuItem                       string
 	PickHunk                            string
 	PickAllHunks                        string
 	ViewMergeRebaseOptions              string
@@ -867,8 +868,9 @@ func EnglishTranslationSet() TranslationSet {
 		ReturnToFilesPanel:                  `Return to files panel`,
 		FastForward:                         `Fast-forward this branch from its upstream`,
 		Fetching:                            "Fetching and fast-forwarding {{.from}} -> {{.to}} ...",
-		FoundConflicts:                      "Conflicts! To abort press 'esc', otherwise press 'enter'",
+		FoundConflictsTitle:                 "Conflicts!",
-		FoundConflictsTitle:                 "Auto-merge failed",
+		ViewConflictsMenuItem:               "View conflicts",
+		AbortMenuItem:                       "Abort the %s",
 		ViewMergeRebaseOptions:              "View merge/rebase options",
 		NotMergingOrRebasing:                "You are currently neither rebasing nor merging",
 		AlreadyRebasing:                     "Can't perform this action during a rebase",

pkg/i18n/korean.go

@@ -176,7 +176,6 @@ func koreanTranslationSet() TranslationSet {
 		ReturnToFilesPanel:                  `파일 목록으로 돌아가기`,
 		FastForward:                         `Fast-forward this branch from its upstream`,
 		Fetching:                            "Fetching and fast-forwarding {{.from}} -> {{.to}} ...",
-		FoundConflicts:                      "Conflicts! To abort press 'esc', otherwise press 'enter'",
 		FoundConflictsTitle:                 "Auto-merge failed",
 		ViewMergeRebaseOptions:              "View merge/rebase options",
 		NotMergingOrRebasing:                "You are currently neither rebasing nor merging",

pkg/i18n/polish.go

@@ -125,8 +125,7 @@ func polishTranslationSet() TranslationSet {
 		MoveUpCommit:                        "Przenieś commit 1 w górę",
 		EditCommit:                          "Edytuj commit",
 		AmendToCommit:                       "Popraw commit zmianami z poczekalni",
-		FoundConflicts:                      "Konflikty! Wciśnij 'esc' żeby przerwać, w przeciwnym razie wciśnij 'enter'",
+		FoundConflictsTitle:                 "Konflikty!",
-		FoundConflictsTitle:                 "Automatyczne scalenie nie powiodło się",
 		ViewMergeRebaseOptions:              "Widok scalenia/opcje zmiany bazy",
 		NotMergingOrRebasing:                "W tej chwili nie scalasz ani nie zmieniasz bazy",
 		RecentRepos:                         "Ostatnie repozytoria",

pkg/i18n/traditional_chinese.go

@@ -240,7 +240,6 @@ func traditionalChineseTranslationSet() TranslationSet {
 		ReturnToFilesPanel:                  `返回檔案面板`,
 		FastForward:                         `從上游快進此分支`,
 		Fetching:                            "{{.from}} -> {{.to}} 的擷取和快進中...",
-		FoundConflicts:                      "衝突！按 'esc' 退出，否則按 'enter'",
 		FoundConflictsTitle:                 "自動合併失敗",
 		ViewMergeRebaseOptions:              "查看合併/變基選項",
 		NotMergingOrRebasing:                "你當前既不在變基也不在合併中",

pkg/integration/components/common.go

@@ -19,9 +19,9 @@ func (self *Common) ContinueRebase() {
 }
 
 func (self *Common) AcknowledgeConflicts() {
-	self.t.ExpectPopup().Confirmation().
+	self.t.ExpectPopup().Menu().
-		Title(Equals("Auto-merge failed")).
+		Title(Equals("Conflicts!")).
-		Content(Contains("Conflicts!")).
+		Select(Contains("View conflicts")).
 		Confirm()
 }
 

pkg/integration/tests/branch/rebase_abort_on_conflict.go

@@ -0,0 +1,49 @@
+package branch
+
+import (
+	"github.com/jesseduffield/lazygit/pkg/config"
+	. "github.com/jesseduffield/lazygit/pkg/integration/components"
+	"github.com/jesseduffield/lazygit/pkg/integration/tests/shared"
+)
+
+var RebaseAbortOnConflict = NewIntegrationTest(NewIntegrationTestArgs{
+	Description:  "Rebase onto another branch, abort when there are conflicts.",
+	ExtraCmdArgs: []string{},
+	Skip:         false,
+	SetupConfig:  func(config *config.AppConfig) {},
+	SetupRepo: func(shell *Shell) {
+		shared.MergeConflictsSetup(shell)
+	},
+	Run: func(t *TestDriver, keys config.KeybindingConfig) {
+		t.Views().Commits().TopLines(
+			Contains("first change"),
+			Contains("original"),
+		)
+
+		t.Views().Branches().
+			Focus().
+			Lines(
+				Contains("first-change-branch"),
+				Contains("second-change-branch"),
+				Contains("original-branch"),
+			).
+			SelectNextItem().
+			Press(keys.Branches.RebaseBranch)
+
+		t.ExpectPopup().Menu().
+			Title(Equals("Rebase 'first-change-branch' onto 'second-change-branch'")).
+			Select(Contains("Simple rebase")).
+			Confirm()
+
+		t.ExpectPopup().Menu().
+			Title(Equals("Conflicts!")).
+			Select(Contains("Abort the rebase")).
+			Confirm()
+
+		t.Views().Branches().
+			IsFocused()
+
+		t.Views().Files().
+			IsEmpty()
+	},
+})

pkg/integration/tests/branch/rebase_cancel_on_conflict.go

@@ -0,0 +1,51 @@
+package branch
+
+import (
+	"github.com/jesseduffield/lazygit/pkg/config"
+	. "github.com/jesseduffield/lazygit/pkg/integration/components"
+	"github.com/jesseduffield/lazygit/pkg/integration/tests/shared"
+)
+
+var RebaseCancelOnConflict = NewIntegrationTest(NewIntegrationTestArgs{
+	Description:  "Rebase onto another branch, cancel when there are conflicts.",
+	ExtraCmdArgs: []string{},
+	Skip:         false,
+	SetupConfig:  func(config *config.AppConfig) {},
+	SetupRepo: func(shell *Shell) {
+		shared.MergeConflictsSetup(shell)
+	},
+	Run: func(t *TestDriver, keys config.KeybindingConfig) {
+		t.Views().Commits().TopLines(
+			Contains("first change"),
+			Contains("original"),
+		)
+
+		t.Views().Branches().
+			Focus().
+			Lines(
+				Contains("first-change-branch"),
+				Contains("second-change-branch"),
+				Contains("original-branch"),
+			).
+			SelectNextItem().
+			Press(keys.Branches.RebaseBranch)
+
+		t.ExpectPopup().Menu().
+			Title(Equals("Rebase 'first-change-branch' onto 'second-change-branch'")).
+			Select(Contains("Simple rebase")).
+			Confirm()
+
+		t.ExpectPopup().Menu().
+			Title(Equals("Conflicts!")).
+			Select(Contains("Abort the rebase")).
+			Cancel()
+
+		t.Views().Branches().
+			IsFocused()
+
+		t.Views().Files().
+			Lines(
+				Contains("UU file"),
+			)
+	},
+})

pkg/integration/tests/test_list.go

@@ -36,7 +36,9 @@ var tests = []*components.IntegrationTest{
 	branch.DetachedHead,
 	branch.OpenWithCliArg,
 	branch.Rebase,
+	branch.RebaseAbortOnConflict,
 	branch.RebaseAndDrop,
+	branch.RebaseCancelOnConflict,
 	branch.RebaseDoesNotAutosquash,
 	branch.Reset,
 	branch.ResetUpstream,