make suggestion box focus more robust, require more explicit enabling (#7123)

webapp/components/suggestion/suggestion_box.jsx

@@ -85,7 +85,12 @@ export default class SuggestionBox extends React.Component {
         /**
          * The number of characters required to show the suggestion list, defaults to 1
          */
-        requiredCharacters: PropTypes.number
+        requiredCharacters: PropTypes.number,
+
+        /**
+         * If true, the suggestion box is opened on focus, default to false
+         */
+        openOnFocus: PropTypes.bool
     }
 
     static defaultProps = {
@@ -94,7 +99,8 @@ export default class SuggestionBox extends React.Component {
         renderDividers: false,
         completeOnTab: true,
         isRHS: false,
-        requiredCharacters: 1
+        requiredCharacters: 1,
+        openOnFocus: false
     }
 
     constructor(props) {
@@ -145,7 +151,7 @@ export default class SuggestionBox extends React.Component {
     }
 
     getTextbox() {
-        if (this.props.type === 'textarea') {
+        if (this.props.type === 'textarea' && this.refs.textbox) {
             const node = this.refs.textbox.getDOMNode();
             return node;
         }
@@ -171,13 +177,17 @@ export default class SuggestionBox extends React.Component {
     }
 
     handleFocus() {
+        if (!this.props.openOnFocus) {
+            return;
+        }
         setTimeout(() => {
             const textbox = this.getTextbox();
+            if (textbox) {
                 const pretext = textbox.value.substring(0, textbox.selectionEnd);
-
                 if (pretext.length >= this.props.requiredCharacters) {
                     GlobalActions.emitSuggestionPretextChanged(this.suggestionId, pretext);
                 }
+            }
         });
     }
 
@@ -346,6 +356,7 @@ export default class SuggestionBox extends React.Component {
         Reflect.deleteProperty(props, 'isRHS');
         Reflect.deleteProperty(props, 'popoverMentionKeyClick');
         Reflect.deleteProperty(props, 'requiredCharacters');
+        Reflect.deleteProperty(props, 'openOnFocus');
 
         const childProps = {
             ref: 'textbox',

webapp/plugins/jira/components/settings.jsx

@@ -206,6 +206,7 @@ export default class JIRASettings extends AdminSettings {
                             disabled={!this.state.enabled}
                             type='input'
                             requiredCharacters={0}
+                            openOnFocus={true}
                         />
                     </div>
                 </Setting>