Issue 2602 - Don't use shadow DOM on mail.google.com, fixing an issue with the feedback form there

include.preload.js

Index: include.preload.js
===================================================================
--- a/include.preload.js
+++ b/include.preload.js
@@ -195,9 +195,11 @@
   // However, creating a shadow root breaks running CSS transitions. So we
   // have to create the shadow root before transistions might start (#452).
   //
-  // Also, we can't use shadow DOM on Google Docs, since it breaks printing
-  // there (#1770).
-  if ("createShadowRoot" in document.documentElement && document.domain != "docs.google.com")
+  // Also, we can't use shadow DOM on Google Docs since it breaks printing
+  // there (#1770), and Google Mail since the feedback from breaks the layout
+  // there when using shadow DOM (#2602).
+  if ("createShadowRoot" in document.documentElement && document.domain != "docs.google.com"

[Thomas Greiner, 2015/06/03 14:24:25]

Let's stop hitting the 80 characters limit here e.g. by writing it like this:

if ("createShadowRoot" in document.documentElement
    && document.domain != "docs.google.com"
    && document.domain != "mail.google.com")

or alternatively (in case we're anticipating more Google domains to join the
list):

if ("createShadowRoot" in document.documentElement
    && !/^(?:docs|mail)\.google\.com$/.test(document.domain))

[Sebastian Noack, 2015/06/03 15:33:07]

Done.
If the list is getting much longer, I'd probably use an object as set here.

+                                                     && document.domain != "mail.google.com")
   {
     shadow = document.documentElement.createShadowRoot();
     shadow.appendChild(document.createElement("shadow"));