Move javascript that was sitting in application.css into separate files

judytuna · Dec 26, 2014 · e9cd1db · e9cd1db
1 parent da4e665
commit e9cd1db
Show file tree

Hide file tree

Showing 3 changed files with 28 additions and 24 deletions.
diff --git a/app/assets/javascripts/application.js b/app/assets/javascripts/application.js
@@ -33,27 +33,3 @@
 //= require dataTables/bootstrap/3/jquery.dataTables.bootstrap
 //= require_tree .
 //= require jquery_nested_form
-
-$(document).ready(function () {
-  if ($(window).height() < $('html').height()) {
-    $('footer').show();
-  }
-
-  function matchHeights ($element) {
-    var maxHeight;
-    var items = $element.find('.js-match-height-item');
-    items.css('min-height', 0);
-    maxHeight = Math.max.apply(Math, _.map(items, function(item) {
-      return $(item).outerHeight();
-    }));
-    items.css({ 'min-height': maxHeight });
-  }
-
-  $(window).resize(function() {
-    if ($(document).width() > 768) {
-      matchHeights($('.upcoming-events'));
-    }
-  });
-
-  matchHeights($('.upcoming-events'));
-});
diff --git a/app/assets/javascripts/services/footer.js b/app/assets/javascripts/services/footer.js
@@ -0,0 +1,5 @@
+$(document).ready(function () {
+  if ($(window).height() < $('html').height()) {
+    $('footer').show();
+  }
+});
diff --git a/app/assets/javascripts/services/matchHeights.js b/app/assets/javascripts/services/matchHeights.js
@@ -0,0 +1,23 @@
+$(document).ready(function () {
+  function matchHeights ($element) {
+    var items = $element.find('.js-match-height-item');
+    items.css('min-height', 0);
+
+    var maxHeight = Math.max.apply(Math, _.map(items, function(item) {
+      return $(item).outerHeight();
+    }));
+    items.css('min-height', maxHeight);
+  }
+
+  function setupHeightMatching ($element) {
+    $(window).resize(_.debounce(function() {
+      if ($(document).width() > 768) {
+        matchHeights($element);
+      }
+    }, 100));
+
+    matchHeights($element);
+  }
+
+  setupHeightMatching($('body'));
+});