[modal] Fix comment location

mui · Oct 6, 2024 · fa09a2c · fa09a2c
1 parent 842e4e7
commit fa09a2c
Show file tree

Hide file tree

Showing 8 changed files with 12 additions and 36 deletions.
diff --git 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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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
@@ -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),