clean up settings

Author: andrewm4894

dashboard/components.py

@@ -16,92 +16,78 @@ def _create_controls(batch_name):
     """
     state = get_state()
     settings_dropdown = DropDownNavContainer(
-        NavHeaderLi("Chart Settings"),
+        NavHeaderLi("Settings"),
         Li(
             DivLAligned(
-                P("Small Charts", cls="text-sm font-medium"),
                 Button(
-                    UkIcon("check" if state.small_charts else "x"),
+                    P("Small Charts", cls="text-sm font-medium"),
                     hx_post=f"/batch/{batch_name}/toggle-size",
                     hx_target="#main-content",
                     cls=ButtonT.ghost,
                     uk_tooltip="Toggle between compact and full-size chart views",
                 ),
-                cls="flex items-center justify-between w-full px-4 py-2",
+                cls="flex items-center justify-between w-full px-2 py-2",
             )
         ),
         Li(
             DivLAligned(
-                P("Two Columns", cls="text-sm font-medium"),
                 Button(
-                    UkIcon("check" if state.two_columns else "x"),
+                    P("Two Columns", cls="text-sm font-medium"),
                     hx_post=f"/batch/{batch_name}/toggle-columns",
                     hx_target="#main-content",
                     cls=ButtonT.ghost,
                     uk_tooltip="Display charts in one or two columns",
                 ),
-                cls="flex items-center justify-between w-full px-4 py-2",
+                cls="flex items-center justify-between w-full px-2 py-2",
             )
         ),
         Li(
             DivLAligned(
-                P("Show Markers", cls="text-sm font-medium"),
                 Button(
-                    UkIcon("check" if state.show_markers else "x"),
+                    P("Show Markers", cls="text-sm font-medium"),
                     hx_post=f"/batch/{batch_name}/toggle-markers",
                     hx_target="#main-content",
                     cls=ButtonT.ghost,
                     uk_tooltip="Show/hide data point markers on the charts",
                 ),
-                cls="flex items-center justify-between w-full px-4 py-2",
+                cls="flex items-center justify-between w-full px-2 py-2",
             )
         ),
         Li(
             DivLAligned(
-                P("Show Legend", cls="text-sm font-medium"),
                 Button(
-                    UkIcon("check" if state.show_legend else "x"),
+                    P("Show Legend", cls="text-sm font-medium"),
                     hx_post=f"/batch/{batch_name}/toggle-legend",
                     hx_target="#main-content",
                     cls=ButtonT.ghost,
-                    uk_tooltip="Display chart legends with metric details",
+                    uk_tooltip="Display chart legends",
                 ),
-                cls="flex items-center justify-between w-full px-4 py-2",
+                cls="flex items-center justify-between w-full px-2 py-2",
             )
         ),
-        NavDividerLi(),
         Li(
             DivLAligned(
-                P("Dark Mode", cls="text-sm font-medium"),
                 Button(
-                    UkIcon("check" if state.dark_mode else "x"),
-                    hx_post=f"/batch/{batch_name}/toggle-theme",
+                    P("Narrow Lines", cls="text-sm font-medium"),
+                    hx_post=f"/batch/{batch_name}/toggle-line-width",
                     hx_target="#main-content",
                     cls=ButtonT.ghost,
-                    uk_tooltip="Switch between light and dark color themes",
+                    uk_tooltip="Toggle between narrow and normal line thickness",
                 ),
-                cls="flex items-center justify-between w-full px-4 py-2",
+                cls="flex items-center justify-between w-full px-2 py-2",
             )
         ),
+        NavDividerLi(),
         Li(
             DivLAligned(
-                P("Line Width", cls="text-sm font-medium"),
-                Input(
-                    type="number",
-                    name="line_width",
-                    value=state.line_width,
-                    min=1,
-                    max=10,
-                    step=1,
-                    cls="uk-input uk-form-small",
-                    style="width: 60px;",
-                    hx_post=f"/batch/{batch_name}/update-line-width",
-                    hx_target="#charts-container",
-                    hx_trigger="change",
-                    hx_swap="innerHTML",
-                    uk_tooltip="Adjust the thickness of chart lines (1-10)",
+                Button(
+                    P("Dark Mode", cls="text-sm font-medium"),
+                    hx_post=f"/batch/{batch_name}/toggle-theme",
+                    hx_target="#main-content",
+                    cls=ButtonT.ghost,
+                    uk_tooltip="Switch between light and dark color themes",
                 ),
-                cls="flex items-center justify-between w-full px-4 py-2",
+                cls="flex items-center justify-between w-full px-2 py-2",
             )
         ),
     )

dashboard/routes.py

@@ -458,25 +458,14 @@ def post(batch_name: str, session=None):
     )
 
 
-@rt("/batch/{batch_name}/update-line-width")
-def post(batch_name: str, line_width: int = 2):
+@rt("/batch/{batch_name}/toggle-line-width")
+def post(batch_name: str, session=None):
     """
-    Update the line width for a given batch name.
+    Toggle between narrow (1px) and normal (2px) line width.
     """
-    try:
-        # Ensure line width is within valid range
-        app.state.line_width = max(1, min(10, int(line_width)))
-        app.state.chart_cache.clear()  # Clear cache to regenerate charts
-
-        # Only regenerate the charts, maintaining the id and grid layout
-        metric_stats = app.state.stats_cache[batch_name]
-        return [
-            *[
-                ChartManager.create_chart_placeholder(
-                    stat["metric_name"], i, batch_name
-                )
-                for i, stat in enumerate(metric_stats)
-            ]
-        ]
-    except ValueError:
-        return "Invalid line width value"
+    app.state.narrow_lines = not getattr(app.state, 'narrow_lines', False)
+    app.state.line_width = 1 if app.state.narrow_lines else 2
+    app.state.chart_cache.clear()  # Clear cache to regenerate charts
+    return get_batch_view(
+        batch_name, session=session, initial_load=DEFAULT_LOAD_N_CHARTS
+    )