chore: Prevent dialog scroll (#7710)

* chore: Prevent dialog scroll * chore: preventBodyScroll should not be passed as a part of restProps
JetBrains · Sep 12, 2024 · 380979e · 380979e
1 parent 062f75f
commit 380979e
Showing 1 changed file with 9 additions and 3 deletions.
diff --git a/src/dialog/dialog.tsx b/src/dialog/dialog.tsx
@@ -36,6 +36,7 @@ export interface DialogProps extends Partial<TabTrapProps> {
   dense?: boolean | null | undefined
   native?: boolean
   modal?: boolean
+  preventBodyScroll?: boolean
 }
 
 /**
@@ -68,7 +69,8 @@ export default class Dialog extends PureComponent<DialogProps> {
     trapFocus: PropTypes.bool,
     portalTarget: PropTypes.instanceOf(HTMLElement),
     autoFocusFirst: PropTypes.bool,
-    'data-test': PropTypes.string
+    'data-test': PropTypes.string,
+    preventBodyScroll: PropTypes.bool
   };
 
   static defaultProps: Partial<DialogProps> = {
@@ -82,7 +84,8 @@ export default class Dialog extends PureComponent<DialogProps> {
     shortcutOptions: {modal: false},
     trapFocus: false,
     autoFocusFirst: true,
-    modal: true
+    modal: true,
+    preventBodyScroll: true
   };
 
   state = {
@@ -128,6 +131,9 @@ export default class Dialog extends PureComponent<DialogProps> {
   }
 
   toggleScrollPreventer() {
+    if (!this.props.preventBodyScroll) {
+      return;
+    }
     if (this.props.show) {
       this.scrollPreventer.prevent();
     } else {
@@ -169,7 +175,7 @@ export default class Dialog extends PureComponent<DialogProps> {
     const {show, showCloseButton, onOverlayClick, onCloseAttempt, onEscPress, onCloseClick,
       children, className, contentClassName, trapFocus, 'data-test': dataTest, closeButtonInside,
       portalTarget, label, closeButtonTitle, dense, shortcutOptions, native, modal,
-      ...restProps} = this.props;
+      preventBodyScroll, ...restProps} = this.props;
     const classes = classNames(styles.container, className);
     const shortcutsMap = this.getShortcutsMap();
     const content = (