From 1f6ec183154369f8a11d63713970cded0b1474fa Mon Sep 17 00:00:00 2001
From: Mike Greiling <mike@pixelcog.com>
Date: Thu, 1 Dec 2016 00:14:12 -0600
Subject: [PATCH] remove selector string option from Subscription constructor
---
app/assets/javascripts/subscription.js.es6 | 8 +++-----
1 file changed, 3 insertions(+), 5 deletions(-)
diff --git a/app/assets/javascripts/subscription.js.es6 b/app/assets/javascripts/subscription.js.es6
index 02be594135d..62d1604fe9e 100644
--- a/app/assets/javascripts/subscription.js.es6
+++ b/app/assets/javascripts/subscription.js.es6
@@ -2,12 +2,10 @@
(() => {
class Subscription {
- constructor(containerSelector) {
- this.containerElm = (typeof containerSelector === 'string')
- ? document.querySelector(containerSelector)
- : containerSelector;
+ constructor(containerElm) {
+ this.containerElm = containerElm;
- const subscribeButton = this.containerElm.querySelector('.js-subscribe-button');
+ const subscribeButton = containerElm.querySelector('.js-subscribe-button');
if (subscribeButton) {
// remove class so we don't bind twice
subscribeButton.classList.remove('js-subscribe-button');
--
GitLab