diff --git a/.yarn/versions/fa6477bc.yml b/.yarn/versions/fa6477bc.yml new file mode 100644 index 000000000..e18060a98 --- /dev/null +++ b/.yarn/versions/fa6477bc.yml @@ -0,0 +1,7 @@ +releases: + "@radix-ui/react-toggle-group": patch + "@radix-ui/react-toolbar": patch + radix-ui: patch + +declined: + - primitives diff --git a/packages/react/toggle-group/src/toggle-group.test.tsx b/packages/react/toggle-group/src/toggle-group.test.tsx index 41c73ad55..f78305006 100644 --- a/packages/react/toggle-group/src/toggle-group.test.tsx +++ b/packages/react/toggle-group/src/toggle-group.test.tsx @@ -31,6 +31,10 @@ describe('given a single ToggleGroup', () => { expect(await axe(rendered.container)).toHaveNoViolations(); }); + it('should have radiogroup role', () => { + expect(rendered.getByRole('radiogroup')).toBeInTheDocument(); + }); + it('should change value to `One`', () => { expect(handleValueChange).toHaveBeenCalledWith('One'); }); @@ -74,6 +78,10 @@ describe('given a multiple ToggleGroup', () => { expect(await axe(rendered.container)).toHaveNoViolations(); }); + it('should have group role', () => { + expect(rendered.getByRole('group')).toBeInTheDocument(); + }); + describe('when clicking `One`', () => { beforeEach(() => { fireEvent.click(one); diff --git a/packages/react/toggle-group/src/toggle-group.tsx b/packages/react/toggle-group/src/toggle-group.tsx index 71fe81c0e..3c922282c 100644 --- a/packages/react/toggle-group/src/toggle-group.tsx +++ b/packages/react/toggle-group/src/toggle-group.tsx @@ -37,12 +37,12 @@ const ToggleGroup = React.forwardRef< if (type === 'single') { const singleProps = toggleGroupProps as ToggleGroupImplSingleProps; - return ; + return ; } if (type === 'multiple') { const multipleProps = toggleGroupProps as ToggleGroupImplMultipleProps; - return ; + return ; } throw new Error(`Missing prop \`type\` expected on \`${TOGGLE_GROUP_NAME}\``); @@ -208,7 +208,7 @@ const ToggleGroupImpl = React.forwardRef {rovingFocus ? (