From a90c52886f84b77c3674d021b8d08aef9046b326 Mon Sep 17 00:00:00 2001 From: Rohan <45748283+r100-stack@users.noreply.github.com> Date: Wed, 23 Oct 2024 09:23:57 -0400 Subject: [PATCH] Revert "Added `tileContainerProps` to `RadioTileGroup` (#2299)" This reverts commit 4bb37b2ec4f4e4d623b2e8d55050ed502d626516. --- .changeset/tough-shirts-smile.md | 5 --- .../core/RadioTiles/RadioTileGroup.test.tsx | 45 ------------------- .../src/core/RadioTiles/RadioTileGroup.tsx | 21 +++------ 3 files changed, 6 insertions(+), 65 deletions(-) delete mode 100644 .changeset/tough-shirts-smile.md diff --git a/.changeset/tough-shirts-smile.md b/.changeset/tough-shirts-smile.md deleted file mode 100644 index bc929b35bd8..00000000000 --- a/.changeset/tough-shirts-smile.md +++ /dev/null @@ -1,5 +0,0 @@ ---- -'@itwin/itwinui-react': minor ---- - -Added new `tileContainer` prop to `RadioTileGroup` to allow further customizing of inner DOM elements. diff --git a/packages/itwinui-react/src/core/RadioTiles/RadioTileGroup.test.tsx b/packages/itwinui-react/src/core/RadioTiles/RadioTileGroup.test.tsx index 208ce4d0af2..858156d9d92 100644 --- a/packages/itwinui-react/src/core/RadioTiles/RadioTileGroup.test.tsx +++ b/packages/itwinui-react/src/core/RadioTiles/RadioTileGroup.test.tsx @@ -42,48 +42,3 @@ it('should render with label', () => { expect(label).toBeTruthy(); expect(label.textContent).toBe('My tiles'); }); - -it('should apply all custom props', () => { - const { container } = render( - - - - , - ); - - expect(container.querySelector('.iui-input-grid.root-props')).toBeTruthy(); - expect( - container.querySelector('.iui-input-grid > .iui-input-label.label-props'), - ).toBeTruthy(); - expect( - container.querySelector('.iui-input-grid > .iui-input-group.inner-props'), - ).toBeTruthy(); - expect( - container.querySelector( - '.iui-input-grid > .iui-input-group > .iui-radio-tile-container.tile-container-props', - ), - ).toBeTruthy(); - expect( - container.querySelector( - '.iui-input-grid > .iui-status-message > .message-content-props', - ), - ).toHaveTextContent('Message'); -}); diff --git a/packages/itwinui-react/src/core/RadioTiles/RadioTileGroup.tsx b/packages/itwinui-react/src/core/RadioTiles/RadioTileGroup.tsx index 52069ee1796..140e90fed38 100644 --- a/packages/itwinui-react/src/core/RadioTiles/RadioTileGroup.tsx +++ b/packages/itwinui-react/src/core/RadioTiles/RadioTileGroup.tsx @@ -6,11 +6,11 @@ import * as React from 'react'; import { InputGroup } from '../InputGroup/InputGroup.js'; import { Box } from '../../utils/index.js'; import type { PolymorphicForwardRefComponent } from '../../utils/index.js'; -import cx from 'classnames'; -type RadioTileGroupProps = { - tileContainerProps?: React.ComponentProps<'div'>; -} & Omit, 'displayStyle' | 'disabled'>; +type RadioTileGroupProps = Omit< + React.ComponentProps, + 'displayStyle' | 'disabled' +>; /** * RadioTileGroup component to group RadioTile components together @@ -21,20 +21,11 @@ type RadioTileGroupProps = { * */ export const RadioTileGroup = React.forwardRef((props, forwardedRef) => { - const { children, label, tileContainerProps, ...rest } = props; + const { children, label, ...rest } = props; return ( - - {children} - + {children} ); }) as PolymorphicForwardRefComponent<'div', RadioTileGroupProps>;