From 949e5b77bbcac5bf9a14c83bf1ed807fe1cc9746 Mon Sep 17 00:00:00 2001 From: mitchellhansen Date: Sun, 26 Apr 2026 21:57:20 -0700 Subject: [PATCH] remove Hulls view tab and its supporting dead code - VIEW_MODES: remove 'hulls' (gradient hull viz is now in the Hull node thumbnail) - App.jsx: remove case 'hulls': from the refresh switch; remove 'hulls' entry from the accent color map - lib.rs: remove the 'hulls' arm from get_pass_viz (per-pixel raster gradient render); update doc comment to reflect contours-only mode Co-Authored-By: Claude Sonnet 4.6 --- src-frontend/src/App.jsx | 5 ++--- src/lib.rs | 24 +----------------------- 2 files changed, 3 insertions(+), 26 deletions(-) diff --git a/src-frontend/src/App.jsx b/src-frontend/src/App.jsx index 5dce8c15..1ecc9354 100644 --- a/src-frontend/src/App.jsx +++ b/src-frontend/src/App.jsx @@ -8,7 +8,7 @@ import { defaultPass, defaultGcodeConfig, PAPER_SIZES } from './store.js' import * as tauri from './hooks/useTauri.js' import { useFps } from './hooks/useFps.js' -const VIEW_MODES = ['source', 'detection', 'hulls', 'contours', 'fill', 'gcode'] +const VIEW_MODES = ['source', 'detection', 'contours', 'fill', 'gcode'] export default function App() { const [image, setImage] = useState(null) @@ -86,7 +86,6 @@ export default function App() { case 'detection': setDisplayB64(passes[activePass]?.vizB64 ?? null) break - case 'hulls': case 'contours': // Don't race getPassViz against generateFill — both need the AppState mutex. // filling=true means fill hasn't finished yet; the effect will re-run when it does. @@ -372,7 +371,7 @@ export default function App() { {/* Top bar — accent colors match the section dots in the left panel */}
{VIEW_MODES.map(m => { - const accent = { detection: '#6366f1', hulls: '#14b8a6', contours: '#14b8a6', fill: '#a855f7', gcode: '#f59e0b' }[m] + const accent = { detection: '#6366f1', contours: '#14b8a6', fill: '#a855f7', gcode: '#f59e0b' }[m] const label = m === 'gcode' ? 'G-code' : m.charAt(0).toUpperCase() + m.slice(1) return (