From 8e4a898bbe876483d37deb291997c1ccacaba272 Mon Sep 17 00:00:00 2001 From: Damien Pellier Date: Wed, 27 Nov 2024 16:23:18 +0100 Subject: [PATCH] feat(doc): separate validityState in custom control group --- packages/storybook/src/constants/controls.ts | 1 + packages/storybook/src/helpers/controls.ts | 3 ++- .../storybook/stories/components/checkbox/checkbox.stories.ts | 2 +- .../stories/components/datepicker/datepicker.stories.ts | 2 +- packages/storybook/stories/components/input/input.stories.ts | 3 +-- .../storybook/stories/components/password/password.stories.ts | 2 +- .../stories/components/phone-number/phone-number.stories.ts | 2 +- .../storybook/stories/components/quantity/quantity.stories.ts | 2 +- packages/storybook/stories/components/radio/radio.stories.ts | 2 +- packages/storybook/stories/components/range/range.stories.ts | 4 ++-- .../storybook/stories/components/select/select.stories.ts | 4 ++-- .../storybook/stories/components/switch/switch.stories.ts | 2 +- .../storybook/stories/components/textarea/textarea.stories.ts | 2 +- .../stories/components/timepicker/timepicker.stories.ts | 2 +- .../storybook/stories/components/toggle/toggle.stories.ts | 2 +- 15 files changed, 18 insertions(+), 17 deletions(-) diff --git a/packages/storybook/src/constants/controls.ts b/packages/storybook/src/constants/controls.ts index bb69cefb78..a20b36823e 100644 --- a/packages/storybook/src/constants/controls.ts +++ b/packages/storybook/src/constants/controls.ts @@ -3,6 +3,7 @@ enum CONTROL_CATEGORY { design = 'Design', general = 'General', slot = 'Slot', + validation = 'Validation', } export { diff --git a/packages/storybook/src/helpers/controls.ts b/packages/storybook/src/helpers/controls.ts index f37d42c521..adc83bb794 100644 --- a/packages/storybook/src/helpers/controls.ts +++ b/packages/storybook/src/helpers/controls.ts @@ -5,13 +5,14 @@ const orderedControlCategories = [ CONTROL_CATEGORY.slot, CONTROL_CATEGORY.general, CONTROL_CATEGORY.accessibility, + CONTROL_CATEGORY.validation, ]; function orderControls(control: Record) { try { return Object.entries(control) .sort((a, b) => { - return orderedControlCategories.indexOf(a[1].category) - orderedControlCategories.indexOf(b[1].category); + return orderedControlCategories.indexOf(a[1].table.category) - orderedControlCategories.indexOf(b[1].table.category); }) .reduce((res, entry) => { res[entry[0]] = entry[1]; diff --git a/packages/storybook/stories/components/checkbox/checkbox.stories.ts b/packages/storybook/stories/components/checkbox/checkbox.stories.ts index 67fbc2736b..974bcdb074 100644 --- a/packages/storybook/stories/components/checkbox/checkbox.stories.ts +++ b/packages/storybook/stories/components/checkbox/checkbox.stories.ts @@ -81,7 +81,7 @@ export const Demo: StoryObj = { }, validityState: { table: { - category: CONTROL_CATEGORY.accessibility, + category: CONTROL_CATEGORY.validation, defaultValue: { summary: false }, type: { summary: 'boolean' }, }, diff --git a/packages/storybook/stories/components/datepicker/datepicker.stories.ts b/packages/storybook/stories/components/datepicker/datepicker.stories.ts index ff3a997924..6a15fc41aa 100644 --- a/packages/storybook/stories/components/datepicker/datepicker.stories.ts +++ b/packages/storybook/stories/components/datepicker/datepicker.stories.ts @@ -143,7 +143,7 @@ export const Demo: StoryObj = { }, validityState: { table: { - category: CONTROL_CATEGORY.accessibility, + category: CONTROL_CATEGORY.validation, defaultValue: { summary: false }, type: { summary: 'boolean' }, }, diff --git a/packages/storybook/stories/components/input/input.stories.ts b/packages/storybook/stories/components/input/input.stories.ts index 9b867e1afb..79c71c29e0 100644 --- a/packages/storybook/stories/components/input/input.stories.ts +++ b/packages/storybook/stories/components/input/input.stories.ts @@ -13,7 +13,6 @@ const meta: Meta = { export default meta; - export const Demo: StoryObj = { render: (arg) => { return html` @@ -193,7 +192,7 @@ export const Demo: StoryObj = { }, validityState: { table: { - category: CONTROL_CATEGORY.accessibility, + category: CONTROL_CATEGORY.validation, defaultValue: { summary: false }, type: { summary: 'boolean' }, }, diff --git a/packages/storybook/stories/components/password/password.stories.ts b/packages/storybook/stories/components/password/password.stories.ts index 0b773e1528..64cf32ea0c 100644 --- a/packages/storybook/stories/components/password/password.stories.ts +++ b/packages/storybook/stories/components/password/password.stories.ts @@ -127,7 +127,7 @@ export const Demo: StoryObj = { }, validityState: { table: { - category: CONTROL_CATEGORY.accessibility, + category: CONTROL_CATEGORY.validation, defaultValue: { summary: false }, type: { summary: 'boolean' }, }, diff --git a/packages/storybook/stories/components/phone-number/phone-number.stories.ts b/packages/storybook/stories/components/phone-number/phone-number.stories.ts index 18c794fd1e..67432b4299 100644 --- a/packages/storybook/stories/components/phone-number/phone-number.stories.ts +++ b/packages/storybook/stories/components/phone-number/phone-number.stories.ts @@ -158,7 +158,7 @@ export const Demo: StoryObj = { }, validityState: { table: { - category: CONTROL_CATEGORY.accessibility, + category: CONTROL_CATEGORY.validation, defaultValue: { summary: false }, type: { summary: 'boolean' }, }, diff --git a/packages/storybook/stories/components/quantity/quantity.stories.ts b/packages/storybook/stories/components/quantity/quantity.stories.ts index 2594b2bb93..9d5a021153 100644 --- a/packages/storybook/stories/components/quantity/quantity.stories.ts +++ b/packages/storybook/stories/components/quantity/quantity.stories.ts @@ -143,7 +143,7 @@ export const Demo: StoryObj = { }, validityState: { table: { - category: CONTROL_CATEGORY.accessibility, + category: CONTROL_CATEGORY.validation, defaultValue: { summary: false }, type: { summary: 'boolean' }, }, diff --git a/packages/storybook/stories/components/radio/radio.stories.ts b/packages/storybook/stories/components/radio/radio.stories.ts index 02c2510367..4adec1837e 100644 --- a/packages/storybook/stories/components/radio/radio.stories.ts +++ b/packages/storybook/stories/components/radio/radio.stories.ts @@ -74,7 +74,7 @@ export const Demo: StoryObj = { }, validityState: { table: { - category: CONTROL_CATEGORY.accessibility, + category: CONTROL_CATEGORY.validation, defaultValue: { summary: false }, type: { summary: 'boolean' }, }, diff --git a/packages/storybook/stories/components/range/range.stories.ts b/packages/storybook/stories/components/range/range.stories.ts index 563fdd16f8..5477c1e577 100644 --- a/packages/storybook/stories/components/range/range.stories.ts +++ b/packages/storybook/stories/components/range/range.stories.ts @@ -92,7 +92,7 @@ export const Demo: StoryObj = { }, validityState: { table: { - category: CONTROL_CATEGORY.accessibility, + category: CONTROL_CATEGORY.validation, defaultValue: { summary: false }, type: { summary: 'boolean' }, }, @@ -220,7 +220,7 @@ export const DemoDual: StoryObj = { }, validityState: { table: { - category: CONTROL_CATEGORY.accessibility, + category: CONTROL_CATEGORY.validation, defaultValue: { summary: false }, type: { summary: 'boolean' }, }, diff --git a/packages/storybook/stories/components/select/select.stories.ts b/packages/storybook/stories/components/select/select.stories.ts index fdd89fda03..eb6b1a35b5 100644 --- a/packages/storybook/stories/components/select/select.stories.ts +++ b/packages/storybook/stories/components/select/select.stories.ts @@ -74,7 +74,7 @@ export const Demo: StoryObj = { }, validityState: { table: { - category: CONTROL_CATEGORY.accessibility, + category: CONTROL_CATEGORY.validation, defaultValue: { summary: false }, type: { summary: 'boolean' }, }, @@ -163,7 +163,7 @@ export const DemoMultiple: StoryObj = { }, validityState: { table: { - category: CONTROL_CATEGORY.accessibility, + category: CONTROL_CATEGORY.validation, defaultValue: { summary: false }, type: { summary: 'boolean' }, }, diff --git a/packages/storybook/stories/components/switch/switch.stories.ts b/packages/storybook/stories/components/switch/switch.stories.ts index 3b5be6c017..c011f524f0 100644 --- a/packages/storybook/stories/components/switch/switch.stories.ts +++ b/packages/storybook/stories/components/switch/switch.stories.ts @@ -55,7 +55,7 @@ export const Demo: StoryObj = { }, validityState: { table: { - category: CONTROL_CATEGORY.accessibility, + category: CONTROL_CATEGORY.validation, defaultValue: { summary: false }, type: { summary: 'boolean' }, }, diff --git a/packages/storybook/stories/components/textarea/textarea.stories.ts b/packages/storybook/stories/components/textarea/textarea.stories.ts index 8e7f8a8a38..1698e5d5f3 100644 --- a/packages/storybook/stories/components/textarea/textarea.stories.ts +++ b/packages/storybook/stories/components/textarea/textarea.stories.ts @@ -135,7 +135,7 @@ export const Demo: StoryObj = { }, validityState: { table: { - category: CONTROL_CATEGORY.accessibility, + category: CONTROL_CATEGORY.validation, defaultValue: { summary: false }, type: { summary: 'boolean' }, }, diff --git a/packages/storybook/stories/components/timepicker/timepicker.stories.ts b/packages/storybook/stories/components/timepicker/timepicker.stories.ts index 8fb3db0747..73f4a7f562 100644 --- a/packages/storybook/stories/components/timepicker/timepicker.stories.ts +++ b/packages/storybook/stories/components/timepicker/timepicker.stories.ts @@ -101,7 +101,7 @@ export const Demo: StoryObj = { }, validityState: { table: { - category: CONTROL_CATEGORY.accessibility, + category: CONTROL_CATEGORY.validation, defaultValue: { summary: false }, type: { summary: 'boolean' }, }, diff --git a/packages/storybook/stories/components/toggle/toggle.stories.ts b/packages/storybook/stories/components/toggle/toggle.stories.ts index f7095d34c6..9055ead17a 100644 --- a/packages/storybook/stories/components/toggle/toggle.stories.ts +++ b/packages/storybook/stories/components/toggle/toggle.stories.ts @@ -120,7 +120,7 @@ export const Demo: StoryObj = { }, validityState: { table: { - category: CONTROL_CATEGORY.accessibility, + category: CONTROL_CATEGORY.validation, defaultValue: { summary: false }, type: { summary: 'boolean' }, },