Single quotes for consistency, cache 'this' selector.

15950477 · Connor Shea · c7702010 · 15950477
Commit 15950477 authored Aug 15, 2016 by Connor Shea
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 3 deletions

app/assets/javascripts/application.js app/assets/javascripts/application.js +4 -3

No files found.
--- a/app/assets/javascripts/application.js
+++ b/app/assets/javascripts/application.js
@@ -223,9 +223,10 @@
      return $('.navbar-toggle').toggleClass('active');
    });
    $body.on("click", ".js-toggle-diff-comments", function(e) {
-      $(this).toggleClass('active');
+      var $this = $(this);
-      var notesHolders = $(this).closest(".diff-file").find(".notes_holder");
+      $this.toggleClass('active');
-      if ($(this).hasClass('active')) {
+      var notesHolders = $this.closest('.diff-file').find('.notes_holder');
+      if ($this.hasClass('active')) {
        notesHolders.show();
      } else {
        notesHolders.hide();