fix([email protected]): prevent scroll when modal container and sentinel get focused

Author: afc163

src/Dialog/Content/Panel.tsx

@@ -48,14 +48,14 @@ const Panel = React.forwardRef<ContentRef, PanelProps>((props, ref) => {
 
   React.useImperativeHandle(ref, () => ({
     focus: () => {
-      sentinelStartRef.current?.focus();
+      sentinelStartRef.current?.focus({ preventScroll: true });
     },
     changeActive: (next) => {
       const { activeElement } = document;
       if (next && activeElement === sentinelEndRef.current) {
-        sentinelStartRef.current.focus();
+        sentinelStartRef.current.focus({ preventScroll: true });
       } else if (!next && activeElement === sentinelStartRef.current) {
-        sentinelEndRef.current.focus();
+        sentinelEndRef.current.focus({ preventScroll: true });
       }
     },
   }));

src/Dialog/index.tsx

@@ -129,10 +129,8 @@ export default function Dialog(props: IDialogPropTypes) {
     }
 
     // keep focus inside dialog
-    if (visible) {
-      if (e.keyCode === KeyCode.TAB) {
-        contentRef.current.changeActive(!e.shiftKey);
-      }
+    if (visible && e.keyCode === KeyCode.TAB) {
+      contentRef.current.changeActive(!e.shiftKey);
     }
   }