remove selector string option from Subscription constructor24281-issue-merge-request-sidebar-subscribe-button-style-improvement

1 files 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');