✨(frontend) add focus trap and enter key support to remove doc modal

improves accessibility by enabling keyboard-triggered modal with proper focus trap

Signed-off-by: Cyril <[email protected]>

Author: Ovgodd

src/frontend/apps/impress/src/components/dropdown-menu/DropdownMenu.tsx

@@ -1,6 +1,7 @@
 import { HorizontalSeparator } from '@gouvfr-lasuite/ui-kit';
 import {
   Fragment,
+  KeyboardEvent,
   PropsWithChildren,
   ReactNode,
   useCallback,
@@ -93,6 +94,18 @@ export const DropdownMenu = ({
     }
   }, [isOpen, options]);
 
+  const handleKeyDown = useCallback(
+    (event: KeyboardEvent<HTMLElement>, option: DropdownMenuOption) => {
+      if (event.key === 'Enter' || event.key === ' ') {
+        event.preventDefault();
+        event.stopPropagation();
+        onOpenChange?.(false);
+        void option.callback?.();
+      }
+    },
+    [onOpenChange],
+  );
+
   if (disabled) {
     return children;
   }
@@ -173,6 +186,7 @@ export const DropdownMenu = ({
                   onOpenChange?.(false);
                   void option.callback?.();
                 }}
+                onKeyDown={(event) => handleKeyDown(event, option)}
                 key={option.label}
                 $align="center"
                 $justify="space-between"

src/frontend/apps/impress/src/features/docs/doc-management/components/ModalRemoveDoc.tsx

@@ -6,17 +6,21 @@ import {
   useToastProvider,
 } from '@openfun/cunningham-react';
 import { useRouter } from 'next/router';
+import { useEffect } from 'react';
 import { Trans, useTranslation } from 'react-i18next';
 
 import { Box, ButtonCloseModal, Text, TextErrors } from '@/components';
 import { useConfig } from '@/core';
 import { KEY_LIST_DOC_TRASHBIN } from '@/docs/docs-grid';
+import { useKeyboardAction } from '@/hooks';
 
 import { KEY_LIST_DOC } from '../api/useDocs';
 import { useRemoveDoc } from '../api/useRemoveDoc';
 import { useDocUtils } from '../hooks';
 import { Doc } from '../types';
 
+const CANCEL_BUTTON_ID = 'modal-remove-doc-cancel-button';
+
 interface ModalRemoveDocProps {
   doc: Doc;
   onClose: () => void;
@@ -57,32 +61,51 @@ export const ModalRemoveDoc = ({
     },
   });
 
+  useEffect(() => {
+    const cancelButton = document.getElementById(CANCEL_BUTTON_ID);
+    if (cancelButton instanceof HTMLButtonElement) {
+      cancelButton.focus();
+    }
+  }, []);
+
+  const handleClose = () => {
+    onClose();
+  };
+
+  const handleDelete = () => {
+    removeDoc({
+      docId: doc.id,
+    });
+  };
+
+  const handleCloseKeyDown = useKeyboardAction(handleClose);
+  const handleDeleteKeyDown = useKeyboardAction(handleDelete);
+
   return (
     <Modal
       isOpen
       closeOnClickOutside
       hideCloseButton
-      onClose={() => onClose()}
+      onClose={handleClose}
       aria-describedby="modal-remove-doc-title"
       rightActions={
         <>
           <Button
+            id={CANCEL_BUTTON_ID}
             aria-label={t('Cancel the deletion')}
             color="secondary"
             fullWidth
-            onClick={() => onClose()}
+            onClick={handleClose}
+            onKeyDown={handleCloseKeyDown}
           >
             {t('Cancel')}
           </Button>
           <Button
             aria-label={t('Delete document')}
             color="danger"
             fullWidth
-            onClick={() =>
-              removeDoc({
-                docId: doc.id,
-              })
-            }
+            onClick={handleDelete}
+            onKeyDown={handleDeleteKeyDown}
           >
             {t('Delete')}
           </Button>
@@ -108,7 +131,8 @@ export const ModalRemoveDoc = ({
           </Text>
           <ButtonCloseModal
             aria-label={t('Close the delete modal')}
-            onClick={() => onClose()}
+            onClick={handleClose}
+            onKeyDown={handleCloseKeyDown}
           />
         </Box>
       }

src/frontend/apps/impress/src/hooks/index.ts

@@ -0,0 +1 @@
+export * from './useKeyboardAction';

src/frontend/apps/impress/src/hooks/useKeyboardAction.ts

@@ -0,0 +1,22 @@
+import { KeyboardEvent, useCallback } from 'react';
+
+/**
+ * Hook to handle keyboard actions (Enter and Space) on interactive elements.
+ * Prevents default behavior and stops propagation to avoid conflicts.
+ *
+ * @param callback - The function to execute when Enter or Space is pressed
+ * @returns A keyboard event handler function
+
+ */
+export const useKeyboardAction = (callback: () => void) => {
+  return useCallback(
+    (event: KeyboardEvent<HTMLElement>) => {
+      if (event.key === 'Enter' || event.key === ' ') {
+        event.preventDefault();
+        event.stopPropagation();
+        callback();
+      }
+    },
+    [callback],
+  );
+};