Bug: ASM/CPU titles whould disappear after re-clicking header button (#…

…293)
nand2tetris · May 8, 2024 · 04baa49 · 04baa49
1 parent 98fb9f9
commit 04baa49
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 3 deletions.
diff --git a/web/src/pages/asm.tsx b/web/src/pages/asm.tsx
@@ -29,12 +29,15 @@ export const Asm = () => {
 
   const [showSymbolTable, setShowSymbolTable] = useState(true);
 
+  useEffect(() => {
+    if (state?.path) {
+      setTitle(state.path.split("/").pop() ?? "");
+    }
+  });
+
   useEffect(() => {
     if (toolStates.asmState) {
       actions.overrideState(toolStates.asmState);
-      if (toolStates.asmState.path) {
-        setTitle(toolStates.asmState.path.split("/").pop() ?? "");
-      }
     }
   }, []);
 

diff --git a/web/src/pages/cpu.tsx b/web/src/pages/cpu.tsx
@@ -43,6 +43,10 @@ export const CPU = () => {
 
   useEffect(() => {
     toolStates.setCpuState(fileName, state.sim.ROM, romFormat);
+    if (fileName) {
+      const name = fileName.split("/").pop() ?? "";
+      setTitle(name);
+    }
   });
 
   useEffect(() => {