Permalink
Browse files

smarter contenteditable test. linking up html5test source, as well. f…

…ixes #570
  • Loading branch information...
1 parent c2a0937 commit d693c56d73b7bb6981d0e1353582cbfbc0e715c5 @paulirish paulirish committed May 11, 2012
Showing with 7 additions and 2 deletions.
  1. +7 −2 feature-detects/contenteditable.js
@@ -1,4 +1,9 @@
// contentEditable
// http://www.whatwg.org/specs/web-apps/current-work/multipage/editing.html#contenteditable
-// by Addy Osmani
-Modernizr.addTest('contenteditable', 'isContentEditable' in document.documentElement);
+
+// this is known to false positive in some mobile browsers
+// here is a whitelist of verified working browsers:
+// https://github.com/NielsLeenheer/html5test/blob/549f6eac866aa861d9649a0707ff2c0157895706/scripts/engine.js#L2083
+
+Modernizr.addTest('contenteditable',
+ 'contentEditable' in document.documentElement);

0 comments on commit d693c56

Please sign in to comment.