Fixes a problem where calculated max-height might get overridden

Under certain circumstances calculated max-height might get overridden
by the one specified in the css (+ if the css isn’t mobile first).

Author: arielsalminen

Diff for: client/dist/responsive-nav.js

@@ -421,7 +421,7 @@
         for (var i = 0; i < nav.inner.length; i++) {
           savedHeight += nav.inner[i].offsetHeight;
         }
-        var innerStyles = "." + opts.navClass + ".opened{max-height:" + savedHeight + "px}";
+        var innerStyles = "." + opts.navClass + ".opened{max-height:" + savedHeight + "px !important}";
 
         if (styleElement.styleSheet) {
           styleElement.styleSheet.cssText = innerStyles;

Diff for: client/dist/responsive-nav.min.js

@@ -303,7 +303,7 @@ var responsiveNav = function (el, options) {
       for (var i = 0; i < nav.inner.length; i++) {
         savedHeight += nav.inner[i].offsetHeight;
       }
-      var innerStyles = "." + opts.navClass + ".opened{max-height:" + savedHeight + "px}";
+      var innerStyles = "." + opts.navClass + ".opened{max-height:" + savedHeight + "px !important}";
 
       if (styleElement.styleSheet) {
         styleElement.styleSheet.cssText = innerStyles;

Diff for: client/src/responsive-nav.js

@@ -45,7 +45,7 @@ describe("responsive-nav", function () {
       insertNav();
       var styleEl = document.getElementsByTagName("style")[0],
         styleContents = styleEl.innerHTML || styleEl.styleSheet.cssText.replace(/\s+/g, "").replace(/\;/g, "");
-      expect(styleContents.replace(/\.opened/g, "")).toBe(".nav-collapse{max-height:16px}");
+      expect(styleContents.replace(/\.opened/g, "")).toBe(".nav-collapse{max-height:16px !important}");
       nav.destroy();
     });
 
@@ -112,7 +112,7 @@ describe("responsive-nav", function () {
       insertNav();
       var styleEl = document.getElementsByTagName("style")[0],
         styleContents = styleEl.innerHTML || styleEl.styleSheet.cssText.replace(/\s+/g, "").replace(/\;/g, "");
-      expect(styleContents.replace(/\.opened/g, "")).toBe(".nav-collapse{max-height:50px}");
+      expect(styleContents.replace(/\.opened/g, "")).toBe(".nav-collapse{max-height:50px !important}");
       nav.destroy();
     });
 

Diff for: client/test/responsive-nav.spec.js

@@ -421,7 +421,7 @@
         for (var i = 0; i < nav.inner.length; i++) {
           savedHeight += nav.inner[i].offsetHeight;
         }
-        var innerStyles = "." + opts.navClass + ".opened{max-height:" + savedHeight + "px}";
+        var innerStyles = "." + opts.navClass + ".opened{max-height:" + savedHeight + "px !important}";
 
         if (styleElement.styleSheet) {
           styleElement.styleSheet.cssText = innerStyles;