Merge pull request #403 from CesiumGS/enableDisableButtons

visually enable/disable top bar buttons based on sign-in status
CesiumGS · Jul 28, 2023 · 92370d8 · 92370d8
2 parents 1517bcd + c43385f
commit 92370d8
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 3 deletions.
diff --git a/exts/cesium.omniverse/cesium/omniverse/ui/main_window.py b/exts/cesium.omniverse/cesium/omniverse/ui/main_window.py
@@ -140,9 +140,8 @@ def _on_update_frame(self, _e: carb.events.IEvent):
             # Since this goes across the pybind barrier, just grab it once.
             is_connected = session.is_connected()
             self._sign_in_widget.visible = not is_connected
-            self._add_button.enabled = is_connected
-            self._upload_button.enabled = is_connected
-            self._sign_out_button.enabled = is_connected
+
+            self._set_top_bar_button_status(is_connected)
 
     def _on_assets_updated(self, _e: carb.events.IEvent):
         self._logger.info("Received ion Assets updated event.")
@@ -180,6 +179,11 @@ def _on_show_troubleshooter_window(self, _e: carb.events.IEvent):
             message,
         )
 
+    def _set_top_bar_button_status(self, enabled: bool):
+        self._add_button.enabled = enabled
+        self._upload_button.enabled = enabled
+        self._sign_out_button.enabled = enabled
+
     def _build_fn(self):
         """Builds all UI components."""
 
@@ -222,6 +226,7 @@ def _build_fn(self):
                 self._sign_out_button = ui.Button(
                     "Sign Out",
                     image_url=f"{self._icon_path}/FontAwesome/sign-out-alt-solid.png",
+                    # style=button_style,
                     style=button_style,
                     clicked_fn=self._sign_out_button_clicked,
                     enabled=False,
@@ -270,6 +275,8 @@ def _sign_out_button_clicked(self) -> None:
         if session is not None:
             session.disconnect()
 
+        self._set_top_bar_button_status(False)
+
     def _show_token_window(self):
         self._cesium_omniverse_interface.get_session().refresh_tokens()
         CesiumOmniverseTokenWindow(self._cesium_omniverse_interface)
diff --git a/exts/cesium.omniverse/cesium/omniverse/ui/styles.py b/exts/cesium.omniverse/cesium/omniverse/ui/styles.py
@@ -42,6 +42,8 @@ class CesiumOmniverseUiStyles:
             "alignment": Alignment.CENTER,
         },
         "Button.Label": {"alignment": Alignment.CENTER_BOTTOM},
+        "Button.Image:disabled": {"color": cl("#808080")},
+        "Button.Label:disabled": {"color": cl("#808080")},
     }
 
     asset_detail_frame = {"ScrollingFrame": {"background_color": cl("#1F2123"), "padding": 10}}