fixup! ✨(frontend) improve NVDA navigation in DocShareModal

Author: Ovgodd

src/frontend/apps/e2e/__tests__/app-impress/doc-editor.spec.ts

@@ -763,9 +763,8 @@ test.describe('Doc Editor', () => {
     await expect(searchContainer.getByText(docChild2)).toBeVisible();
     await expect(searchContainer.getByText(randomDoc)).toBeHidden();
 
-    // use keydown to select the second result
-    await page.keyboard.press('ArrowDown');
-    await page.keyboard.press('Enter');
+    // Click directly on the second result instead of using keyboard navigation
+    await searchContainer.getByText(docChild2).click();
 
     // Wait for the search container to disappear, indicating selection was made
     await expect(searchContainer).toBeHidden();

src/frontend/apps/e2e/__tests__/app-impress/doc-member-create.spec.ts

@@ -118,7 +118,9 @@ test.describe('Document create member', () => {
 
     await page.getByRole('button', { name: 'Share' }).click();
 
-    const inputSearch = page.getByTestId('quick-search-input');
+    const inputSearch = page.getByRole('combobox', {
+      name: 'Quick search input',
+    });
 
     const [email] = randomName('[email protected]', browserName, 1);
     await inputSearch.fill(email);
@@ -166,7 +168,9 @@ test.describe('Document create member', () => {
 
     await page.getByRole('button', { name: 'Share' }).click();
 
-    const inputSearch = page.getByTestId('quick-search-input');
+    const inputSearch = page.getByRole('combobox', {
+      name: 'Quick search input',
+    });
 
     const email = randomName('[email protected]', browserName, 1)[0];
     await inputSearch.fill(email);

src/frontend/apps/impress/src/components/quick-search/QuickSearch.tsx

@@ -44,11 +44,10 @@ export const QuickSearch = ({
 }: PropsWithChildren<QuickSearchProps>) => {
   const ref = useRef<HTMLDivElement | null>(null);
   const listId = useId();
-  const hasResults = Boolean(children);
   const NO_SELECTION_VALUE = '__none__';
   const [userInteracted, setUserInteracted] = useState(false);
   const [selectedValue, setSelectedValue] = useState(NO_SELECTION_VALUE);
-  const isExpanded = userInteracted && hasResults;
+  const isExpanded = userInteracted;
 
   const handleValueChange = (val: string) => {
     if (userInteracted) {
@@ -82,7 +81,6 @@ export const QuickSearch = ({
               onFilter={onFilter}
               placeholder={placeholder}
               listId={listId}
-              hasResults={hasResults}
               isExpanded={isExpanded}
               onUserInteract={handleUserInteract}
             >

src/frontend/apps/impress/src/components/quick-search/QuickSearchInput.tsx

@@ -17,7 +17,6 @@ type Props = {
   children?: ReactNode;
   withSeparator?: boolean;
   listId?: string;
-  hasResults?: boolean;
   onUserInteract?: () => void;
   isExpanded?: boolean;
 };
@@ -29,7 +28,6 @@ export const QuickSearchInput = ({
   children,
   withSeparator: separator = true,
   listId,
-  hasResults,
   onUserInteract,
   isExpanded,
 }: Props) => {
@@ -65,7 +63,7 @@ export const QuickSearchInput = ({
         <Command.Input
           autoFocus={true}
           aria-label={t('Quick search input')}
-          aria-expanded={isExpanded ?? hasResults}
+          aria-expanded={isExpanded}
           aria-controls={listId}
           onClick={(e) => {
             e.stopPropagation();