diff --git a/docs/Config.md b/docs/Config.md index 9b48a32d8..079ff5ce9 100644 --- a/docs/Config.md +++ b/docs/Config.md @@ -517,6 +517,7 @@ keybinding: universal: quit: q quit-alt1: + suspendApp: return: quitWithoutChangingDirectory: Q togglePanel: @@ -583,7 +584,7 @@ keybinding: nextScreenMode: + prevScreenMode: _ undo: z - redo: + redo: Z filteringMenu: diffingMenu: W diffingMenu-alt: diff --git a/docs/keybindings/Keybindings_en.md b/docs/keybindings/Keybindings_en.md index b73a505c3..2154880e6 100644 --- a/docs/keybindings/Keybindings_en.md +++ b/docs/keybindings/Keybindings_en.md @@ -30,9 +30,10 @@ _Legend: `` means ctrl+b, `` means alt+b, `B` means shift+b_ | `` W `` | View diffing options | View options relating to diffing two refs e.g. diffing against selected ref, entering ref to diff against, and reversing the diff direction. | | `` `` | View diffing options | View options relating to diffing two refs e.g. diffing against selected ref, entering ref to diff against, and reversing the diff direction. | | `` q `` | Quit | | +| `` `` | Suspend the application | | | `` `` | Toggle whitespace | Toggle whether or not whitespace changes are shown in the diff view.

The default can be changed in the config file with the key 'git.ignoreWhitespaceInDiffView'. | | `` z `` | Undo | The reflog will be used to determine what git command to run to undo the last git command. This does not include changes to the working tree; only commits are taken into consideration. | -| `` `` | Redo | The reflog will be used to determine what git command to run to redo the last git command. This does not include changes to the working tree; only commits are taken into consideration. | +| `` Z `` | Redo | The reflog will be used to determine what git command to run to redo the last git command. This does not include changes to the working tree; only commits are taken into consideration. | ## List panel navigation diff --git a/docs/keybindings/Keybindings_ja.md b/docs/keybindings/Keybindings_ja.md index 9d8460d96..490b0f36d 100644 --- a/docs/keybindings/Keybindings_ja.md +++ b/docs/keybindings/Keybindings_ja.md @@ -30,9 +30,10 @@ _凡例：`＜c-b＞` はctrl+b、`＜a-b＞` はalt+b、`B` はshift+bを意味 | `` W `` | 差分オプションを表示 | ２つのrefの差分に関連するオプションを表示します（例：選択したrefとの差分表示、差分を取るrefの入力、差分方向の反転など）。 | | `` `` | 差分オプションを表示 | ２つのrefの差分に関連するオプションを表示します（例：選択したrefとの差分表示、差分を取るrefの入力、差分方向の反転など）。 | | `` q `` | 終了 | | +| `` `` | Suspend the application | | | `` `` | 空白表示の切り替え | Toggle whether or not whitespace changes are shown in the diff view.

The default can be changed in the config file with the key 'git.ignoreWhitespaceInDiffView'. | | `` z `` | 元に戻す | 最後のgitコマンドを元に戻すために実行するgitコマンドを決定するためにreflogが使用されます。これにはワーキングツリーへの変更は含まれません。コミットのみが考慮されます。 | -| `` `` | やり直す | 最後のgitコマンドをやり直すために実行するgitコマンドを決定するためにreflogが使用されます。これにはワーキングツリーへの変更は含まれません。コミットのみが考慮されます。 | +| `` Z `` | やり直す | 最後のgitコマンドをやり直すために実行するgitコマンドを決定するためにreflogが使用されます。これにはワーキングツリーへの変更は含まれません。コミットのみが考慮されます。 | ## リストパネルのナビゲーション diff --git a/docs/keybindings/Keybindings_ko.md b/docs/keybindings/Keybindings_ko.md index 1dd33ba54..ac6d5fb03 100644 --- a/docs/keybindings/Keybindings_ko.md +++ b/docs/keybindings/Keybindings_ko.md @@ -30,9 +30,10 @@ _Legend: `` means ctrl+b, `` means alt+b, `B` means shift+b_ | `` W `` | Diff 메뉴 열기 | View options relating to diffing two refs e.g. diffing against selected ref, entering ref to diff against, and reversing the diff direction. | | `` `` | Diff 메뉴 열기 | View options relating to diffing two refs e.g. diffing against selected ref, entering ref to diff against, and reversing the diff direction. | | `` q `` | 종료 | | +| `` `` | Suspend the application | | | `` `` | 공백문자를 Diff 뷰에서 표시 여부 전환 | Toggle whether or not whitespace changes are shown in the diff view.

The default can be changed in the config file with the key 'git.ignoreWhitespaceInDiffView'. | | `` z `` | 되돌리기 (reflog) (실험적) | The reflog will be used to determine what git command to run to undo the last git command. This does not include changes to the working tree; only commits are taken into consideration. | -| `` `` | 다시 실행 (reflog) (실험적) | The reflog will be used to determine what git command to run to redo the last git command. This does not include changes to the working tree; only commits are taken into consideration. | +| `` Z `` | 다시 실행 (reflog) (실험적) | The reflog will be used to determine what git command to run to redo the last git command. This does not include changes to the working tree; only commits are taken into consideration. | ## List panel navigation diff --git a/docs/keybindings/Keybindings_nl.md b/docs/keybindings/Keybindings_nl.md index 658e69772..5ef08427a 100644 --- a/docs/keybindings/Keybindings_nl.md +++ b/docs/keybindings/Keybindings_nl.md @@ -30,9 +30,10 @@ _Legend: `` means ctrl+b, `` means alt+b, `B` means shift+b_ | `` W `` | Open diff menu | View options relating to diffing two refs e.g. diffing against selected ref, entering ref to diff against, and reversing the diff direction. | | `` `` | Open diff menu | View options relating to diffing two refs e.g. diffing against selected ref, entering ref to diff against, and reversing the diff direction. | | `` q `` | Quit | | +| `` `` | Suspend the application | | | `` `` | Toggle whitespace | Toggle whether or not whitespace changes are shown in the diff view.

The default can be changed in the config file with the key 'git.ignoreWhitespaceInDiffView'. | | `` z `` | Ongedaan maken (via reflog) (experimenteel) | The reflog will be used to determine what git command to run to undo the last git command. This does not include changes to the working tree; only commits are taken into consideration. | -| `` `` | Redo (via reflog) (experimenteel) | The reflog will be used to determine what git command to run to redo the last git command. This does not include changes to the working tree; only commits are taken into consideration. | +| `` Z `` | Redo (via reflog) (experimenteel) | The reflog will be used to determine what git command to run to redo the last git command. This does not include changes to the working tree; only commits are taken into consideration. | ## Lijstpaneel navigatie diff --git a/docs/keybindings/Keybindings_pl.md b/docs/keybindings/Keybindings_pl.md index f599d6cff..52c6d666c 100644 --- a/docs/keybindings/Keybindings_pl.md +++ b/docs/keybindings/Keybindings_pl.md @@ -30,9 +30,10 @@ _Legenda: `` oznacza ctrl+b, `` oznacza alt+b, `B` oznacza shift+b_ | `` W `` | Pokaż opcje różnicowania | Pokaż opcje dotyczące różnicowania dwóch refów, np. różnicowanie względem wybranego refa, wprowadzanie refa do różnicowania i odwracanie kierunku różnic. | | `` `` | Pokaż opcje różnicowania | Pokaż opcje dotyczące różnicowania dwóch refów, np. różnicowanie względem wybranego refa, wprowadzanie refa do różnicowania i odwracanie kierunku różnic. | | `` q `` | Wyjdź | | +| `` `` | Suspend the application | | | `` `` | Przełącz białe znaki | Toggle whether or not whitespace changes are shown in the diff view.

The default can be changed in the config file with the key 'git.ignoreWhitespaceInDiffView'. | | `` z `` | Cofnij | Dziennik reflog zostanie użyty do określenia, jakie polecenie git należy uruchomić, aby cofnąć ostatnie polecenie git. Nie obejmuje to zmian w drzewie roboczym; brane są pod uwagę tylko commity. | -| `` `` | Ponów | Dziennik reflog zostanie użyty do określenia, jakie polecenie git należy uruchomić, aby ponowić ostatnie polecenie git. Nie obejmuje to zmian w drzewie roboczym; brane są pod uwagę tylko commity. | +| `` Z `` | Ponów | Dziennik reflog zostanie użyty do określenia, jakie polecenie git należy uruchomić, aby ponowić ostatnie polecenie git. Nie obejmuje to zmian w drzewie roboczym; brane są pod uwagę tylko commity. | ## Nawigacja panelu listy diff --git a/docs/keybindings/Keybindings_pt.md b/docs/keybindings/Keybindings_pt.md index 66c92a90e..3c323f489 100644 --- a/docs/keybindings/Keybindings_pt.md +++ b/docs/keybindings/Keybindings_pt.md @@ -30,9 +30,10 @@ _Legend: `` means ctrl+b, `` means alt+b, `B` means shift+b_ | `` W `` | View diffing options | View options relating to diffing two refs e.g. diffing against selected ref, entering ref to diff against, and reversing the diff direction. | | `` `` | View diffing options | View options relating to diffing two refs e.g. diffing against selected ref, entering ref to diff against, and reversing the diff direction. | | `` q `` | Sair | | +| `` `` | Suspend the application | | | `` `` | Toggle whitespace | Toggle whether or not whitespace changes are shown in the diff view.

The default can be changed in the config file with the key 'git.ignoreWhitespaceInDiffView'. | | `` z `` | Desfazer | O reflog será usado para determinar qual comando git para executar para desfazer o último comando git. Isto não inclui mudanças na árvore de trabalho; apenas compromissos são tidos em consideração. | -| `` `` | Refazer | O reflog será usado para determinar qual comando git para executar para refazer o último comando git. Isto não inclui mudanças na árvore de trabalho; apenas compromissos são tidos em consideração. | +| `` Z `` | Refazer | O reflog será usado para determinar qual comando git para executar para refazer o último comando git. Isto não inclui mudanças na árvore de trabalho; apenas compromissos são tidos em consideração. | ## List panel navigation diff --git a/docs/keybindings/Keybindings_ru.md b/docs/keybindings/Keybindings_ru.md index efd8a34bf..05749d5d9 100644 --- a/docs/keybindings/Keybindings_ru.md +++ b/docs/keybindings/Keybindings_ru.md @@ -30,9 +30,10 @@ _Связки клавиш_ | `` W `` | Открыть меню сравнении | View options relating to diffing two refs e.g. diffing against selected ref, entering ref to diff against, and reversing the diff direction. | | `` `` | Открыть меню сравнении | View options relating to diffing two refs e.g. diffing against selected ref, entering ref to diff against, and reversing the diff direction. | | `` q `` | Выйти | | +| `` `` | Suspend the application | | | `` `` | Переключить отображение изменении пробелов в просмотрщике сравнении | Toggle whether or not whitespace changes are shown in the diff view.

The default can be changed in the config file with the key 'git.ignoreWhitespaceInDiffView'. | | `` z `` | Отменить (через reflog) (экспериментальный) | Журнал ссылок (reflog) будет использоваться для определения того, какую команду git запустить, чтобы отменить последнюю команду git. Сюда не входят изменения в рабочем дереве; учитываются только коммиты. | -| `` `` | Повторить (через reflog) (экспериментальный) | Журнал ссылок (reflog) будет использоваться для определения того, какую команду git нужно запустить, чтобы повторить последнюю команду git. Сюда не входят изменения в рабочем дереве; учитываются только коммиты. | +| `` Z `` | Повторить (через reflog) (экспериментальный) | Журнал ссылок (reflog) будет использоваться для определения того, какую команду git нужно запустить, чтобы повторить последнюю команду git. Сюда не входят изменения в рабочем дереве; учитываются только коммиты. | ## Навигация по панели списка diff --git a/docs/keybindings/Keybindings_zh-CN.md b/docs/keybindings/Keybindings_zh-CN.md index 26f7cc4a2..bfc09bfa4 100644 --- a/docs/keybindings/Keybindings_zh-CN.md +++ b/docs/keybindings/Keybindings_zh-CN.md @@ -30,9 +30,10 @@ _图例：`` 意味着ctrl+b, `意味着Alt+b, `B` 意味着shift+b_ | `` W `` | 打开 diff 菜单 | 查看与比较两个引用相关的选项，例如与选定的 ref 进行比较，输入要比较的 ref，然后反转比较方向。 | | `` `` | 打开 diff 菜单 | 查看与比较两个引用相关的选项，例如与选定的 ref 进行比较，输入要比较的 ref，然后反转比较方向。 | | `` q `` | 退出 | | +| `` `` | Suspend the application | | | `` `` | 切换是否在差异视图中显示空白字符差异 | Toggle whether or not whitespace changes are shown in the diff view.

The default can be changed in the config file with the key 'git.ignoreWhitespaceInDiffView'. | | `` z `` | 撤销 | Reflog将用于确定运行哪个git命令来撤消最后一个git命令。这并不包括对工作树的更改，只考虑提交。 | -| `` `` | 重做 | Reflog将用于确定运行哪个git命令来重做上一个git命令。这并不包括对工作树的更改，只考虑提交。 | +| `` Z `` | 重做 | Reflog将用于确定运行哪个git命令来重做上一个git命令。这并不包括对工作树的更改，只考虑提交。 | ## 列表面板导航 diff --git a/docs/keybindings/Keybindings_zh-TW.md b/docs/keybindings/Keybindings_zh-TW.md index a716e6cfd..56ae29243 100644 --- a/docs/keybindings/Keybindings_zh-TW.md +++ b/docs/keybindings/Keybindings_zh-TW.md @@ -30,9 +30,10 @@ _說明：`` 表示 Ctrl＋B、`` 表示 Alt＋B，`B`表示 Shift＋B | `` W `` | 開啟差異比較選單 | View options relating to diffing two refs e.g. diffing against selected ref, entering ref to diff against, and reversing the diff direction. | | `` `` | 開啟差異比較選單 | View options relating to diffing two refs e.g. diffing against selected ref, entering ref to diff against, and reversing the diff direction. | | `` q `` | 結束 | | +| `` `` | Suspend the application | | | `` `` | 切換是否在差異檢視中顯示空格變更 | Toggle whether or not whitespace changes are shown in the diff view.

The default can be changed in the config file with the key 'git.ignoreWhitespaceInDiffView'. | | `` z `` | 復原 | 將使用 reflog 確任 git 指令以復原。這不包括工作區更改；只考慮提交。 | -| `` `` | 取消復原 | 將使用 reflog 確任 git 指令以重作。這不包括工作區更改；只考慮提交。 | +| `` Z `` | 取消復原 | 將使用 reflog 確任 git 指令以重作。這不包括工作區更改；只考慮提交。 | ## 移動 diff --git a/go.mod b/go.mod index 801681c92..01ac2c369 100644 --- a/go.mod +++ b/go.mod @@ -37,6 +37,7 @@ require ( github.com/xo/terminfo v0.0.0-20210125001918-ca9a967f8778 golang.org/x/exp v0.0.0-20240719175910-8a7402abbf56 golang.org/x/sync v0.16.0 + golang.org/x/sys v0.34.0 gopkg.in/ozeidan/fuzzy-patricia.v3 v3.0.0 gopkg.in/yaml.v3 v3.0.1 ) @@ -77,7 +78,6 @@ require ( github.com/xanzy/ssh-agent v0.3.3 // indirect golang.org/x/crypto v0.37.0 // indirect golang.org/x/net v0.39.0 // indirect - golang.org/x/sys v0.34.0 // indirect golang.org/x/term v0.33.0 // indirect golang.org/x/text v0.27.0 // indirect gopkg.in/fsnotify.v1 v1.4.7 // indirect diff --git a/pkg/commands/oscommands/cmd_obj_runner_default.go b/pkg/commands/oscommands/cmd_obj_runner_default.go index 891b4c4ee..72cbc26c6 100644 --- a/pkg/commands/oscommands/cmd_obj_runner_default.go +++ b/pkg/commands/oscommands/cmd_obj_runner_default.go @@ -1,5 +1,4 @@ //go:build !windows -// +build !windows package oscommands diff --git a/pkg/commands/oscommands/cmd_obj_runner_win.go b/pkg/commands/oscommands/cmd_obj_runner_windows.go similarity index 86% rename from pkg/commands/oscommands/cmd_obj_runner_win.go rename to pkg/commands/oscommands/cmd_obj_runner_windows.go index 2189da2e6..f92e36c69 100644 --- a/pkg/commands/oscommands/cmd_obj_runner_win.go +++ b/pkg/commands/oscommands/cmd_obj_runner_windows.go @@ -1,6 +1,3 @@ -//go:build windows -// +build windows - package oscommands import ( diff --git a/pkg/commands/oscommands/os_default_platform.go b/pkg/commands/oscommands/os_default_platform.go index 3d7eae481..06684434e 100644 --- a/pkg/commands/oscommands/os_default_platform.go +++ b/pkg/commands/oscommands/os_default_platform.go @@ -1,5 +1,4 @@ //go:build !windows -// +build !windows package oscommands diff --git a/pkg/commands/oscommands/os_default_test.go b/pkg/commands/oscommands/os_default_test.go index 94da4b078..ff8c59cd2 100644 --- a/pkg/commands/oscommands/os_default_test.go +++ b/pkg/commands/oscommands/os_default_test.go @@ -1,5 +1,4 @@ //go:build !windows -// +build !windows package oscommands diff --git a/pkg/commands/oscommands/os_windows_test.go b/pkg/commands/oscommands/os_windows_test.go index fcf6a9577..60ba495bf 100644 --- a/pkg/commands/oscommands/os_windows_test.go +++ b/pkg/commands/oscommands/os_windows_test.go @@ -1,6 +1,3 @@ -//go:build windows -// +build windows - package oscommands import ( diff --git a/pkg/config/app_config_test.go b/pkg/config/app_config_test.go index c8c7d1d11..faba2e4e8 100644 --- a/pkg/config/app_config_test.go +++ b/pkg/config/app_config_test.go @@ -865,7 +865,7 @@ keybinding: nextScreenMode: + prevScreenMode: _ undo: z - redo: + redo: Z filteringMenu: diffingMenu: W diffingMenu-alt: diff --git a/pkg/config/config_default_platform.go b/pkg/config/config_default_platform.go index d05201454..9375e1851 100644 --- a/pkg/config/config_default_platform.go +++ b/pkg/config/config_default_platform.go @@ -1,5 +1,4 @@ //go:build !windows && !linux -// +build !windows,!linux package config diff --git a/pkg/config/user_config.go b/pkg/config/user_config.go index fc702d6da..0fa706bb2 100644 --- a/pkg/config/user_config.go +++ b/pkg/config/user_config.go @@ -390,6 +390,7 @@ type KeybindingConfig struct { type KeybindingUniversalConfig struct { Quit string `yaml:"quit"` QuitAlt1 string `yaml:"quit-alt1"` + SuspendApp string `yaml:"suspendApp"` Return string `yaml:"return"` QuitWithoutChangingDirectory string `yaml:"quitWithoutChangingDirectory"` TogglePanel string `yaml:"togglePanel"` @@ -854,6 +855,7 @@ func GetDefaultConfig() *UserConfig { Universal: KeybindingUniversalConfig{ Quit: "q", QuitAlt1: "", + SuspendApp: "", Return: "", QuitWithoutChangingDirectory: "Q", TogglePanel: "", @@ -912,7 +914,7 @@ func GetDefaultConfig() *UserConfig { NextScreenMode: "+", PrevScreenMode: "_", Undo: "z", - Redo: "", + Redo: "Z", FilteringMenu: "", DiffingMenu: "W", DiffingMenuAlt: "", diff --git a/pkg/gui/controllers.go b/pkg/gui/controllers.go index f83a3f8e9..20b52b2cb 100644 --- a/pkg/gui/controllers.go +++ b/pkg/gui/controllers.go @@ -109,6 +109,7 @@ func (gui *Gui) resetHelpersAndControllers() { AmendHelper: helpers.NewAmendHelper(helperCommon, gpgHelper), FixupHelper: helpers.NewFixupHelper(helperCommon), Commits: commitsHelper, + SuspendResume: helpers.NewSuspendResumeHelper(helperCommon), Snake: helpers.NewSnakeHelper(helperCommon), Diff: diffHelper, Repos: reposHelper, diff --git a/pkg/gui/controllers/global_controller.go b/pkg/gui/controllers/global_controller.go index fcf281cf4..734e65516 100644 --- a/pkg/gui/controllers/global_controller.go +++ b/pkg/gui/controllers/global_controller.go @@ -123,6 +123,20 @@ func (self *GlobalController) GetKeybindings(opts types.KeybindingsOpts) []*type Modifier: gocui.ModNone, Handler: self.quitWithoutChangingDirectory, }, + { + Key: opts.GetKey(opts.Config.Universal.SuspendApp), + Modifier: gocui.ModNone, + Handler: self.c.Helpers().SuspendResume.SuspendApp, + Description: self.c.Tr.SuspendApp, + GetDisabledReason: func() *types.DisabledReason { + if !self.c.Helpers().SuspendResume.CanSuspendApp() { + return &types.DisabledReason{ + Text: self.c.Tr.CannotSuspendApp, + } + } + return nil + }, + }, { Key: opts.GetKey(opts.Config.Universal.ToggleWhitespaceInDiffView), Handler: self.toggleWhitespace, diff --git a/pkg/gui/controllers/helpers/helpers.go b/pkg/gui/controllers/helpers/helpers.go index 1f1050dc9..4c9c79f3d 100644 --- a/pkg/gui/controllers/helpers/helpers.go +++ b/pkg/gui/controllers/helpers/helpers.go @@ -35,6 +35,7 @@ type Helpers struct { AmendHelper *AmendHelper FixupHelper *FixupHelper Commits *CommitsHelper + SuspendResume *SuspendResumeHelper Snake *SnakeHelper // lives in context package because our contexts need it to render to main Diff *DiffHelper diff --git a/pkg/gui/controllers/helpers/signal_handling.go b/pkg/gui/controllers/helpers/signal_handling.go new file mode 100644 index 000000000..40d04f689 --- /dev/null +++ b/pkg/gui/controllers/helpers/signal_handling.go @@ -0,0 +1,59 @@ +//go:build !windows + +package helpers + +import ( + "os" + "os/signal" + "syscall" + + "github.com/sirupsen/logrus" + "golang.org/x/sys/unix" +) + +func canSuspendApp() bool { + return true +} + +func sendStopSignal() error { + return syscall.Kill(0, syscall.SIGSTOP) +} + +// setForegroundPgrp sets the current process group as the foreground process group +// for the terminal, allowing the program to read input after resuming from suspension. +func setForegroundPgrp() error { + fd, err := unix.Open("/dev/tty", unix.O_RDWR, 0) + if err != nil { + return err + } + defer unix.Close(fd) + + pgid := syscall.Getpgrp() + + return unix.IoctlSetPointerInt(fd, unix.TIOCSPGRP, pgid) +} + +func handleResumeSignal(log *logrus.Entry, onResume func() error) { + if err := setForegroundPgrp(); err != nil { + log.Warning(err) + return + } + + if err := onResume(); err != nil { + log.Warning(err) + } +} + +func installResumeSignalHandler(log *logrus.Entry, onResume func() error) { + go func() { + sigs := make(chan os.Signal, 1) + signal.Notify(sigs, syscall.SIGCONT) + + for sig := range sigs { + switch sig { + case syscall.SIGCONT: + handleResumeSignal(log, onResume) + } + } + }() +} diff --git a/pkg/gui/controllers/helpers/signal_handling_windows.go b/pkg/gui/controllers/helpers/signal_handling_windows.go new file mode 100644 index 000000000..8517e775d --- /dev/null +++ b/pkg/gui/controllers/helpers/signal_handling_windows.go @@ -0,0 +1,16 @@ +package helpers + +import ( + "github.com/sirupsen/logrus" +) + +func canSuspendApp() bool { + return false +} + +func sendStopSignal() error { + return nil +} + +func installResumeSignalHandler(log *logrus.Entry, onResume func() error) { +} diff --git a/pkg/gui/controllers/helpers/suspend_resume_helper.go b/pkg/gui/controllers/helpers/suspend_resume_helper.go new file mode 100644 index 000000000..cc63af495 --- /dev/null +++ b/pkg/gui/controllers/helpers/suspend_resume_helper.go @@ -0,0 +1,31 @@ +package helpers + +type SuspendResumeHelper struct { + c *HelperCommon +} + +func NewSuspendResumeHelper(c *HelperCommon) *SuspendResumeHelper { + return &SuspendResumeHelper{ + c: c, + } +} + +func (s *SuspendResumeHelper) CanSuspendApp() bool { + return canSuspendApp() +} + +func (s *SuspendResumeHelper) SuspendApp() error { + if !canSuspendApp() { + return nil + } + + if err := s.c.Suspend(); err != nil { + return err + } + + return sendStopSignal() +} + +func (s *SuspendResumeHelper) InstallResumeSignalHandler() { + installResumeSignalHandler(s.c.Log, s.c.Resume) +} diff --git a/pkg/gui/gui.go b/pkg/gui/gui.go index c02221c04..1bd5788f2 100644 --- a/pkg/gui/gui.go +++ b/pkg/gui/gui.go @@ -848,6 +848,8 @@ func (gui *Gui) Run(startArgs appTypes.StartArgs) error { gui.BackgroundRoutineMgr.startBackgroundRoutines() + gui.Helpers().SuspendResume.InstallResumeSignalHandler() + gui.c.Log.Info("starting main loop") // setting here so we can use it in layout.go @@ -899,21 +901,36 @@ func (gui *Gui) runSubprocessWithSuspenseAndRefresh(subprocess *oscommands.CmdOb return nil } +func (gui *Gui) suspend() error { + if err := gui.g.Suspend(); err != nil { + return err + } + + gui.BackgroundRoutineMgr.PauseBackgroundRefreshes(true) + return nil +} + +func (gui *Gui) resume() error { + if err := gui.g.Resume(); err != nil { + return err + } + + gui.BackgroundRoutineMgr.PauseBackgroundRefreshes(false) + return nil +} + // returns whether command exited without error or not func (gui *Gui) runSubprocessWithSuspense(subprocess *oscommands.CmdObj) (bool, error) { gui.Mutexes.SubprocessMutex.Lock() defer gui.Mutexes.SubprocessMutex.Unlock() - if err := gui.g.Suspend(); err != nil { + if err := gui.suspend(); err != nil { return false, err } - gui.BackgroundRoutineMgr.PauseBackgroundRefreshes(true) - defer gui.BackgroundRoutineMgr.PauseBackgroundRefreshes(false) - cmdErr := gui.runSubprocess(subprocess) - if err := gui.g.Resume(); err != nil { + if err := gui.resume(); err != nil { return false, err } diff --git a/pkg/gui/gui_common.go b/pkg/gui/gui_common.go index c77c81e79..d946659d1 100644 --- a/pkg/gui/gui_common.go +++ b/pkg/gui/gui_common.go @@ -42,6 +42,14 @@ func (self *guiCommon) RunSubprocess(cmdObj *oscommands.CmdObj) (bool, error) { return self.gui.runSubprocessWithSuspense(cmdObj) } +func (self *guiCommon) Suspend() error { + return self.gui.suspend() +} + +func (self *guiCommon) Resume() error { + return self.gui.resume() +} + func (self *guiCommon) Context() types.IContextMgr { return self.gui.State.ContextMgr } diff --git a/pkg/gui/keybindings.go b/pkg/gui/keybindings.go index 98850cf2d..c966b148a 100644 --- a/pkg/gui/keybindings.go +++ b/pkg/gui/keybindings.go @@ -75,7 +75,6 @@ func (gui *Gui) keybindingOpts() types.KeybindingsOpts { } } -// renaming receiver to 'self' to aid refactoring. Will probably end up moving all Gui handlers to this pattern eventually. func (gui *Gui) GetInitialKeybindings() ([]*types.Binding, []*gocui.ViewMouseBinding) { opts := gui.c.KeybindingsOpts() diff --git a/pkg/gui/pty.go b/pkg/gui/pty.go index 84c9e6da8..4253438bb 100644 --- a/pkg/gui/pty.go +++ b/pkg/gui/pty.go @@ -1,5 +1,4 @@ //go:build !windows -// +build !windows package gui diff --git a/pkg/gui/pty_windows.go b/pkg/gui/pty_windows.go index dfb2fb307..3324fa87d 100644 --- a/pkg/gui/pty_windows.go +++ b/pkg/gui/pty_windows.go @@ -1,6 +1,3 @@ -//go:build windows -// +build windows - package gui import ( diff --git a/pkg/gui/types/common.go b/pkg/gui/types/common.go index be8c10a15..bea52fbae 100644 --- a/pkg/gui/types/common.go +++ b/pkg/gui/types/common.go @@ -56,6 +56,9 @@ type IGuiCommon interface { RunSubprocess(cmdObj *oscommands.CmdObj) (bool, error) RunSubprocessAndRefresh(*oscommands.CmdObj) error + Suspend() error + Resume() error + Context() IContextMgr ContextForKey(key ContextKey) Context diff --git a/pkg/i18n/english.go b/pkg/i18n/english.go index faedeb6e5..3b4d6d95a 100644 --- a/pkg/i18n/english.go +++ b/pkg/i18n/english.go @@ -382,6 +382,8 @@ type TranslationSet struct { ScrollUp string ScrollUpMainWindow string ScrollDownMainWindow string + SuspendApp string + CannotSuspendApp string AmendCommitTitle string AmendCommitPrompt string AmendCommitWithConflictsMenuPrompt string @@ -1456,6 +1458,8 @@ func EnglishTranslationSet() *TranslationSet { ScrollUp: "Scroll up", ScrollUpMainWindow: "Scroll up main window", ScrollDownMainWindow: "Scroll down main window", + SuspendApp: "Suspend the application", + CannotSuspendApp: "Suspending the application is not supported on Windows", AmendCommitTitle: "Amend commit", AmendCommitPrompt: "Are you sure you want to amend this commit with your staged files?", AmendCommitWithConflictsMenuPrompt: "WARNING: you are about to amend the last finished commit with your resolved conflicts. This is very unlikely to be what you want at this point. More likely, you simply want to continue the rebase instead.\n\nDo you still want to amend the previous commit?", @@ -2154,6 +2158,13 @@ git: gui: useHunkModeInStagingView: false +`, + "0.55.0": `- The 'redo' command, which used to be bound to ctrl-z, is now bound to shift-Z instead. This is because ctrl-z is now used for suspending the application; it is a commonly known keybinding for that in the Linux world. If you want to revert this change, you can do so by adding the following to your config: + +keybinding: + universal: + suspendApp: + redo: `, }, } diff --git a/pkg/integration/clients/go_test.go b/pkg/integration/clients/go_test.go index 26f092a81..8984c759a 100644 --- a/pkg/integration/clients/go_test.go +++ b/pkg/integration/clients/go_test.go @@ -1,5 +1,4 @@ //go:build !windows -// +build !windows package clients diff --git a/pkg/logs/tail/logs_default.go b/pkg/logs/tail/logs_default.go index 1b866500f..3daa6c4b1 100644 --- a/pkg/logs/tail/logs_default.go +++ b/pkg/logs/tail/logs_default.go @@ -1,5 +1,4 @@ //go:build !windows -// +build !windows package tail diff --git a/pkg/logs/tail/logs_windows.go b/pkg/logs/tail/logs_windows.go index 88bf8581e..3c45d70af 100644 --- a/pkg/logs/tail/logs_windows.go +++ b/pkg/logs/tail/logs_windows.go @@ -1,6 +1,3 @@ -//go:build windows -// +build windows - package tail import ( diff --git a/schema/config.json b/schema/config.json index d2d00e4b8..db9c79638 100644 --- a/schema/config.json +++ b/schema/config.json @@ -1217,6 +1217,10 @@ "type": "string", "default": "\u003cc-c\u003e" }, + "suspendApp": { + "type": "string", + "default": "\u003cc-z\u003e" + }, "return": { "type": "string", "default": "\u003cesc\u003e" @@ -1458,7 +1462,7 @@ }, "redo": { "type": "string", - "default": "\u003cc-z\u003e" + "default": "Z" }, "filteringMenu": { "type": "string",