Repository: jesseduffield/lazygit (language: go)

Issue:
Incorrect display of some keybindings
**Describe the bug**
When keybindings are displayed, `<space>`, `<enter>` and `/` are being shown strike through and not with the proper color

**To Reproduce**
Steps to reproduce the behavior:

1. Go to any panel
2. Click on `?`
3. Scroll down to any of the mentioned keybindings
4. See error

**Expected behavior**
The keybindings should be shown with the proper color and format

**Screenshots**
![bug_report](https://github.com/jesseduffield/lazygit/assets/38757307/a423f8c6-6414-4a33-9fce-d89273f36545)

**Version info:**
- **lazygit**
`commit=, build date=, build source=homebrew, version=0.39.3, os=darwin, arch=arm64, git version=2.41.0`

- **git**
`git version 2.41.0`



Discussion / hints:
The intention behind striking through enter and space is to tell you that you can't invoke the menu item with that keybinding, because the keybinding is reserved for invoking the currently selected menu item. As for '/', I'm not sure why that's struck-through, perhaps you've rebound that as a confirmation key?
The `/` keybinding is also formatted with the strikethrough on mine (version 0.40.2) on a completely fresh install with no customizations. The filter functions correctly in the help view even though the bind is marked with the strike.

config.yml is blank and state.yml contains:

```
lastupdatecheck: 0
recentrepos:
- *redacted*
startuppopupversion: 5
customcommandshistory: []
hidecommandlog: false
ignorewhitespaceindiffview: true
```
@rholak thats because the keybinding in the menu refers to filtering the original panel rather than the menu panel
@jesseduffield Thank you;  that makes sense now. 
I suggest adding this formatting convention as a FAQ entry or under the Keybinding section of the readme  - I got some co-workers to start using lazygit as well, and they were also wondering why some of the items in the keybindings screen were crossed out.

Fix patch:
diff --git a/pkg/gui/context/menu_context.go b/pkg/gui/context/menu_context.go
index 7f8656fe7b9..2158d5c7ac6 100644
--- a/pkg/gui/context/menu_context.go
+++ b/pkg/gui/context/menu_context.go
@@ -92,21 +92,8 @@ func (self *MenuViewModel) GetDisplayStrings(_ int, _ int) [][]string {
 			return displayStrings
 		}
 
-		// These keys are used for general navigation so we'll strike them out to
-		// avoid confusion
-		reservedKeys := []string{
-			self.c.UserConfig.Keybinding.Universal.Confirm,
-			self.c.UserConfig.Keybinding.Universal.Select,
-			self.c.UserConfig.Keybinding.Universal.Return,
-			self.c.UserConfig.Keybinding.Universal.StartSearch,
-		}
 		keyLabel := keybindings.LabelFromKey(item.Key)
-		keyStyle := style.FgCyan
-		if lo.Contains(reservedKeys, keyLabel) {
-			keyStyle = style.FgDefault.SetStrikethrough()
-		}
-
-		displayStrings = utils.Prepend(displayStrings, keyStyle.Sprint(keyLabel))
+		displayStrings = utils.Prepend(displayStrings, style.FgCyan.Sprint(keyLabel))
 		return displayStrings
 	})
 }


Test patch: