Merge pull request php-debugbar#252 from codewizz/codewizz-dynamic-margin

Offset calculation change to body margin-bottom

Author: barryvdh

src/DebugBar/Resources/debugbar.js

@@ -395,8 +395,8 @@ if (typeof(PhpDebugBar) == 'undefined') {
         className: "phpdebugbar " + csscls('minimized'),
 
         options: {
-            bodyPaddingBottom: true,
-            bodyPaddingBottomHeight: parseInt($('body').css('padding-bottom'))
+            bodyMarginBottom: true,
+            bodyMarginBottomHeight: parseInt($('body').css('margin-bottom'))
         },
 
         initialize: function() {
@@ -793,7 +793,16 @@ if (typeof(PhpDebugBar) == 'undefined') {
             this.$el.addClass(csscls('closed'));
             this.recomputeBottomOffset();
         },
-
+        
+        /**
+         * Checks if the panel is closed
+         *
+         * @return {Boolean}
+         */
+        isClosed: function() {
+            return this.$el.hasClass(csscls('closed'));
+        },
+        
         /**
          * Restore the debug bar
          *
@@ -813,13 +822,17 @@ if (typeof(PhpDebugBar) == 'undefined') {
         },
 
         /**
-         * Recomputes the padding-bottom css property of the body so
+         * Recomputes the margin-bottom css property of the body so
          * that the debug bar never hides any content
          */
         recomputeBottomOffset: function() {
-            if (this.options.bodyPaddingBottom) {
-                var height = parseInt(this.$el.height()) + this.options.bodyPaddingBottomHeight;
-                $('body').css('padding-bottom', height);
+            if (this.options.bodyMarginBottom) {
+                if (this.isMinimized() || this.isClosed()) {
+                    return $('body').css('margin-bottom', this.options.bodyMarginBottomHeight || '');
+                }
+                
+                var offset = parseInt(this.$el.height()) + this.options.bodyMarginBottomHeight;
+                $('body').css('margin-bottom', offset);
             }
         },