LP#1646166 Hatch attach point is documentElement
authorBill Erickson <berickxx@gmail.com>
Fri, 27 Jan 2017 15:39:31 +0000 (10:39 -0500)
committerKathy Lussier <klussier@masslnc.org>
Thu, 16 Feb 2017 20:21:36 +0000 (15:21 -0500)
Hatch now inserts its attribute into the root documentElement instead of
the body.  This is part of the move to using content_scripts instead of
declarativeContent for the extension.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
Open-ILS/src/templates/staff/base.tt2
Open-ILS/web/js/ui/default/staff/services/hatch.js

index 85cab84..445b1ad 100644 (file)
@@ -27,7 +27,7 @@
     <link rel="stylesheet" href="[% ctx.base_path %]/staff/css/cat.css" />
     <link rel="stylesheet" href="[% ctx.base_path %]/staff/css/style.css" />
   </head>
-  <body hatch-is-welcome-here>
+  <body>
     <toast></toast>
 
     <!-- load the navbar template inline since it's used on every page -->
index 0dc1677..4fee7cb 100644 (file)
@@ -123,9 +123,8 @@ angular.module('egCoreMod')
     service.openHatch = function() {
 
         // When the Hatch extension loads, it tacks an attribute onto
-        // the page body to indicate it's available.
-
-        if (!$window.document.body.getAttribute('hatch-is-open')) {
+        // the top-level documentElement to indicate it's available.
+        if (!$window.document.documentElement.getAttribute('hatch-is-open')) {
             console.debug("Hatch is not available");
             return;
         }