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

単語登録中に単語登録削除に遷移しキャンセルしたら単語登録中に戻るように修正 #171

Merged
merged 3 commits into from
Jun 16, 2024

Conversation

mtgto
Copy link
Owner

@mtgto mtgto commented Jun 11, 2024

単語登録中に単語登録解除に遷移してキャンセルしたときに想定された挙動になってなかったのを修正します。

再現手順

  1. 単語登録を開始
  2. 単語登録中に単語変換を開始
  3. 単語変換候補選択時にShift-Xで単語登録解除に遷移
  4. C-gやESCでキャンセル

想定された動作

単語登録中の変換候補の選択中

実際の挙動

単語登録中ではない変換候補の選択中

@@ -1032,8 +1039,15 @@ final class StateMachine {
}
return true
case .unregister:
let prevRegisterState: (RegisterState, [RegisterState])?
if case .register(let registerState, _) = specialState {
prevRegisterState = (registerState, [])
Copy link
Owner Author

Choose a reason for hiding this comment

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

これだと再帰で単語登録しているときにそれを保持するようになってないので直す

@mtgto mtgto merged commit 5d01f4b into main Jun 16, 2024
2 checks passed
@mtgto mtgto deleted the fix-register branch June 16, 2024 09:12
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.

1 participant