From 117f2a54cd6dbfd5f75765131a59d8f4bb7e7f37 Mon Sep 17 00:00:00 2001
From: Kyle Conroy <kyleconroy@gmail.com>
Date: Fri, 8 Mar 2024 12:35:44 -0500
Subject: [PATCH] support deleting all viewers without crashing app

---
 jdaviz/configs/default/plugins/plot_options/plot_options.vue | 4 ++--
 jdaviz/core/template_mixin.py                                | 4 ++++
 2 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/jdaviz/configs/default/plugins/plot_options/plot_options.vue b/jdaviz/configs/default/plugins/plot_options/plot_options.vue
index 0ff83416ac..cb601b9623 100644
--- a/jdaviz/configs/default/plugins/plot_options/plot_options.vue
+++ b/jdaviz/configs/default/plugins/plot_options/plot_options.vue
@@ -39,7 +39,7 @@
       :hint="viewer_multiselect ? 'Select viewers to set options simultaneously' : 'Select the viewer to set options.'"
     />
 
-    <v-row>
+    <v-row v-if="viewer_selected.length > 0">
       <v-expansion-panels accordion>
         <v-expansion-panel>
           <v-expansion-panel-header v-slot="{ open }">
@@ -159,7 +159,7 @@
     </div>
 
     <!-- GENERAL:AXES -->
-    <glue-state-sync-wrapper v-if="axes_visible_sync.in_subscribed_states && config !== 'imviz'" :sync="axes_visible_sync" :multiselect="viewer_multiselect" @unmix-state="unmix_state('axes_visible')">
+    <glue-state-sync-wrapper v-if="axes_visible_sync.in_subscribed_states && viewer_selected.length > 0 && config !== 'imviz'" :sync="axes_visible_sync" :multiselect="viewer_multiselect" @unmix-state="unmix_state('axes_visible')">
       <v-switch
         v-model="axes_visible_value"
         label="Show Axes"
diff --git a/jdaviz/core/template_mixin.py b/jdaviz/core/template_mixin.py
index 7a99f2785c..259641df23 100644
--- a/jdaviz/core/template_mixin.py
+++ b/jdaviz/core/template_mixin.py
@@ -2680,6 +2680,8 @@ def _get_selected_item(self, selected):
 
     @property
     def selected_id(self):
+        if self.selected_item is None:
+            return None
         return self.selected_item.get('id', None)
 
     @property
@@ -3582,6 +3584,8 @@ def _state_item(item):
     @cached_property
     def subscribed_icons(self):
         # dictionary items giving information about the entries in subscribed_states
+        if self._viewer_select.selected_item is None:
+            return []
         viewer_icons = self._viewer_select.selected_item.get('icon', [])
         if not isinstance(viewer_icons, list):
             viewer_icons = [viewer_icons]