Merge pull request #2674 from migupry:master

Greedy-nav fix

_sass/minimal-mistakes/_navigation.scss

@@ -184,6 +184,8 @@
     margin: 0 1rem;
     color: $masthead-link-color;
     text-decoration: none;
+    -webkit-transition: none;
+    transition: none;
 
     &:hover {
       color: $masthead-link-color-hover;
@@ -199,6 +201,11 @@
     }
   }
   
+  img{
+    -webkit-transition: none;
+    transition: none;
+  }
+
   &__toggle {
     -ms-flex-item-align: center;
     align-self: center;

assets/js/plugins/jquery.greedy-navigation.js

@@ -9,26 +9,69 @@ $(function() {
   var $btn = $("nav.greedy-nav .greedy-nav__toggle");
   var $vlinks = $("nav.greedy-nav .visible-links");
   var $hlinks = $("nav.greedy-nav .hidden-links");
+  var $nav = $("nav.greedy-nav");
+  var $logo = $('nav.greedy-nav .site-logo');
+  var $logoImg = $('nav.greedy-nav .site-logo img');
+  var $title = $("nav.greedy-nav .site-title");
+  var $search = $('nav.greedy-nav button.search__toggle');
   
-  var numOfItems = 0;
-  var totalSpace = 0;
-  var closingTime = 1000;
-  var breakWidths = [];
+  var numOfItems, totalSpace, closingTime, breakWidths;
 
-  // Get initial state
-  $vlinks.children().outerWidth(function(i, w) {
+  // This function measures both hidden and visible links and sets the navbar breakpoints
+  // This is called the first time the script runs and everytime the "check()" function detects a change of window width that reached a different CSS width breakpoint, which affects the size of navbar Items 
+  // Please note that "CSS width breakpoints" (which are only 4) !== "navbar breakpoints" (which are as many as the number of items on the navbar)
+  function measureLinks(){
+    numOfItems = 0;
+    totalSpace = 0;
+    closingTime = 1000;
+    breakWidths = [];
+
+    // Adds the width of a navItem in order to create breakpoints for the navbar
+    function addWidth(i, w) {
       totalSpace += w;
       numOfItems += 1;
       breakWidths.push(totalSpace);
-  });
+    }
+
+    // Measures the width of hidden links by making a temporary clone of them and positioning under visible links
+    function hiddenWidth(obj){
+      var clone = obj.clone();
+      clone.css("visibility","hidden");
+      $vlinks.append(clone);
+      addWidth(0, clone.outerWidth());
+      clone.remove();
+    }
+    // Measure both visible and hidden links widths
+    $vlinks.children().outerWidth(addWidth);
+    $hlinks.children().each(function(){hiddenWidth($(this))});
+  }
+  // Get initial state
+  measureLinks();
+
+  var winWidth = $( window ).width();
+  // Set the last measured CSS width breakpoint: 0: <768px, 1: <1024px, 2: < 1280px, 3: >= 1280px.
+  var lastBreakpoint = winWidth < 768 ? 0 : winWidth < 1024 ? 1 : winWidth < 1280 ? 2 : 3;
 
   var availableSpace, numOfVisibleItems, requiredSpace, timer;
 
   function check() {
 
+    winWidth = $( window ).width();
+    // Set the current CSS width breakpoint: 0: <768px, 1: <1024px, 2: < 1280px, 3: >= 1280px.
+    var curBreakpoint = winWidth < 768 ? 0 : winWidth < 1024 ? 1 : winWidth < 1280 ? 2 : 3;
+    // If current breakpoint is different from last measured breakpoint, measureLinks again
+    if(curBreakpoint !== lastBreakpoint) measureLinks();
+    // Set the last measured CSS width breakpoint with the current breakpoint
+    lastBreakpoint = curBreakpoint;
+
     // Get instant state
-    availableSpace = $vlinks.width() - 10;
     numOfVisibleItems = $vlinks.children().length;
+    // Decrease the width of visible elements from the nav innerWidth to find out the available space for navItems
+    availableSpace = /* nav */ $nav.innerWidth()
+                   - /* logo */ ($logo.length !== 0 ? $logo.outerWidth(true) : 0)
+                   - /* title */ $title.outerWidth(true)
+                   - /* search */ ($search.length !== 0 ? $search.outerWidth(true) : 0)
+                   - /* toggle */ (numOfVisibleItems !== breakWidths.length ? $btn.outerWidth(true) : 0);
     requiredSpace = breakWidths[numOfVisibleItems - 1];
 
     // There is not enought space
@@ -36,8 +79,8 @@ $(function() {
       $vlinks.children().last().prependTo($hlinks);
       numOfVisibleItems -= 1;
       check();
-      // There is more than enough space
-    } else if (availableSpace > breakWidths[numOfVisibleItems]) {
+      // There is more than enough space. If only one element is hidden, add the toggle width to the available space
+    } else if (availableSpace + (numOfVisibleItems === breakWidths.length - 1?$btn.outerWidth(true):0) > breakWidths[numOfVisibleItems]) {
       $hlinks.children().first().appendTo($vlinks);
       numOfVisibleItems += 1;
       check();
@@ -69,6 +112,15 @@ $(function() {
     clearTimeout(timer);
   })
 
-  check();
+  // check if page has a logo
+  if($logoImg.length !== 0){
+    // check if logo is not loaded
+    if(!($logoImg[0].complete || $logoImg[0].naturalWidth !== 0)){
+      // if logo is not loaded wait for logo to load or fail to check
+      $logoImg.one("load error", check);
+    // if logo is already loaded just check
+    } else check();
+  // if page does not have a logo just check
+  } else check();
   
 });