feat: smoothDeleteに半角文字を追加
Is your feature request related to a problem? Please describe. (提案は利用上の問題に関連していますか?記述してください。)
- 日本語キーボードの一括削除機能が半角?や!を認識しないため、英文のセンテンス単位での削除やurlのパラメータ削除に使用できない
Describe the solution you'd like (あなたの考える解決策を記述してください)
- InputManager.swift#L525, InputManager.swift#L581, InputManager.swift#L631の配列に"?"と"!"を追加する
Describe alternatives you've considered (他の解決方法があれば記述してください)
If we were to implement this feature, will you help implement it? (もし提案する機能を実装する場合、実装を手伝う余裕はありますか?)
Yes
Additional context (その他共有したい事項があれば記述してください)
- ピリオドまで削除ができるのにクエスチョンマークまで削除ができないのは直感に反する。
- そもそも需要がニッチなのでデフォルトに追加するかは選択の余地があるかと思いますが、設定に追加するほどでもないかなとも思ってます。
良いと思います! 実装は @ph0ryn にお願いしても良いでしょうか?
ありがとうございます!今気づいたのですが同じ配列を採用してるInputManager.swift#L712のsmartMoveCursorForwardも変えちゃっていいでしょうか?
良いと思うのですが、カスタムタブ側がどうなるのかだけちょっと気になりました。もしかするとこの辺も触らないとダメかも https://github.com/azooKey/CustardKit/blob/563635caf1213dd6b2baff63ed1b0cf254b9d78a/swift/sources/CustardActions.swift#L248
なるほど、そっちは確認できてませんでした...すみません。CustardKitは1,2回しか使ったことがなくであまり把握できていないので見てきます
一通り目を通してきました。あくまでazookey側のデフォルトの引数の値なので特に問題はないかと思います。custardが渡す引数nextsが型違反しない限りなんでも読めるという理解です。ただ、デフォルトの挙動が異なるのはそれこそ直感に反するので合わせて修正したいですね。
https://github.com/ph0ryn/azooKey/commit/82641b0d00b3a9241007f4157b648f8953e30a62
今回追加した?と!だけ前のスペースが残ってしまうんですが、もし何かご存知でしたら教えてください
[.] It's pen.| Isn't it
[,] Yes,| It's pen
[?] Is it pen? |No
[!] It's pen! |However
前のスペースが残る理由はわからないです。ロジックを読む限り動くと思うのですが、printデバッグすればわかりそうです