Fix aspect ratio slider: invert direction, allow shorter views

Author: olearypatrick

src/e3sm_quickview/components/toolbars.py

@@ -47,11 +47,11 @@ def __init__(self, apply_size=None):
             v3.VSpacer()
 
             v3.VSlider(
-                v_model=("aspect_ratio", 2),
-                prepend_icon="mdi-arrow-expand-horizontal",
-                min=1,
+                v_model=("aspect_ratio", 0.5),
+                prepend_icon="mdi-arrow-expand-vertical",
+                min=0.25,
                 max=2,
-                step=0.1,
+                step=0.05,
                 density="compact",
                 hide_details=True,
                 style="max-width: 400px;",

src/e3sm_quickview/view_manager.py

@@ -956,7 +956,7 @@ def _build_ui(self):
                     style=(
                         """
                         {
-                            aspectRatio: active_layout === 'auto_layout' ? aspect_ratio : null,
+                            aspectRatio: active_layout === 'auto_layout' ? (1.0 / aspect_ratio) : null,
                             height: active_layout !== 'auto_layout' ? 'calc(100% - 2.4rem)' : null,
                             pointerEvents: lock_views ? 'none': null,
                         }

src/e3sm_quickview/view_manager2.py

@@ -955,7 +955,7 @@ def _build_ui(self):
                     style=(
                         """
                         {
-                            aspectRatio: active_layout === 'auto_layout' ? aspect_ratio : null,
+                            aspectRatio: active_layout === 'auto_layout' ? (1.0 / aspect_ratio) : null,
                             height: active_layout !== 'auto_layout' ? 'calc(100% - 2.4rem)' : null,
                             pointerEvents: lock_views ? 'none': null,
                         }