Merge branch 'main' into migrate-to-lsp-execution

src/components/ModelingSidebar/ModelingPane.tsx

@@ -1,5 +1,6 @@
 import { useStore } from 'useStore'
 import styles from './ModelingPane.module.css'
+import { useSettingsAuthContext } from 'hooks/useSettingsAuthContext'
 
 export interface ModelingPaneProps
   extends React.PropsWithChildren,
@@ -29,18 +30,21 @@ export const ModelingPane = ({
   detailsTestId,
   ...props
 }: ModelingPaneProps) => {
+  const { settings } = useSettingsAuthContext()
+  const onboardingStatus = settings.context.app.onboardingStatus
   const { buttonDownInStream } = useStore((s) => ({
     buttonDownInStream: s.buttonDownInStream,
   }))
+  const pointerEventsCssClass =
+    buttonDownInStream || onboardingStatus.current === 'camera'
+      ? 'pointer-events-none '
+      : 'pointer-events-auto '
   return (
     <section
       {...props}
       data-testid={detailsTestId}
       className={
-        (buttonDownInStream ? 'pointer-events-none ' : 'pointer-events-auto ') +
-        styles.panel +
-        ' group ' +
-        (className || '')
+        pointerEventsCssClass + styles.panel + ' group ' + (className || '')
       }
     >
       <ModelingPaneHeader title={title} Menu={Menu} />

src/components/ModelingSidebar/ModelingSidebar.tsx

@@ -15,19 +15,18 @@ interface ModelingSidebarProps {
 }
 
 export function ModelingSidebar({ paneOpacity }: ModelingSidebarProps) {
-  const { buttonDownInStream, openPanes } = useStore((s) => ({
+  const { settings } = useSettingsAuthContext()
+  const onboardingStatus = settings.context.app.onboardingStatus
+  const { openPanes, buttonDownInStream } = useStore((s) => ({
     buttonDownInStream: s.buttonDownInStream,
     openPanes: s.openPanes,
   }))
-  const { settings } = useSettingsAuthContext()
-  const {
-    app: { onboardingStatus },
-  } = settings.context
-
   const pointerEventsCssClass =
-    buttonDownInStream || onboardingStatus.current === 'camera'
+    buttonDownInStream ||
+    onboardingStatus.current === 'camera' ||
+    openPanes.length === 0
       ? 'pointer-events-none '
-      : 'pointer-events-auto'
+      : 'pointer-events-auto '
 
   return (
     <Resizable
@@ -41,8 +40,7 @@ export function ModelingSidebar({ paneOpacity }: ModelingSidebarProps) {
       handleClasses={{
         right:
           (openPanes.length === 0 ? 'hidden ' : 'block ') +
-          'translate-x-1/2 hover:bg-chalkboard-10 hover:dark:bg-chalkboard-110 bg-transparent transition-colors duration-75 transition-ease-out delay-100 ' +
-          pointerEventsCssClass,
+          'translate-x-1/2 hover:bg-chalkboard-10 hover:dark:bg-chalkboard-110 bg-transparent transition-colors duration-75 transition-ease-out delay-100 ',
       }}
     >
       <div className={styles.grid + ' flex-1'}>
@@ -118,6 +116,7 @@ function ModelingSidebarSection({
     >
       <Tab.List
         className={
+          'pointer-events-auto ' +
           (alignButtons === 'start'
             ? 'justify-start self-start'
             : 'justify-end self-end') +

src/index.css

@@ -135,9 +135,13 @@ code {
   overflow: auto;
 }
 
+#code-mirror-override .cm-gutter {
+  @apply select-none;
+}
+
 #code-mirror-override .cm-activeLine,
 #code-mirror-override .cm-activeLineGutter {
-  @apply bg-primary/20;
+  @apply bg-primary/10;
 }
 
 .dark #code-mirror-override .cm-activeLine,

src/wasm-lib/tests/executor/main.rs

@@ -1737,10 +1737,13 @@ const sketch001 = startSketchOn(box, "revolveAxis")
 
 "#;
 
-    let result = execute_and_snapshot(code, kittycad::types::UnitLength::Mm)
-        .await
-        .unwrap();
-    twenty_twenty::assert_image("tests/executor/outputs/revolve_on_edge_get_edge.png", &result, 1.0);
+    let result = execute_and_snapshot(code, kittycad::types::UnitLength::Mm).await;
+
+    assert!(result.is_err());
+    assert_eq!(
+        result.err().unwrap().to_string(),
+        r#"engine: KclErrorDetails { source_ranges: [SourceRange([349, 409])], message: "Modeling command failed: Some([ApiError { error_code: InternalEngine, message: \"Solid3D revolve failed:  sketch profile must lie entirely on one side of the revolution axis\" }])" }"#
+    );
 }
 
 #[tokio::test(flavor = "multi_thread")]