From 401f70d35bb1e3b5ccd9038a0fc08024d3d1e35b Mon Sep 17 00:00:00 2001
From: Kevin Ball <kmball11@gmail.com>
Date: Thu, 8 Dec 2016 14:26:55 -0800
Subject: [PATCH] Fix issues with long titles and other drilldown height
 calculation bugs

---
 js/foundation.drilldown.js      | 19 ++++++++++++++-----
 scss/components/_drilldown.scss |  5 ++++-
 2 files changed, 18 insertions(+), 6 deletions(-)

diff --git a/js/foundation.drilldown.js b/js/foundation.drilldown.js
index 72f2f22ab..aae933011 100644
--- a/js/foundation.drilldown.js
+++ b/js/foundation.drilldown.js
@@ -99,6 +99,11 @@ class Drilldown {
       }
       _this._back($menu);
     });
+
+    if(!this.options.autoHeight) {
+      this.$submenus.addClass('drilldown-submenu-cover-previous');
+    }
+
     if(!this.$element.parent().hasClass('is-drilldown')){
       this.$wrapper = $(this.options.wrapper).addClass('is-drilldown');
       if(this.options.animateHeight) this.$wrapper.addClass('animate-height');
@@ -361,17 +366,18 @@ class Drilldown {
    * @private
    */
   _getMaxDims() {
-    var max = 0, result = {}, oneHeight = this.$menuItems[0].getBoundingClientRect().height,_this = this;
+    var  maxHeight = 0, result = {}, _this = this;
     this.$submenus.add(this.$element).each(function(){
       var numOfElems = $(this).children('li').length;
-      max = numOfElems > max ? numOfElems : max;
+      var height = Foundation.Box.GetDimensions(this).height;
+      maxHeight = height > maxHeight ? height : maxHeight;
       if(_this.options.autoHeight) {
-        $(this).data('calcHeight',numOfElems * oneHeight);
-        if (!$(this).hasClass('is-drilldown-submenu')) result['height'] = numOfElems * oneHeight;
+        $(this).data('calcHeight',height);
+        if (!$(this).hasClass('is-drilldown-submenu')) result['height'] = height;
       }
     });
 
-    if(!this.options.autoHeight) result['min-height'] = `${max * oneHeight}px`;
+    if(!this.options.autoHeight) result['min-height'] = `${maxHeight}px`;
 
     result['max-width'] = `${this.$element[0].getBoundingClientRect().width}px`;
 
@@ -394,6 +400,9 @@ class Drilldown {
     this.$submenuAnchors.each(function() {
       $(this).off('.zf.drilldown');
     });
+
+    this.$submenus.removeClass('drilldown-submenu-cover-previous');
+
     this.$element.find('a').each(function(){
       var $link = $(this);
       $link.removeAttr('tabindex');
diff --git a/scss/components/_drilldown.scss b/scss/components/_drilldown.scss
index 80c721cbb..82398e759 100644
--- a/scss/components/_drilldown.scss
+++ b/scss/components/_drilldown.scss
@@ -48,7 +48,6 @@ $drilldown-background: $white !default;
     #{$global-left}: 100%;
     z-index: -1;
 
-    min-height: 100%;
     width: 100%;
     background: $drilldown-background;
     transition: $drilldown-transition;
@@ -64,6 +63,10 @@ $drilldown-background: $white !default;
     }
   }
 
+  .drilldown-submenu-cover-previous {
+    min-height: 100%;
+  }
+
   @if $drilldown-arrows {
     .is-drilldown-submenu-parent > a {
       position: relative;
-- 
2.47.2