Fix a false-negative for number fields in Safari 5.0.3.

Modernizr · Dec 21, 2010 · f84b3d9 · f84b3d9
1 parent 6968d6e
commit f84b3d9
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/modernizr.js b/modernizr.js
@@ -752,10 +752,9 @@ window.Modernizr = (function(window,document,undefined){
                 inputElem.setAttribute('type', props[i]);
                 bool = inputElem.type !== 'text';
 
-                // Chrome likes to falsely purport support, so we feed it a textual value;
+                // Webkit likes to falsely purport support, so we feed it a textual value;
                 // if that doesnt succeed then we know there's a custom UI
                 if (bool){  
-
                     inputElem.value = smile;
 
                     if (/^range$/.test(inputElem.type) && inputElem.style.WebkitAppearance !== undefined){
@@ -781,10 +780,13 @@ window.Modernizr = (function(window,document,undefined){
                       //  even make it here.
 
                     } else if (/^(url|email)$/.test(inputElem.type)) {
-
                       // Real url and email support comes with prebaked validation.
                       bool = inputElem.checkValidity && inputElem.checkValidity() === false;
-
+
+                    } else if (inputElem.type === "number"){
+                        // Safari 5.0.3 accepts the smile for number fields, which it actually does support...
+                        bool = inputElem.value != smile || inputElem.checkValidity && inputElem.checkValidity() === false
+
                     } else {
                       // If the upgraded input compontent rejects the :) text, we got a winner
                       bool = inputElem.value != smile;