Use `toggleClass` to manage classes

fff2100f · kushalpandya · f432d71e · fff2100f
Commit fff2100f authored Jul 06, 2017 by kushalpandya
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 7 deletions

app/assets/javascripts/ee_trial_banner/ee_trial_banner.js app/assets/javascripts/ee_trial_banner/ee_trial_banner.js +2 -7

No files found.
--- a/app/assets/javascripts/ee_trial_banner/ee_trial_banner.js
+++ b/app/assets/javascripts/ee_trial_banner/ee_trial_banner.js
@@ -73,13 +73,8 @@ export default class EETrialBanner {
  }

  toggleBanner(state) {
-    if (state) {
-      this.$trialBanner.removeClass('hidden');
-      this.$navbar.addClass('has-trial-banner');
-    } else {
-      this.$trialBanner.addClass('hidden');
-      this.$navbar.removeClass('has-trial-banner');
-    }
+    this.$trialBanner.toggleClass('hidden', !state);
+    this.$navbar.toggleClass('has-trial-banner', state);
  }

  handleTrialBannerDismiss() {