CRM-15341 - move cms-specific code to appropriate file
authorColeman Watts <coleman@civicrm.org>
Fri, 26 Sep 2014 17:09:19 +0000 (13:09 -0400)
committerColeman Watts <coleman@civicrm.org>
Fri, 26 Sep 2014 17:13:13 +0000 (13:13 -0400)
js/Common.js
js/crm.drupal.js

index db365f3bb9913741f599defe5adc3589a8b024e9..2758eb91bae0a5eb9649fb5f9bba4283ccc503c2 100644 (file)
@@ -553,15 +553,11 @@ CRM.strings = CRM.strings || {};
           e.preventDefault();
         });
       }
-      // FIXME: D7 hack to get the toolbar out of the way (CRM-15341)
-      if (CRM.config.userFramework === 'Drupal') $('#toolbar').css('z-index', '100');
     })
     .on('dialogclose', function(e) {
       // Restore scrollbars when closing modal
       if ($('.ui-dialog .modal-dialog:visible').not(e.target).length < 1) {
         $('body').css({overflow: ''});
-        // FIXME: D7 hack, restore toolbar (CRM-15341)
-        if (CRM.config.userFramework === 'Drupal') $('#toolbar').css('z-index', '');
       }
     })
     .on('submit', function(e) {
index 0e352f996a814b9501b8e8056d82cbaa8deb9228..0d8dadcf9e423416e3f60c60b099d597f81c522c 100644 (file)
@@ -1,7 +1,18 @@
 // http://civicrm.org/licensing
 CRM.$(function($) {
-  $(document).on('crmLoad', function() {
-    // This is drupal's old-school way of listening for 'load' type events. It has to be called manually.
-    Drupal.attachBehaviors(this);
-  });
+  $(document)
+    .on('crmLoad', function() {
+      // This is drupal's old-school way of listening for 'load' type events. It has to be called manually.
+      Drupal.attachBehaviors(this);
+    })
+    .on('dialogopen', function(e) {
+      // D7 hack to get the toolbar out of the way (CRM-15341)
+      $('#toolbar').css('z-index', '100');
+    })
+    .on('dialogclose', function(e) {
+      if ($('.ui-dialog-content:visible').not(e.target).length < 1) {
+        // D7 hack, restore toolbar position (CRM-15341)
+        $('#toolbar').css('z-index', '');
+      }
+    })
 });