Remove button instead of hiding it

yue · Apr 25, 2018 · 9915026 · 9915026
1 parent 6e8bb4d
commit 9915026
Showing 1 changed file with 10 additions and 4 deletions.
diff --git a/lib/service/slack/index.js b/lib/service/slack/index.js
@@ -30,7 +30,10 @@ class SlackService extends Service {
   }
 
   loginWithToken(token, button) {
-    button.setTitle("Loading...")
+    if (button)
+      button.setTitle("Loading...")
+    else
+      this.loginWindow.setContentView(gui.Label.create('Loading...'))
     const rtm = new RTMClient(token)
     require('./private-apis').extend(rtm)
     rtm.once('unable_to_rtm_start', (error) => {
@@ -39,9 +42,12 @@ class SlackService extends Service {
     })
     rtm.once('authenticated', (data) => {
       new SlackAccount(this, data, rtm)
-      button.setVisible(false)
-      this.adujstLoginWindowSize()
-
+      if (button) {
+        this.loginWindow.getContentView().removeChildView(button)
+        this.adujstLoginWindowSize()
+      } else {
+        this.loginWindow.close()
+      }
     })
     rtm.start({batch_presence_aware: true})
   }