Skip to content

feat(tmux): Tmuxのネストトグルキーを変更#537

Merged
ncaq merged 4 commits intomasterfrom
tmux-key
Feb 8, 2026
Merged

feat(tmux): Tmuxのネストトグルキーを変更#537
ncaq merged 4 commits intomasterfrom
tmux-key

Conversation

@ncaq
Copy link
Copy Markdown
Owner

@ncaq ncaq commented Feb 8, 2026

Androidで押しづらいので。
これの一部。#530

Androidで押しづらいので。
これの一部。#530
@ncaq ncaq self-assigned this Feb 8, 2026
@ncaq ncaq added the Type: Feature New Feature label Feb 8, 2026
Copilot AI review requested due to automatic review settings February 8, 2026 09:22
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Android(Termux)環境で押しづらいF12を避け、tmuxのネスト切り替えトグルを押しやすいキーに変更して、Issue #530(Androidフレンドリーなtmux設定)の一部を進めるPRです。

Changes:

  • ネストtmux切り替えトグルのキーをF12からC-S-zへ変更
  • root/off両キーテーブル側のバインドを同じキーに揃えて更新

Comment thread home/core/tmux.nix Outdated
Comment thread home/core/tmux.nix Outdated
Comment thread home/core/tmux.nix Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings February 8, 2026 09:30
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comment thread home/core/tmux.nix Outdated
@ncaq ncaq changed the title feat(tmux): F12トグルをCtrl+Shift+zに変更 feat(tmux): Tmuxのネストトグルキーを変更 Feb 8, 2026
Copy link
Copy Markdown
Contributor

@claude claude bot left a comment

Choose a reason for hiding this comment

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

test

@claude
Copy link
Copy Markdown
Contributor

claude bot commented Feb 8, 2026

レビュー結果

セキュリティ、パフォーマンス、テストカバレッジ、ドキュメント精度の観点で問題は見つかりませんでした。変更は最小限で、コメントも正しく更新されています。

C-S-zの互換性について(既存指摘の補足)

Copilotから既に指摘されている通り、C-S-zの認識にはtmux 3.5a以降とextended-keys対応の端末が必要です。NixOS最新安定版であればtmux 3.5a以上が含まれるため問題ありませんが、もし動作しない環境があった場合はset -g extended-keys onの追加を検討してください。

Android(Termux)環境で実際にC-S-zのキーシーケンスがtmuxに正しく届くかの動作確認が取れていれば、この変更で問題ないと思います。

ncaq added 2 commits February 8, 2026 18:40
あまりEmacsとかで使っていないキーを発見したので変更。
これなら互換性問題も少なそうです。

- SSH等でネストしたtmux利用時の優先操作トグルキーをC-S-zからC-M-jに変更
- root/offテーブル両方のバインドを修正
Copilot AI review requested due to automatic review settings February 8, 2026 09:42
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

@claude
Copy link
Copy Markdown
Contributor

claude bot commented Feb 8, 2026

レビュー結果

コード品質、セキュリティ、パフォーマンス、ドキュメント精度、テストカバレッジの各観点でレビューしました。

特筆すべき問題はありません。変更は最小限で正確です:

  • rootテーブルとoffテーブルの両方のバインドが一貫してC-M-jに更新されている
  • インラインコメントもキー表記が正しく更新されている
  • トグルロジック自体に変更はなく、キーバインドの識別子のみの変更

LGTM

@claude
Copy link
Copy Markdown
Contributor

claude bot commented Feb 8, 2026

レビュー結果

コード品質、ドキュメント正確性、パフォーマンス、セキュリティ、テストカバレッジの各観点でレビューしました。特筆すべき問題はありません。

  • コメント内のキー記述(C-M-j)と実際のbindコマンドが一致しており、ドキュメントは正確です
  • C-S-zからC-M-jへの変更により、前回指摘されたtmux 3.5a以降/extended-keysの互換性問題も解消されています
  • C-M-j(Ctrl+Alt+j)はターミナルエミュレータ間の互換性が高く、適切な選択です

@ncaq ncaq merged commit 6192201 into master Feb 8, 2026
13 checks passed
@ncaq ncaq deleted the tmux-key branch February 8, 2026 09:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Type: Feature New Feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants