Don't setPlaceholder if the element has focus. Fixes mathiasbynens#56.

Asterodeia · Feb 23, 2012 · cb53e48 · cb53e48
1 parent 5148335
commit cb53e48
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/jquery.placeholder.js b/jquery.placeholder.js
@@ -45,8 +45,11 @@
 				}
 				if (value == '') {
 					element.value = value;
-					// We can’t use `triggerHandler` here because of dummy text/password inputs :(
-					setPlaceholder.call(element);
+					// Issue #56: Setting the placeholder causes problems if the element continues to have focus.
+					if (!$element.is(':focus')) {
+						// We can’t use `triggerHandler` here because of dummy text/password inputs :(
+						setPlaceholder.call(element);
+					}
 				} else if ($element.hasClass('placeholder')) {
 					clearPlaceholder.call(element, true, value) || (element.value = value);
 				} else {