Fix selection resetting in compose form after unrelated data updates

diff --git a/app/assets/javascripts/components/features/compose/components/compose_form.jsx b/app/assets/javascripts/components/features/compose/components/compose_form.jsx
index 44c44bcb0ece442fab463d79aac42b19bc227cb2..8128bb382afbfc7b9e7685eb489ee226ff7cbb46 100644 (file)
--- a/app/assets/javascripts/components/features/compose/components/compose_form.jsx
+++ b/app/assets/javascripts/components/features/compose/components/compose_form.jsx
@@ -85,14 +85,14 @@ const ComposeForm = React.createClass({
   },
 
   componentDidUpdate (prevProps) {
-    if (prevProps.in_reply_to !== this.props.in_reply_to) {
+    if (!prevProps.in_reply_to || !this.props.in_reply_to || prevProps.in_reply_to.get('id') !== this.props.in_reply_to.get('id')) {
       // If replying to zero or one users, places the cursor at the end of the textbox.
       // If replying to more than one user, selects any usernames past the first;
       // this provides a convenient shortcut to drop everyone else from the conversation.
-      let selectionStart = this.props.text.search(/\s/) + 1;
-      let selectionEnd = this.props.text.length;
-      this.autosuggestTextarea.textarea.setSelectionRange(selectionStart, selectionEnd);
+      const selectionStart = this.props.text.search(/\s/) + 1;
+      const selectionEnd = this.props.text.length;
 
+      this.autosuggestTextarea.textarea.setSelectionRange(selectionStart, selectionEnd);
       this.autosuggestTextarea.textarea.focus();
     }
   },