Refactor equal heights javascript function - dynamic items per row

[driesd]

Zie EANDIS:

$(window).load(function() {
var per_row = 4;
var $articles = $(".article");
var items = [];

      for (var i = 0; i <= $articles.length - 1; i += per_row) {
        items.push($articles.slice(i, (i + per_row)));
      }

      _.each(items, function(item) {
        if(item.length) {
          helper.equalheights(item, helper.highest(item));
        }
      });
    });

[rob-bar]

Deze werkt op offset:

equalHeights: {
      attach: function () {
        $(window).load(function() {
          if ($(window).width() > 480) {
            var selectors = [
              ".selector1",
              ".selector2",
            ];

            var $items = $(selectors.join(", "));

            var items = _.groupBy($items, function(element, index, list) {
              return list.selector + $(element).offset().top;
            });

            items = _.values(items);

            _.each(items, function(item) {
              if(item.length) {
                helper.equalheights($(item), helper.highest($(item)));
              }
            });
          }
        });
      }
    },

[rob-bar]

beter de equal heights calc per rij doen en dan op offset checken, en niet enkel in het begin op offset checken

equalHeights: {
      attach: function () {
        $(window).load(function() {
          if ($(window).width() > 480) {
            var selectors = [
              "selector1",
              "selector2",
            ];

            var filter = function(element, index, list) {
              return $(list[0]).offset().top === $(element).offset().top;
            };

            _.each(selectors, function(selector) {
              var $selector = $(selector);

              while($selector.length > 0) {
                var el = _.filter($selector, filter);

                helper.equalheights($(el), helper.highest($(el)));
                $selector = _.difference($selector, el);
              }
            });
          }
        });
      }
    },