Make placeholder logic smarter (fixes tiff#110)

abpin · Jun 14, 2012 · ecb0a29 · ecb0a29
1 parent e090eec
commit ecb0a29
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/src/dom/simulate_placeholder.js b/src/dom/simulate_placeholder.js
@@ -19,10 +19,12 @@
           if (view.hasPlaceholderSet()) {
             view.clear();
           }
+          view.placeholderSet = false;
           dom.removeClass(view.element, CLASS_NAME);
         },
         set = function() {
           if (view.isEmpty()) {
+            view.placeholderSet = true;
             view.setValue(placeholderText);
             dom.addClass(view.element, CLASS_NAME);
           }

diff --git a/src/views/composer.js b/src/views/composer.js
@@ -92,7 +92,7 @@
     },
 
     hasPlaceholderSet: function() {
-      return this.getTextContent() == this.textarea.element.getAttribute("placeholder");
+      return this.getTextContent() == this.textarea.element.getAttribute("placeholder") && this.placeholderSet;
     },
 
     isEmpty: function() {