fix poor 1000ms wait UX

KittyCAD · Dec 17, 2024 · e60cabb · e60cabb
1 parent 1e9cf6f
commit e60cabb
Showing 1 changed file with 3 additions and 6 deletions.
diff --git a/src/clientSideScene/sceneEntities.ts b/src/clientSideScene/sceneEntities.ts
@@ -1159,13 +1159,12 @@ export class SceneEntities {
 
         // Update the primary AST and unequip the rectangle tool
         await kclManager.executeAstMock(_ast)
+        sceneInfra.modelingSend({ type: 'Finish rectangle' })
 
         // lee: I had this at the bottom of the function, but it's
         // possible sketchFromKclValue "fails" when sketching on a face,
         // and this couldn't wouldn't run.
         await codeManager.updateEditorWithAstAndWriteToFile(_ast)
-
-        sceneInfra.modelingSend({ type: 'Finish rectangle' })
       },
     })
     return { updatedEntryNodePath, updatedSketchNodePaths }
@@ -1350,13 +1349,12 @@ export class SceneEntities {
 
           // Update the primary AST and unequip the rectangle tool
           await kclManager.executeAstMock(_ast)
+          sceneInfra.modelingSend({ type: 'Finish center rectangle' })
 
           // lee: I had this at the bottom of the function, but it's
           // possible sketchFromKclValue "fails" when sketching on a face,
           // and this couldn't wouldn't run.
           await codeManager.updateEditorWithAstAndWriteToFile(_ast)
-
-          sceneInfra.modelingSend({ type: 'Finish center rectangle' })
         }
       },
     })
@@ -1539,9 +1537,8 @@ export class SceneEntities {
 
           // Update the primary AST and unequip the rectangle tool
           await kclManager.executeAstMock(_ast)
-          await codeManager.updateEditorWithAstAndWriteToFile(_ast)
-
           sceneInfra.modelingSend({ type: 'Finish circle' })
+          await codeManager.updateEditorWithAstAndWriteToFile(_ast)
         }
       },
     })