Bug 1120018 - [RTL][Messages] Fix composer text content direction iss…

…ue, r=julienw

apps/sms/js/subject_composer.js

@@ -62,6 +62,17 @@
     }
   }
 
+  /** Without bug 1103011 fixed, subject's caret would not be set at correct
+   * position with dir attribute set to auto. So we need to specify the auto
+   * direction only when subject is not empty.
+   */
+  function updateDirection(value) {
+    /* jshint validthis: true */
+    var priv = privateMembers.get(this);
+
+    priv.input.dir = value ? 'auto': '';
+  }
+
   function updateValue(newValue) {
     /* jshint validthis: true */
     var priv = privateMembers.get(this);
@@ -75,6 +86,7 @@
     if (priv.value !== newValue) {
       priv.value = newValue;
       priv.updatePlaceholder();
+      priv.updateDirection(newValue);
 
       this.emit('change');
     }
@@ -106,7 +118,8 @@
 
       // methods
       updateValue: updateValue.bind(this),
-      updatePlaceholder: updatePlaceholder.bind(this)
+      updatePlaceholder: updatePlaceholder.bind(this),
+      updateDirection: updateDirection.bind(this)
     };
     privateMembers.set(this, priv);
 

apps/sms/style/composer.css

@@ -107,6 +107,12 @@ form[role="search"] button[type="submit"] + p > textarea {
 
   font-size: 1.7rem;
   line-height: 2.1rem;
+
+  unicode-bidi: -moz-plaintext;
+}
+
+#messages-input.placeholder {
+  unicode-bidi: unset;
 }
 
 #messages-input:after {

apps/sms/test/unit/subject_composer_test.js

@@ -206,6 +206,21 @@ suite('SubjectComposer >', function() {
         'Placeholder should be presented'
       );
     });
+
+    // Should be revisited again once bug 1103011 landed because we might
+    // not need this logic anymore.
+    test('correctly update dir attribute depends on input value', function() {
+      // Should be empty by default
+      assert.equal(input.dir, '');
+
+      // Should be auto when text input
+      subjectComposer.setValue('a');
+      assert.equal(input.dir, 'auto');
+
+      // Should reset to empty when text removed
+      subjectComposer.setValue('');
+      assert.equal(input.dir, '');
+    });
   });
 
   suite('Backspace key handling >', function() {