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

[v2] Pressing <Enter> in IME mode should not select a option #2767

Merged
merged 1 commit into from
Jul 23, 2018

Conversation

shamabe
Copy link
Contributor

@shamabe shamabe commented Jul 3, 2018

In composing mode, enter key is for selecting a suggest of conversion.
So that, the behavior of enter key is collision in "react-select" with IME.

Like this...
react-select-before

It is necessary to check whether it is in composing-mode or not.
Following is the preferred behavior.

react-select-after

@shamabe shamabe changed the title Pressing <Enter> in IME mode should not select a option [v2] Pressing <Enter> in IME mode should not select a option Jul 3, 2018
Copy link
Collaborator

@gwyneplaine gwyneplaine left a comment

Choose a reason for hiding this comment

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

@shamabe this looks good to me, but wanted to get @JedWatson eyes on this, this seems like core behaviour and makes sense to include in the base select. However perhaps you'd like this in a wrapping component instead? Would be good to get your thoughts on this Jed.

@jossmac jossmac added the v2 label Jul 4, 2018
@JedWatson
Copy link
Owner

This looks great @shamabe, thanks!

@JedWatson JedWatson merged commit 1704201 into JedWatson:v2 Jul 23, 2018
@maplesyrupsucker
Copy link

@GitCash send 0.005 BCH to @shamabe

@maplesyrupsucker
Copy link

@GitCash send 0.0025 to @shamabe for being amazing at vim

@maplesyrupsucker
Copy link

@GitCash send 0.005 BCH to @shamabe for being amazing at vim

@maplesyrupsucker
Copy link

@GitCash send 0.0025 BCH to @shamabe

@GitCash
Copy link

GitCash commented Aug 3, 2018

Hey shamabe, user maplesyrupsucker tipped you 2500 bits in Bitcoin Cash ( ~ $1.8125 ).

Click here to claim it!

You can also add the "thumbs down" reaction to maplesyrupsucker's comment above to prevent future tips.

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.

None yet

6 participants