[TodoMVC Example] [enter] key event to determin transform of Japanese characters makes the item added to list

[FumioNonaka]

概要

日本語変換を確定する[enter]キーで項目が追加されてしまいます。

詳細

「TodoMVC の例」で日本語入力時、[enter]キーで変換候補を選ぶと、ユーザーはまだ入力中のつもりなのに、その項目がリストに追加されてしまいます。

参考資料など

「Vue.js: TodoMVCの例で日本語の項目が正しく入力できるようにする」

キーイベントをkeypressに改め、[enter]キーであることを確かめて追加するようにすれば、変換確定で処理はされません。

修正例「TodoMVC revised」

なお、v2.3.4では、keydownイベントは日本語変換を確定する[enter]キーは拾いませんでした。標準JavaScriptのkeydownイベントも同じです。現在の動作は仕様変更でしょうか。

[kazupon]

Issue レポートありがとうございます ✨

確認したところ、バージョン 2.5.14 で仕様変更したようです。
参考 issue はこちらです。
vuejs/vue#9000

ただ、Vue.js 公式 example の TodoMVC、この仕様変更に対応していないのは、あまりよろしくないので、本家公式ドキュメントレポジトリ vuejs/vuejs.org レポジトリに issue 報告した方がよさそうです。
https://github.com/vuejs/vuejs.org/issues

もしくは OSS なので PR 送って頂けると、コアチーム側としては大変助かります。 🙇

[FumioNonaka]

examples/todomvc/index.htmlを修正しました。Pull requestするにはpermissionをいただく必要があるでしょうか。

Git: remote: Permission to vuejs/vue.git denied to FumioNonaka.

[kazupon]

ありがとうございます！

Pull Request するには、 vuejs/vue レポジトリを自分のアカウントにフォークし、そのレポジトリ上で編集する必要があります。
別ブランチを作って修正をコミット後、自分の該当レポジトリにpushし、その後、GitHub上で vuejs/vue レポジトリに Pull Requestする形になります。

[FumioNonaka]

ありがとうございます。pull requestに加えました。

「Changing keyup to keypress to properly deal transform of Japanese chars」 #9172

[kazupon]

マージされたのcloseします。
ありがとうございました！