Default initialFocus to "null" to avoid breaking existing modals

kraenhansen · kraenhansen · commit 602106a798b5 · 2025-11-12T22:17:56.000+01:00
diff --git a/packages/compass-components/src/components/modals/modal.tsx b/packages/compass-components/src/components/modals/modal.tsx
@@ -39,6 +39,8 @@ function UnwrappedModal({
       className={cx(styles, className, {
         [fullScreenStyles]: fullScreen,
       })}
+      /* For now, we're defaulting to not auto-focus the first focusable element */
+      initialFocus={null}
       {...props}
     >
       <Body as="div">{children}</Body>

Original file line number	Diff line number	Diff line change
`@@ -39,6 +39,8 @@ function UnwrappedModal({`
`39`	`39`	`className={cx(styles, className, {`
`40`	`40`	`[fullScreenStyles]: fullScreen,`
`41`	`41`	`})}`
	`42`	`+ /* For now, we're defaulting to not auto-focus the first focusable element */`
	`43`	`+ initialFocus={null}`
`42`	`44`	`{...props}`
`43`	`45`	`>`
`44`	`46`	`<Body as="div">{children}</Body>`