From 36890cfafeba6e30782df1260d7f9e678c7da4bf Mon Sep 17 00:00:00 2001 From: Armand Philippot Date: Mon, 2 Oct 2023 17:01:57 +0200 Subject: refactor(components): rewrite DescriptionList component * add a `spacing` prop * replace `layout` prop with `isInline` prop * remove `items` prop (and classNames props) in favor of new components: Description, Group, Term * remove `withSeparator` prop (CSS content is announced by screen readers and Firefox/Safari have no support for alternative text so the consumer should add itself an element with `aria-hidden` if it need a separator) Be aware, Meta component and its consumers can be visually broken, they should be refactored before using them in production. --- .../atoms/lists/description-list-group.test.tsx | 18 ------------------ 1 file changed, 18 deletions(-) delete mode 100644 src/components/atoms/lists/description-list-group.test.tsx (limited to 'src/components/atoms/lists/description-list-group.test.tsx') diff --git a/src/components/atoms/lists/description-list-group.test.tsx b/src/components/atoms/lists/description-list-group.test.tsx deleted file mode 100644 index 205dad5..0000000 --- a/src/components/atoms/lists/description-list-group.test.tsx +++ /dev/null @@ -1,18 +0,0 @@ -import { describe, expect, it } from '@jest/globals'; -import { render, screen } from '../../../../tests/utils'; -import { DescriptionListGroup } from './description-list-group'; - -const itemLabel = 'Repellendus corporis facilis'; -const itemValue = ['quos', 'eum']; - -describe('DescriptionListGroup', () => { - it('renders a couple of label', () => { - render(); - expect(screen.getByRole('term')).toHaveTextContent(itemLabel); - }); - - it('renders the right number of values', () => { - render(); - expect(screen.getAllByRole('definition')).toHaveLength(itemValue.length); - }); -}); -- cgit v1.2.3