Unified Diff: include.preload.js
Issue 5715983079571456:
Issue 309 - Don't use Shadow DOM in Chrome 32 (Closed)
Patch Set: Just check for Chrome 32
Created April 17, 2014, 4:45 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« no previous file
|
no next file »
|
no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: include.preload.js |
| =================================================================== |
| --- a/include.preload.js |
| +++ b/include.preload.js |
| @@ -28,8 +28,8 @@ |
| // Use Shadow DOM if available to don't mess with web pages that |
| // rely on the order of their own <style> tags. However |
| - // the <shadow> element is broken in Chrome 32.0.1700 (#309) |
| - if ("webkitCreateShadowRoot" in document.documentElement && !/\bChrome\/32\.0\.1700\b/.test(navigator.userAgent)) |
| + // the <shadow> element is broken in some Chrome 32 builts (#309) |
|
Thomas Greiner
2014/04/22 09:47:04
Nit: Correct "builts" to "builds"
|
| + if ("webkitCreateShadowRoot" in document.documentElement && !/\bChrome\/32\b/.test(navigator.userAgent)) |
| { |
| var shadow = document.documentElement.webkitCreateShadowRoot(); |
| shadow.appendChild(document.createElement("shadow")); |
« no previous file with comments
|
« no previous file
|
no next file »
|
no next file with comments »
