Merge pull request #795 from aholachek/next-click

switch to currenttarget instead of target
adsabs · Feb 14, 2016 · 2171235 · 2171235
2 parents ec41a88 + eb9a3b0
commit 2171235
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/src/js/widgets/list_of_things/paginated_view.js b/src/js/widgets/list_of_things/paginated_view.js
@@ -199,13 +199,13 @@ define([
       changePageWithButton: function (e) {
 
         e.preventDefault();
-        var $target = $(e.target);
+        var $target = $(e.currentTarget);
         if ($target.parent().hasClass("disabled")) return;
         var transform = $target.hasClass("next-page") ? 1 : -1;
         var pageVal = this.model.get("page") + transform;
         this.trigger('pagination:select', pageVal);
 
-        if (this.resultsWidget) {analytics('send', 'event', 'interaction', 'results-list-pagination', pageVal); }
+        if (this.resultsWidget) {analytics('send', 'event', 'interaction', 'results-list-pagination', pageVal) }
       },
 
       tabOrEnterChangePageWithInput : function (e) {
@@ -215,6 +215,9 @@ define([
         if (e.keyCode == 13 || e.keyCode == 9){
           this.trigger('pagination:select', pageVal);
         }
+
+        if (this.resultsWidget) {analytics('send', 'event', 'interaction', 'results-list-pagination', pageVal) }
+
       },