Fix Dialog background scroll

.changeset/three-years-visit.md

@@ -0,0 +1,5 @@
+---
+"@salt-ds/core": minor
+---
+
+Added `disableScroll` prop to FloatingComponent.

packages/core/src/__tests__/__e2e__/utils/useFloatingUI.cy.tsx

@@ -11,11 +11,13 @@ const TestComponent = ({
   contentId = "test-1-content",
   focusManager,
   open = true,
+  disableScroll = false,
 }: {
   id?: string;
   contentId?: string;
   focusManager?: boolean;
   open?: boolean;
+  disableScroll?: boolean;
 }) => {
   const { Component: FloatingComponent } = useFloatingComponent();
   const { context } = useFloatingUI({
@@ -26,6 +28,7 @@ const TestComponent = ({
       <FloatingComponent
         open={Boolean(open)}
         focusManagerProps={focusManager ? { context } : undefined}
+        disableScroll={disableScroll}
       >
         <div id={contentId} />
       </FloatingComponent>
@@ -84,4 +87,28 @@ describe("Use useFloatingComponent", () => {
       );
     });
   });
+  describe("without disableScroll", () => {
+    it("the document body should not have hidden overflow", () => {
+      mount(
+        <SaltProvider>
+          <TestComponent disableScroll={false} />
+        </SaltProvider>,
+      );
+
+      cy.document().its("documentElement.style.overflow").should("equal", "");
+    });
+  });
+  describe("with disableScroll", () => {
+    it("the document body should have hidden overflow", () => {
+      mount(
+        <SaltProvider>
+          <TestComponent disableScroll={true} />
+        </SaltProvider>,
+      );
+
+      cy.document()
+        .its("documentElement.style.overflow")
+        .should("equal", "hidden");
+    });
+  });
 });

packages/core/src/dialog/Dialog.tsx

@@ -148,6 +148,7 @@ export const Dialog = forwardRef<HTMLDivElement, DialogProps>(
             ref={floatingRef}
             width={elements.floating?.offsetWidth}
             height={elements.floating?.offsetHeight}
+            disableScroll={true}
             focusManagerProps={{
               context: context,
               initialFocus,

packages/core/src/utils/useFloatingUI/useFloatingUI.tsx

@@ -19,6 +19,7 @@ import {
   createContext,
   forwardRef,
   useContext,
+  useEffect,
   useMemo,
 } from "react";
 import { SaltProvider, SaltProviderNext, useTheme } from "../../salt-provider";
@@ -45,6 +46,10 @@ export interface FloatingComponentProps
   width?: number;
   height?: number;
   position?: Strategy;
+  /**
+   * Disables scrolling the document body while the floating component is open.
+   */
+  disableScroll?: boolean;
 }
 
 const DefaultFloatingComponent = forwardRef<
@@ -59,6 +64,7 @@ const DefaultFloatingComponent = forwardRef<
     width,
     height,
     focusManagerProps,
+    disableScroll,
     ...rest
   } = props;
   const style = {
@@ -71,6 +77,20 @@ const DefaultFloatingComponent = forwardRef<
 
   const ChosenSaltProvider = themeNext ? SaltProviderNext : SaltProvider;
 
+  useEffect(() => {
+    if (!disableScroll) {
+      return;
+    }
+    if (open) {
+      document.documentElement.style.overflow = "hidden";
+    } else {
+      document.documentElement.style.removeProperty("overflow");
+    }
+    return () => {
+      document.documentElement.style.removeProperty("overflow");
+    };
+  }, [disableScroll, open]);
+
   if (focusManagerProps && open) {
     return (
       <FloatingPortal>