Add onInput() function as listener for input events, remove unnecessary compositionupdate listener

Fix #775 (closed)

can you test and report that it's working correctly on:

• desktop chromium
• desktop firefox
• mobile chromium
• mobile firefox

also tests failed so something is definitely wrong.

Thanks for pointing that out. It looks like the failing tests are looking at the first input event when they should be looking at the last input event; previously there was only one input event fired during each test, but my change is causing another input event to be fired while the test condition is being set up. I'll adjust the tests accordingly and check all of those browsers.

added 1 commit

• 525fba0a - EmojiInput tests should be checking the input value on the last input event, not the first

Compare with previous version

Corrected the tests, everything's passing on my machine and hopefully here as well. I'll let you know when I've manually tested the browsers.

added 1 commit

• c1e38a44 - EmojiInput tests should be checking the input value on the last input event, not the first

Compare with previous version

Confirmed emoji input now works as expected on both Firefox and Chromium on both desktop and Android.

mentioned in commit 12cd96bf

merged