Skip to content

Commit 618fa3a

Browse files
Barsnesmimarz
authored andcommitted
chore(combobox): sync with main changes (#2576)
this PR is only to reflect changes made in an older version in PR #2575
1 parent 00ece54 commit 618fa3a

File tree

2 files changed

+3
-6
lines changed

2 files changed

+3
-6
lines changed

packages/react/src/components/form/Combobox/Combobox.tsx

+2-5
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
import { FloatingFocusManager, FloatingPortal } from '@floating-ui/react';
22
import { useVirtualizer } from '@tanstack/react-virtual';
33
import cl from 'clsx/lite';
4-
import { forwardRef, useEffect, useId, useRef, useState } from 'react';
4+
import { forwardRef, useEffect, useRef, useState } from 'react';
55
import type { InputHTMLAttributes, ReactNode } from 'react';
66

77
import type { PortalProps } from '../../../types/Portal';
@@ -156,8 +156,6 @@ export const ComboboxComponent = forwardRef<HTMLInputElement, ComboboxProps>(
156156
const portalRef = useRef<HTMLDivElement>(null);
157157
const listRef = useRef<Array<HTMLElement | null>>([]);
158158

159-
const listId = useId();
160-
161159
const [inputValue, setInputValue] = useState<string>(rest.inputValue || '');
162160

163161
useEffect(() => {
@@ -392,7 +390,7 @@ export const ComboboxComponent = forwardRef<HTMLInputElement, ComboboxProps>(
392390
<ComboboxInput
393391
{...omit(['inputValue'], rest)}
394392
hideClearButton={hideClearButton}
395-
listId={listId}
393+
listId={context.floatingId}
396394
error={error}
397395
hideChips={hideChips}
398396
handleKeyDown={handleKeyDown}
@@ -413,7 +411,6 @@ export const ComboboxComponent = forwardRef<HTMLInputElement, ComboboxProps>(
413411
visuallyHiddenDismiss
414412
>
415413
<div
416-
id={listId}
417414
aria-labelledby={formFieldProps.inputProps.id}
418415
aria-autocomplete='list'
419416
tabIndex={-1}

packages/react/src/components/form/Combobox/internal/ComboboxInput.tsx

+1-1
Original file line numberDiff line numberDiff line change
@@ -137,7 +137,7 @@ const ComboboxInput = ({
137137
aria-autocomplete='list'
138138
role='combobox'
139139
aria-expanded={open}
140-
aria-controls={listId}
140+
aria-controls={open ? listId : undefined}
141141
autoComplete='off'
142142
size={htmlSize}
143143
value={inputValue}

0 commit comments

Comments
 (0)