Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ノーマル時のかなキーの動作をCtrl-Jと同じにする #133

Merged
merged 1 commit into from
Mar 8, 2024

Conversation

y-yu
Copy link
Contributor

@y-yu y-yu commented Mar 3, 2024

  • Ctrl-jC-j)はノーマルモード時にはOSのIME切り替えとは別にIME状態を切り替えるようになっている
    • よって、たとえばCtrl-qで半角英数入力となっている状態からCtrl-jでかな入力へ戻ることができる
    • この振る舞いも(別のテストで利用されていたが)Ctrl-jのテストに追加した
  • README.mdのワークアラウンドとして、特定のアプリでCtrl-jをかなキーへ変換するようなものが紹介されているが、この際にかなキーの動作がCtrl-jと同じではないと混乱が生じると考えられる
  • よって、ノーマルモード時のかなキー入力時はCtrl-jと同じ分岐へ進入するようにし、テストを追加した

@y-yu
Copy link
Contributor Author

y-yu commented Mar 3, 2024

差分ゼロの #134 が落ちているので、上流の問題かな 🤔

@mtgto
Copy link
Owner

mtgto commented Mar 3, 2024

testがこけたのはmainブランチに入れた修正による問題でした。すみません。
6a9fd4a で修正しました。

#122pull_request_target で動かしているので、テストが実行されるリビジョンに上記の修正が含まれないので、おてすきのときにmainを取り込んでもらえるとうれしいです (rebase & force pushでOKです)

@y-yu y-yu force-pushed the kana-key-as-same-as-ctrl-j branch from 816c823 to 40b7afd Compare March 3, 2024 12:51
@y-yu
Copy link
Contributor Author

y-yu commented Mar 3, 2024

✅ rebase & force push

@mtgto
Copy link
Owner

mtgto commented Mar 8, 2024

なるほど、ありがとうございます。

かなキーを押したときの挙動のBefore/After (最初わからなかったので自分用メモ)

状態 Before After
カタカナ (q) 🆖 カタカナのまま 全角かな
半角カナ (C-q) 🆖 半角カナのまま 全角かな
半角英数 (l) 🆗 全角かな 全角かな
全角英数 (Shift-L) 🆖 全角英数のまま 全角かな

@mtgto mtgto merged commit bfe47f5 into mtgto:main Mar 8, 2024
2 checks passed
@mtgto
Copy link
Owner

mtgto commented Mar 8, 2024

テストもていねいで助かります。ありがとうございました!

@y-yu y-yu deleted the kana-key-as-same-as-ctrl-j branch March 8, 2024 14:08
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