refactor: reorganize UI components and clean up unused code

  - Extract ControlPanel to pan3d.ui.control_panel module
  - Extract RenderingSettingsBasic to pan3d.ui.rendering_settings module
  - Create SummaryToolbar widget from common.py code
  - Remove duplicate imports and unused properties
  - Add defauly ctx_name property to widgets
  - Remove vtk_view file and Pan3DView is moved to widgets

Author: ayenpure

src/pan3d/custom/contour.py

@@ -25,9 +25,9 @@
 )
 
 from pan3d.ui.css import base, preview
-from pan3d.ui.vtk_view import Pan3DView
 from pan3d.utils.convert import to_float, to_image
 from pan3d.utils.presets import PRESETS, set_preset
+from pan3d.widgets.pan3d_view import Pan3DView
 from trame.app import get_server
 from trame.decorators import TrameApp, change
 from trame.ui.vuetify3 import VAppLayout

src/pan3d/explorers/analytics.py

@@ -23,10 +23,11 @@
     plot_options,
     zonal_axes,
 )
+from pan3d.ui.layouts import StandardExplorerLayout
 from pan3d.ui.preview import RenderingSettings
-from pan3d.ui.vtk_view import Pan3DView
 from pan3d.utils.common import Explorer
 from pan3d.utils.convert import to_float
+from pan3d.widgets.pan3d_view import Pan3DView
 from pan3d.xarray.algorithm import vtkXArrayRectilinearSource
 from trame.decorators import change
 from trame.widgets import html
@@ -371,22 +372,23 @@ def _build_ui(self, **kwargs):
         self.state.setdefault("time_groups", 0)
         self.state.setdefault("figure_height", 50)
 
-        # Use the standard UI creation method
-        layout = self._create_standard_ui(
-            panel_label="Analytics Explorer",
-            view_class=Pan3dAnalyticsView,
-            rendering_settings_class=RenderingSettings,
-            view_kwargs={
-                "render_window": self.render_window,
-                "local_rendering": self.local_rendering,
-                "widgets": [self.widget],
-            },
-            save_path_default="",
-            error_style="position:absolute;bottom:1rem;right:1rem;",
-        )
+        ## New way to build UI
+        with StandardExplorerLayout(
+            explorer=self, title="Analytics Explorer"
+        ) as self.ui:
+            with self.ui.canvas:
+                Pan3dAnalyticsView(
+                    render_window=self.render_window,
+                    local_rendering=self.local_rendering,
+                    widget=[self.widget],
+                )
+            with self.ui.control_panel:
+                RenderingSettings(
+                    ctx_name="rendering",
+                    source=self.source,
+                    update_rendering=self.update_rendering,
+                )
 
-        # Add navigation drawer within the VAppLayout
-        with layout:
             with v3.VNavigationDrawer(
                 disable_resize_watcher=True,
                 disable_route_watcher=True,
@@ -397,7 +399,13 @@ def _build_ui(self, **kwargs):
             ):
                 self.plotting = Plotting(source=self.source, toggle="chart_expanded")
 
-        return layout
+        self.ctx.save_dialog.save_callback = self._save_dataset
+        if (
+            self.source
+            and hasattr(self.source, "input")
+            and self.source.input is not None
+        ):
+            self.ctx.rendering.update_from_source(self.source)
 
     # -----------------------------------------------------
     # State change callbacks

src/pan3d/explorers/contour.py

@@ -24,9 +24,10 @@
 )
 
 from pan3d.ui.contour import ContourRenderingSettings
-from pan3d.ui.vtk_view import Pan3DView
+from pan3d.ui.layouts import StandardExplorerLayout
 from pan3d.utils.common import Explorer
 from pan3d.utils.convert import to_float
+from pan3d.widgets.pan3d_view import Pan3DView
 from pan3d.xarray.algorithm import vtkXArrayRectilinearSource
 from trame.decorators import change
 
@@ -42,10 +43,6 @@ def __init__(
                 input=self.xarray
             )  # To initialize the pipeline
 
-        # setup
-        self.last_field = None
-        self.last_preset = None
-
         self._setup_vtk(pipeline)
         self._build_ui()
 
@@ -131,18 +128,21 @@ def _build_ui(self, **_):
                 "scale_z": 0.01,
             }
         )
+
         # Use the standard UI creation method
-        return self._create_standard_ui(
-            panel_label="Contour Explorer",
-            view_class=Pan3DView,
-            rendering_settings_class=ContourRenderingSettings,
-            view_kwargs={
-                "render_window": self.render_window,
-                "local_rendering": self.local_rendering,
-                "widgets": [self.widget],
-            },
-            save_path_default="output.nc",
-        )
+        with StandardExplorerLayout(explorer=self, title="Contour Explorer") as self.ui:
+            with self.ui.canvas:
+                Pan3DView(
+                    render_window=self.render_window,
+                    local_rendering=self.local_rendering,
+                    widget=[self.widget],
+                )
+            with self.ui.control_panel:
+                ContourRenderingSettings(
+                    ctx_name="rendering",
+                    source=self.source,
+                    update_rendering=self.update_rendering,
+                )
 
     def update_rendering(self, reset_camera=False):
         self.state.dirty_data = False

src/pan3d/explorers/globe.py

@@ -19,9 +19,10 @@
 
 from pan3d.filters.globe import ProjectToSphere
 from pan3d.ui.globe import GlobeRenderingSettings
-from pan3d.ui.vtk_view import Pan3DView
+from pan3d.ui.layouts import StandardExplorerLayout
 from pan3d.utils.common import Explorer
 from pan3d.utils.globe import get_continent_outlines, get_globe, get_globe_textures
+from pan3d.widgets.pan3d_view import Pan3DView
 from pan3d.xarray.algorithm import vtkXArrayRectilinearSource
 from trame.decorators import change
 
@@ -118,20 +119,19 @@ def _build_ui(self, **kwargs):
             }
         )
         # Use the standard UI creation method
-        return self._create_standard_ui(
-            panel_label="Globe Explorer",
-            view_class=Pan3DView,
-            rendering_settings_class=GlobeRenderingSettings,
-            view_kwargs={
-                "render_window": self.render_window,
-                "local_rendering": self.local_rendering,
-                "widgets": [self.widget],
-                "disable_style_toggle": True,
-                "disable_roll": True,
-                "disable_axis_align": True,
-            },
-            error_max_width=700,
-        )
+        with StandardExplorerLayout(explorer=self, title="Globe Explorer") as self.ui:
+            with self.ui.canvas:
+                Pan3DView(
+                    render_window=self.render_window,
+                    local_rendering=self.local_rendering,
+                    widget=[self.widget],
+                )
+            with self.ui.control_panel:
+                GlobeRenderingSettings(
+                    ctx_name="rendering",
+                    source=self.source,
+                    update_rendering=self.update_rendering,
+                )
 
     # -----------------------------------------------------
     # State change callbacks

src/pan3d/explorers/slicer.py

@@ -24,9 +24,10 @@
     vtkRenderWindowInteractor,
 )
 
+from pan3d.ui.layouts import StandardExplorerLayout
 from pan3d.ui.slicer import SliceRenderingSettings
-from pan3d.ui.vtk_view import Pan3DView
 from pan3d.utils.common import Explorer
+from pan3d.widgets.pan3d_view import Pan3DView
 from pan3d.xarray.algorithm import vtkXArrayRectilinearSource
 from trame.decorators import change
 from trame.widgets import html
@@ -265,25 +266,24 @@ def _build_ui(self, *args, **kwargs):
             }
         )
 
-        # Add SliceSummary as an additional component
-        def add_slice_summary():
-            SliceSummary(
-                v_show="!control_expended",
-                style="position: absolute; right: 1rem; top: 50%; transform: translateY(-50%); z-index: 2;",
-            )
-
         # Use the standard UI creation method
-        return self._create_standard_ui(
-            panel_label="Slice Explorer",
-            view_class=Pan3DSlicerView,
-            rendering_settings_class=SliceRenderingSettings,
-            view_kwargs={
-                "render_window": self.render_window,
-                "local_rendering": self.local_rendering,
-                "widgets": [self.widget],
-            },
-            additional_components=add_slice_summary,
-        )
+        with StandardExplorerLayout(explorer=self, title="Slice Explorer") as self.ui:
+            with self.ui.canvas:
+                Pan3DSlicerView(
+                    render_window=self.render_window,
+                    local_rendering=self.local_rendering,
+                    widget=[self.widget],
+                )
+                SliceSummary(
+                    v_show="!control_expended",
+                    style="position: absolute; right: 1rem; top: 50%; transform: translateY(-50%); z-index: 2;",
+                )
+            with self.ui.control_panel:
+                SliceRenderingSettings(
+                    ctx_name="rendering",
+                    source=self.source,
+                    update_rendering=self.update_rendering,
+                )
 
     def update_rendering(self, reset_camera=False):
         self.state.dirty_data = False
@@ -320,27 +320,6 @@ def slice_axis(self, axis: str) -> None:
         with self.state:
             self.state.slice_axis = axis
 
-    @property
-    def slice_value(self):
-        """
-        Returns the value(origin) for the dimension along which the slice
-        is performed
-        """
-        s = self.state
-        axis = "xyz"[s.slice_axes.index(s.slice_axis)]
-        return s[f"cut_{axis}"]
-
-    @slice_value.setter
-    def slice_value(self, value: float) -> None:
-        """
-        Sets the value(origin) for the dimension along which the slice
-        is performed
-        """
-        with self.state:
-            s = self.state
-            axis = "xyz"[s.slice_axes.index(s.slice_axis)]
-            s[f"cut_{axis}"] = value
-
     @property
     def view_mode(self):
         """
@@ -357,22 +336,6 @@ def view_mode(self, mode):
         with self.state:
             self.state.view_mode = mode
 
-    @property
-    def scale_axis(self):
-        s = self.state
-        return [s.x_scale, s.y_scale, s.z_scale]
-
-    @scale_axis.setter
-    def scale_axis(self, sfac):
-        s = self.state
-        s.x_scale = float(sfac[0])
-        s.y_scale = float(sfac[1])
-        s.z_scale = float(sfac[2])
-        self.slice_actor.SetScale(*sfac)
-        self.data_actor.SetScale(*sfac)
-        self.outline_actor.SetScale(*sfac)
-        self.on_view_mode_change(s.view_mode)
-
     # -------------------------------------------------------------------------
     # UI triggers
     # -------------------------------------------------------------------------

src/pan3d/ui/contour.py

@@ -1,4 +1,4 @@
-from pan3d.utils.common import RenderingSettingsBasic
+from pan3d.ui.rendering_settings import RenderingSettingsBasic
 from pan3d.widgets.time_navigation import TimeNavigation
 from pan3d.widgets.vector_property_control import VectorPropertyControl
 from trame.widgets import html
@@ -85,6 +85,5 @@ def update_from_source(self, source=None):
             state.slice_extents = source.slice_extents
 
             # Update TimeNavigation widget through context
-            if hasattr(self.ctx, "time_nav"):
-                self.ctx.time_nav.labels = source.t_labels
-                self.ctx.time_nav.index = source.t_index
+            self.ctx.time_nav.labels = source.t_labels
+            self.ctx.time_nav.index = source.t_index