From e5b81c8cec4c872d638676dedbe0fbe232d235b6 Mon Sep 17 00:00:00 2001
From: Olivier Tassinari <olivier.tassinari@gmail.com>
Date: Sun, 6 Oct 2024 23:10:59 +0200
Subject: [PATCH] [modal] Fix comment location (#44026)

---
 docs/data/base/components/modal/UseModal.js         | 6 ------
 docs/data/base/components/modal/UseModal.tsx        | 6 ------
 packages/mui-base/src/Modal/Modal.tsx               | 6 ------
 packages/mui-base/src/unstable_useModal/useModal.ts | 6 ++++++
 packages/mui-joy/src/Drawer/Drawer.tsx              | 6 ------
 packages/mui-joy/src/Modal/Modal.tsx                | 6 ------
 packages/mui-material/src/Modal/Modal.js            | 6 ------
 packages/mui-material/src/Modal/useModal.ts         | 6 ++++++
 8 files changed, 12 insertions(+), 36 deletions(-)

diff --git a/docs/data/base/components/modal/UseModal.js b/docs/data/base/components/modal/UseModal.js
index b35a6642a33844..02be482df395da 100644
--- a/docs/data/base/components/modal/UseModal.js
+++ b/docs/data/base/components/modal/UseModal.js
@@ -118,12 +118,6 @@ const Modal = React.forwardRef(function Modal(props, forwardedRef) {
 
   return (
     <Portal ref={portalRef} container={container} disablePortal={disablePortal}>
-      {/*
-       * Marking an element with the role presentation indicates to assistive technology
-       * that this element should be ignored; it exists to support the web application and
-       * is not meant for humans to interact with directly.
-       * https://github.com/evcohen/eslint-plugin-jsx-a11y/blob/master/docs/rules/no-static-element-interactions.md
-       */}
       <CustomModalRoot {...rootProps}>
         {!hideBackdrop ? <CustomModalBackdrop {...backdropProps} /> : null}
         <FocusTrap
diff --git a/docs/data/base/components/modal/UseModal.tsx b/docs/data/base/components/modal/UseModal.tsx
index 6faefd0c8d7417..38d4679a13fb01 100644
--- a/docs/data/base/components/modal/UseModal.tsx
+++ b/docs/data/base/components/modal/UseModal.tsx
@@ -142,12 +142,6 @@ const Modal = React.forwardRef(function Modal(
 
   return (
     <Portal ref={portalRef} container={container} disablePortal={disablePortal}>
-      {/*
-       * Marking an element with the role presentation indicates to assistive technology
-       * that this element should be ignored; it exists to support the web application and
-       * is not meant for humans to interact with directly.
-       * https://github.com/evcohen/eslint-plugin-jsx-a11y/blob/master/docs/rules/no-static-element-interactions.md
-       */}
       <CustomModalRoot {...rootProps}>
         {!hideBackdrop ? <CustomModalBackdrop {...backdropProps} /> : null}
         <FocusTrap
diff --git a/packages/mui-base/src/Modal/Modal.tsx b/packages/mui-base/src/Modal/Modal.tsx
index 285175bb3f12e5..93637a56e2c529 100644
--- a/packages/mui-base/src/Modal/Modal.tsx
+++ b/packages/mui-base/src/Modal/Modal.tsx
@@ -157,12 +157,6 @@ const Modal = React.forwardRef<HTMLElement, ModalProps>(function Modal<
 
   return (
     <Portal ref={portalRef} container={container} disablePortal={disablePortal}>
-      {/*
-       * Marking an element with the role presentation indicates to assistive technology
-       * that this element should be ignored; it exists to support the web application and
-       * is not meant for humans to interact with directly.
-       * https://github.com/evcohen/eslint-plugin-jsx-a11y/blob/master/docs/rules/no-static-element-interactions.md
-       */}
       <Root {...rootProps}>
         {!hideBackdrop && BackdropComponent ? <BackdropComponent {...backdropProps} /> : null}
         <FocusTrap
diff --git a/packages/mui-base/src/unstable_useModal/useModal.ts b/packages/mui-base/src/unstable_useModal/useModal.ts
index 3166acd289d6eb..ff025dd334e4cc 100644
--- a/packages/mui-base/src/unstable_useModal/useModal.ts
+++ b/packages/mui-base/src/unstable_useModal/useModal.ts
@@ -180,6 +180,12 @@ export function useModal(parameters: UseModalParameters): UseModalReturnValue {
     };
 
     return {
+      /*
+       * Marking an element with the role presentation indicates to assistive technology
+       * that this element should be ignored; it exists to support the web application and
+       * is not meant for humans to interact with directly.
+       * https://github.com/evcohen/eslint-plugin-jsx-a11y/blob/master/docs/rules/no-static-element-interactions.md
+       */
       role: 'presentation',
       ...externalEventHandlers,
       onKeyDown: createHandleKeyDown(externalEventHandlers),
diff --git a/packages/mui-joy/src/Drawer/Drawer.tsx b/packages/mui-joy/src/Drawer/Drawer.tsx
index 9c49525f6cc86b..1bf3d613322e2e 100644
--- a/packages/mui-joy/src/Drawer/Drawer.tsx
+++ b/packages/mui-joy/src/Drawer/Drawer.tsx
@@ -245,12 +245,6 @@ const Drawer = React.forwardRef(function Drawer(inProps, ref) {
       <ModalDialogSizeContext.Provider value={size}>
         <ModalDialogVariantColorContext.Provider value={contextValue}>
           <Portal ref={portalRef} container={container} disablePortal={disablePortal}>
-            {/*
-             * Marking an element with the role presentation indicates to assistive technology
-             * that this element should be ignored; it exists to support the web application and
-             * is not meant for humans to interact with directly.
-             * https://github.com/evcohen/eslint-plugin-jsx-a11y/blob/master/docs/rules/no-static-element-interactions.md
-             */}
             <SlotRoot {...rootProps}>
               {!hideBackdrop ? <SlotBackdrop {...backdropProps} /> : null}
               <FocusTrap
diff --git a/packages/mui-joy/src/Modal/Modal.tsx b/packages/mui-joy/src/Modal/Modal.tsx
index bdcce97f6f58ac..404dd1e5e66c28 100644
--- a/packages/mui-joy/src/Modal/Modal.tsx
+++ b/packages/mui-joy/src/Modal/Modal.tsx
@@ -146,12 +146,6 @@ const Modal = React.forwardRef(function Modal(inProps, ref) {
   return (
     <CloseModalContext.Provider value={onClose}>
       <Portal ref={portalRef} container={container} disablePortal={disablePortal}>
-        {/*
-         * Marking an element with the role presentation indicates to assistive technology
-         * that this element should be ignored; it exists to support the web application and
-         * is not meant for humans to interact with directly.
-         * https://github.com/evcohen/eslint-plugin-jsx-a11y/blob/master/docs/rules/no-static-element-interactions.md
-         */}
         <SlotRoot {...rootProps}>
           {!hideBackdrop ? <SlotBackdrop {...backdropProps} /> : null}
           <FocusTrap
diff --git a/packages/mui-material/src/Modal/Modal.js b/packages/mui-material/src/Modal/Modal.js
index 7e121bf29e7b5e..20c8747d5229c3 100644
--- a/packages/mui-material/src/Modal/Modal.js
+++ b/packages/mui-material/src/Modal/Modal.js
@@ -213,12 +213,6 @@ const Modal = React.forwardRef(function Modal(inProps, ref) {
 
   return (
     <Portal ref={portalRef} container={container} disablePortal={disablePortal}>
-      {/*
-       * Marking an element with the role presentation indicates to assistive technology
-       * that this element should be ignored; it exists to support the web application and
-       * is not meant for humans to interact with directly.
-       * https://github.com/evcohen/eslint-plugin-jsx-a11y/blob/master/docs/rules/no-static-element-interactions.md
-       */}
       <RootSlot {...rootProps}>
         {!hideBackdrop && BackdropComponent ? (
           <BackdropSlot {...backdropProps} ref={backdropRef} />
diff --git a/packages/mui-material/src/Modal/useModal.ts b/packages/mui-material/src/Modal/useModal.ts
index 9aca8a03c46a18..4cb2b992beae60 100644
--- a/packages/mui-material/src/Modal/useModal.ts
+++ b/packages/mui-material/src/Modal/useModal.ts
@@ -179,6 +179,12 @@ function useModal(parameters: UseModalParameters): UseModalReturnValue {
     };
 
     return {
+      /*
+       * Marking an element with the role presentation indicates to assistive technology
+       * that this element should be ignored; it exists to support the web application and
+       * is not meant for humans to interact with directly.
+       * https://github.com/evcohen/eslint-plugin-jsx-a11y/blob/master/docs/rules/no-static-element-interactions.md
+       */
       role: 'presentation',
       ...externalEventHandlers,
       onKeyDown: createHandleKeyDown(externalEventHandlers),