diff options
author | Misty De Meo <mistydemeo@gmail.com> | 2017-01-22 22:57:58 -0800 |
---|---|---|
committer | Misty De Meo <mistydemeo@gmail.com> | 2017-01-22 22:57:58 -0800 |
commit | 33fd8a7d95034069a4c31bec31be2feb609709e4 (patch) | |
tree | f781997d81aeb99cdeebc597dc245acc886e79ba /app/assets/javascripts | |
parent | 1953e3b6ed255870d46915db3d910345c230f076 (diff) |
Textarea: fix clicking on name suggestions
This was broken in c3e9ba6a, which added an onBlur to hide the suggestions when the textarea loses focus. Unfortunately, this fired even when the textarea lost focus to its own suggestions box, and that meant that onSuggestionClick is never called. Setting a short window in which onSuggestionClick can still happen before hiding the suggestion area ensures that clicking still works, while still hiding the suggestions if the user clicks on something else.
Diffstat (limited to 'app/assets/javascripts')
-rw-r--r-- | app/assets/javascripts/components/components/autosuggest_textarea.jsx | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/app/assets/javascripts/components/components/autosuggest_textarea.jsx b/app/assets/javascripts/components/components/autosuggest_textarea.jsx index 57352be90..81ec7a236 100644 --- a/app/assets/javascripts/components/components/autosuggest_textarea.jsx +++ b/app/assets/javascripts/components/components/autosuggest_textarea.jsx @@ -118,12 +118,19 @@ const AutosuggestTextarea = React.createClass({ }, onBlur () { - this.setState({ suggestionsHidden: true }); + // If we hide the suggestions immediately, then this will prevent the + // onClick for the suggestions themselves from firing. + // Setting a short window for that to take place before hiding the + // suggestions ensures that can't happen. + setTimeout(() => { + this.setState({ suggestionsHidden: true }); + }, 100); }, onSuggestionClick (suggestion, e) { e.preventDefault(); this.props.onSuggestionSelected(this.state.tokenStart, this.state.lastToken, suggestion); + this.textarea.focus(); }, componentWillReceiveProps (nextProps) { |