Skip to content

feat: いい感じ変換のショートカット(Ctrl+S固定)を設定で変更できるようにする #352

Description

@tomippe

提案

関連: #280 #265

いい感じ変換のトリガーが Ctrl+S 固定のため、他アプリのキーバインドとバッティングします。#280 のEmacsインクリメンタルサーチのほか、私の場合は仮想マシン内Windowsの「上書き保存(Ctrl+S)」と衝突し、VM作業中に保存のつもりでいい感じ変換が起動してしまいます。

#280 で共有された「システム設定のアプリケーションショートカットでメニュータイトルに割り当てる」方法は現在も使えますが、IMEの設定画面内で完結できるとより発見しやすく親切だと思います。

現状のコード

配線が途中の状態に見えます。

  • Config.TransformShortcutKeyboardShortcutConfigItem.swift、デフォルト Ctrl+S)と、ショートカット録画UI KeyboardShortcutRecorder は既に存在する
  • 一方、キー判定は UserAction.getUserAction("s", [.control]) ハードコードのままで、Config.TransformShortcut を参照していない
  • 設定画面にもこの項目は露出していない

フォークでの実装

以下をフォークで実装し、動作確認済みです。

  1. getUserActionConfig.TransformShortcut の値と charactersIgnoringModifiers + 修飾キーを照合して .suggest を返す(ハードコードの ("s", [.control]) は削除)
  2. 設定画面「基本」タブの「いい感じ変換」セクションに KeyboardShortcutRecorder を配置(Deleteでデフォルト⌃Sにリセット)
  3. メニューバーの「いい感じ変換」項目の keyEquivalent 表示も設定値に追従
  4. ユニットテスト追加(デフォルトCtrl+Sの動作、変更後は新キーのみ反応することを確認)

方向性に問題がなければPRとして送ります。

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions