mats.bosson/PettyUI
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Fix ToggleGroup disabled handling

Browse Source
This commit is contained in:
Mats Bosson 2026-03-29 07:57:25 +07:00
parent 40e57715f9
commit 315f5e4ae2
2 changed files with 14 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
packages/core/src/components/toggle-group/toggle-group-item.tsx
View File

@ -24,8 +24,10 @@ export function ToggleGroupItem(props: ToggleGroupItemProps): JSX.Element {
disabled={isDisabled()} disabled={isDisabled()}
{...rest} {...rest}
onClick={(e) => { onClick={(e) => {
if (typeof rest.onClick === "function") rest.onClick(e); if (!isDisabled()) {
if (!isDisabled()) ctx.onItemPress(local.value); if (typeof rest.onClick === "function") rest.onClick(e);
ctx.onItemPress(local.value);
}
}} }}
> >
{local.children} {local.children}

10
packages/core/tests/components/toggle-group/toggle-group.test.tsx
View File

@ -71,4 +71,14 @@ describe("ToggleGroup multiple", () => {
fireEvent.click(screen.getByRole("button")); fireEvent.click(screen.getByRole("button"));
expect(screen.getByRole("button").getAttribute("aria-pressed")).toBe("false"); expect(screen.getByRole("button").getAttribute("aria-pressed")).toBe("false");
}); });
it("disabled item is not pressable", () => {
render(() => (
<ToggleGroup type="single">
<ToggleGroup.Item value="a" disabled>A</ToggleGroup.Item>
</ToggleGroup>
));
fireEvent.click(screen.getByRole("button"));
expect(screen.getByRole("button").getAttribute("aria-pressed")).toBe("false");
});
}); });