Merge pull request mathiasbynens#57 from sbull/empty-string-val-issue-56

Don’t `setPlaceholder` if the element has focus Fixes mathiasbynens#56.
Asterodeia · Feb 23, 2012 · a9fd43d · a9fd43d
2 parents 5148335 + cb53e48
commit a9fd43d
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 {