#7326 improve spark connection status widget

twosigma · May 24, 2018 · 629af48 · 629af48
1 parent 8b282b5
commit 629af48
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 1 deletion.
diff --git a/js/notebook/src/SparkUI.ts b/js/notebook/src/SparkUI.ts
@@ -33,8 +33,14 @@ class SparkUIModel extends widgets.VBoxModel {
 }
 
 class SparkUIView extends widgets.VBoxView {
+  private connectionStatus: HTMLElement;
+
   public render() {
     super.render();
+
+    this.el.classList.add('bx-spark-status-panel');
+
+    this.processConnectionWidget();
     this.updateLabels();
   }
 
@@ -69,6 +75,14 @@ class SparkUIView extends widgets.VBoxView {
       width: 'auto',
     }).outerWidth();
   }
+
+  private processConnectionWidget() {
+    this.children_views.update(this.model.get('children')).then((views) => {
+      views.forEach((view) => {
+        debugger;
+      });
+    });
+  }
 }
 
 export default {

diff --git a/kernel/sparkex/src/main/java/com/twosigma/beakerx/widget/SparkUIManager.java b/kernel/sparkex/src/main/java/com/twosigma/beakerx/widget/SparkUIManager.java
@@ -148,7 +148,7 @@ public void applicationEnd() {
   private HBox createStatusPanel() {
     Label appStatus = createAppStatus();
     Button disconnect = createDisconnectButton();
-    HBox statusPanel = new HBox(Arrays.asList(uiLink(), disconnect, appStatus));
+    HBox statusPanel = new HBox(Arrays.asList(uiLink(), appStatus, disconnect));
     sparkUI.add(statusPanel);
     return statusPanel;
   }