New tag should be added at the `keypress` event instead of `keyup`

[miyauchi]

keyup event will be always fired when Japanese kanji will be submitted.
The event should be changed from keyup to keypress.

Related: #42233

[miyauchi]

It's a screencast to reproduce this problem.

[miyauchi]

It's to reproduce this problem, I made mistake after.gif is a movie after fix this problem. :)

[miyauchi]

Sorry, I made a mistake in order to upload files... :)

• before.gif - It is a gif animation to reproduce this problem.
• after.gif - It is a gif animation after solved the problem.

Japanese kanji has to be confirmed by enter key.
keyup event will fire then, but keypress or keydown will never been fire when we confirm kanji.

Another place for example theme customizer are using keypress or keydown, so adding tag should use keypress too.

[nukaga]

Hi!
It is the same situation as # 42233. ( @miyauchi also writes.)
Press Enter Key to confirm.
The issue with Japanese input.
Japanese input has a meaning of selection with the Enter key, but when the Enter key press that will decide Add.
I confirmed that #42233 solve with 4.9 Beta3.
I hope the same problem will be solved as well.

[swissspidy]

@westonruter Since you worked on #42233, would you mind having a look at this one as well?

[SergeyBiryukov]

Replying to miyauchi:

Another place for example theme customizer are using keypress or keydown, so adding tag should use keypress too.

[41873] used keydown, should we be using keydown here as well instead of keypress?

[miyauchi]

Dump keycode for Japanese input

[miyauchi]

@SergeyBiryukov

I tried them to develop simple app. keypress.gif is a screencast for it.
​https://miya0001.github.io/japanese-input/

They are almost same but I guess keypress would be better.
Also, /wp-admin/nav-menus.php is using keypress.

Unfortunately, I tried keypress on [41873] but it wasn't fired, so we are using keydown on [41873].

Thanks :)