mcp/carbon-components-svelte
1
0
Fork 0
mirror of https://github.com/carbon-design-system/carbon-components-svelte.git synced 2025-09-15 02:11:05 +00:00
Code Issues Projects Releases Packages Wiki Activity

fix(list-box): set aria-disabled if disabled (#2125)

Browse source
This commit is contained in:
Eric Liu 2025-03-19 12:43:05 -07:00
commit 96d45e9b81
2 changed files with 2 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

1
src/ListBox/ListBoxMenuItem.svelte
View file

@ -25,6 +25,7 @@
class:bx--list-box__menu-item--active={active} class:bx--list-box__menu-item--active={active}
class:bx--list-box__menu-item--highlighted={highlighted || active} class:bx--list-box__menu-item--highlighted={highlighted || active}
aria-selected={active} aria-selected={active}
aria-disabled={disabled ? true : undefined}
disabled={disabled ? true : undefined} disabled={disabled ? true : undefined}
{...$$restProps} {...$$restProps}
on:click on:click

1
tests/ComboBox/ComboBox.test.ts
View file

@ -166,6 +166,7 @@ describe("ComboBox", () => {
const disabledOption = screen.getByText(/Fax/).closest('[role="option"]'); const disabledOption = screen.getByText(/Fax/).closest('[role="option"]');
assert(disabledOption); assert(disabledOption);
expect(disabledOption).toHaveAttribute("disabled", "true"); expect(disabledOption).toHaveAttribute("disabled", "true");
expect(disabledOption).toHaveAttribute("aria-disabled", "true");
await user.click(disabledOption); await user.click(disabledOption);
expect(screen.getByRole("textbox")).toHaveValue(""); expect(screen.getByRole("textbox")).toHaveValue("");