Skip to content

Fix Tmux hotkeys in README to match actual tmux.conf bindings - #9

Open
japaz wants to merge 1 commit into
omacom-io:masterfrom
japaz:fix/tmux-hotkeys-readme
Open

Fix Tmux hotkeys in README to match actual tmux.conf bindings#9
japaz wants to merge 1 commit into
omacom-io:masterfrom
japaz:fix/tmux-hotkeys-readme

Conversation

@japaz

@japaz japaz commented Apr 18, 2026

Copy link
Copy Markdown

Summary

  • All Ctrl + Cmd + … and Ctrl + Shift + Page* chords in the Tmux section were impossible
  • The real bindings from tmux.conf use prefix-based keys (h/v/x), Ctrl + ⌥ + arrow for pane focus/resize, ⌥ + arrow for window navigation, and Prefix + Shift + c/k/r/p/n / ⌥ + ↑↓ for sessions.
  • Fixed "Kill window" which incorrectly pointed to Prefix + x (that kills a pane; kill-window is Prefix + k).
  • Added missing ⌥ + 1–9 window-jump and ⌥ + Shift + ←/→ swap-window bindings.

Test plan

  • Open a tmux session and verify each listed hotkey fires its action with no "unknown key" bell.
  • Confirm Prefix + h / Prefix + v create pane splits.
  • Confirm Ctrl + ⌥ + arrow moves between panes.
  • Confirm ⌥ + ←/→ switches windows, ⌥ + 1–9 jumps to window N.
  • Confirm Prefix + Shift + c creates a new session and ⌥ + ↑/↓ switches between sessions.

Cmd key cannot be forwarded through macOS terminals to tmux, so all
Ctrl+Cmd and Ctrl+Shift+Page* chords were impossible. Replace with the
real bindings from tmux.conf: prefix-based h/v/x for pane splits/kill,
Ctrl+Option+arrows for pane focus/resize, Option+arrows and prefix+c/k/r
for windows, and prefix+Shift+c/k/r/p/n plus Option+Up/Down for sessions.
Also fix 'kill window' which incorrectly listed prefix+x (kills a pane).
Add missing Option+1-9 window jump and Option+Shift+arrow swap bindings.
Copilot AI review requested due to automatic review settings April 18, 2026 20:59

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the README’s tmux hotkey documentation so it reflects usable macOS key chords and the intended tmux prefix-based workflow.

Changes:

  • Replaced impossible Ctrl + Cmd + … and Ctrl + Shift + Page* tmux chords with prefix-based and Option-based bindings.
  • Corrected window vs pane kill shortcuts and expanded window navigation/swap/jump shortcuts.
  • Updated session management shortcuts to match the documented prefix/Option navigation model.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread README.md
Comment on lines +74 to +76
| `Prefix + h` | Split horizontally (new pane below) |
| `Prefix + v` | Split vertically (new pane to the right) |
| `Prefix + x` | Kill pane |

Copilot AI Apr 18, 2026

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These keybindings are described as coming from tmux.conf, but there’s no tmux.conf (or tmux config) in this repo (it looks like tmux config comes indirectly via the Omadots installer). Consider linking to the exact upstream config file/commit that defines these bindings (or adding the tmux config here) to prevent the README hotkey list from drifting again.

Copilot uses AI. Check for mistakes.
Comment thread README.md
Comment on lines +81 to +84
| `Ctrl + ⌥ + Shift + ←` | Resize left |
| `Ctrl + ⌥ + Shift + ↓` | Resize down |
| `Ctrl + ⌥ + Shift + ↑` | Resize up |
| `Ctrl + ⌥ + Shift + →` | Resize right |

Copilot AI Apr 18, 2026

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR description says pane focus/resize uses Ctrl + ⌥ + arrow, but the README documents resize as Ctrl + ⌥ + Shift + arrow. Please align the PR description/test plan with what’s documented here (or update the documented shortcuts if Shift isn’t actually required).

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants