remove fillet button in production build

KittyCAD · Jul 15, 2024 · 59b2f68 · 59b2f68
1 parent daf2b80
commit 59b2f68
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 7 deletions.
diff --git a/src/Toolbar.tsx b/src/Toolbar.tsx
@@ -16,6 +16,7 @@ import {
   canRectangleTool,
   isEditingExistingSketch,
 } from 'machines/modelingMachine'
+import { DEV } from 'env'
 
 export function Toolbar({
   className = '',
@@ -118,14 +119,15 @@ export function Toolbar({
       }),
     { enabled: !disableAllButtons, scopes: ['modeling'] }
   )
+  const disableFillet = !state.can('Fillet') || disableAllButtons
   useHotkeys(
     'f',
     () =>
       commandBarSend({
         type: 'Find and select command',
         data: { name: 'Fillet', groupId: 'modeling' },
       }),
-    { enabled: !disableAllButtons, scopes: ['modeling'] }
+    { enabled: !disableFillet, scopes: ['modeling'] }
   )
 
   function handleToolbarButtonsWheelEvent(ev: WheelEvent<HTMLSpanElement>) {
@@ -413,7 +415,7 @@ export function Toolbar({
             </ActionButton>
           </li>
         )}
-        {state.matches('idle') && (
+        {state.matches('idle') && DEV && (
           <li className="contents">
             <ActionButton
               className={buttonClassName}
@@ -424,10 +426,8 @@ export function Toolbar({
                   data: { name: 'Fillet', groupId: 'modeling' },
                 })
               }
-              disabled={!state.can('Fillet') || disableAllButtons}
-              title={
-                state.can('Fillet') ? 'fillet' : 'sketches need to be closed'
-              }
+              disabled={disableFillet}
+              title={disableFillet ? 'fillet' : "edge can't be filleted"}
               iconStart={{
                 icon: 'fillet', // todo: add fillet icon
                 iconClassName,

diff --git a/src/lang/std/sketch.ts b/src/lang/std/sketch.ts
@@ -314,7 +314,6 @@ function getTag(index = 2): SketchLineHelper['getTag'] {
       return new Error('Not a CallExpression')
     const arg = callExp.arguments?.[index]
     if (!arg) return new Error('No argument')
-    console.log('arg', arg)
     if (arg.type !== 'TagDeclarator')
       return new Error('Tag not a TagDeclarator')
     return arg.value