From a5df28fad0dae266a857ae110c43b3cb8b23c996 Mon Sep 17 00:00:00 2001 From: Armand Philippot Date: Fri, 8 Apr 2022 19:41:40 +0200 Subject: refactor: use a consistent classname prop and avoid children prop I was using the FunctionComponent type for some component that do not use children. So I change the type to VoidFunctionComponent to avoid mistakes. I also rename all the "classes" or "additionalClasses" props to "className" to keep consistency between each components. --- .../molecules/layout/flipping-logo.stories.tsx | 23 +++++++++++++++++----- 1 file changed, 18 insertions(+), 5 deletions(-) (limited to 'src/components/molecules/layout/flipping-logo.stories.tsx') diff --git a/src/components/molecules/layout/flipping-logo.stories.tsx b/src/components/molecules/layout/flipping-logo.stories.tsx index 1508269..1ac8de8 100644 --- a/src/components/molecules/layout/flipping-logo.stories.tsx +++ b/src/components/molecules/layout/flipping-logo.stories.tsx @@ -5,11 +5,21 @@ export default { title: 'Molecules/Layout', component: FlippingLogoComponent, argTypes: { - additionalClasses: { + altText: { + control: { + type: 'text', + }, + description: 'Photo alternative text.', + type: { + name: 'string', + required: true, + }, + }, + className: { control: { type: 'text', }, - description: 'Adds additional classes to the logo wrapper.', + description: 'Set additional classnames to the logo wrapper.', table: { category: 'Options', }, @@ -18,14 +28,17 @@ export default { required: false, }, }, - altText: { + logoTitle: { control: { type: 'text', }, - description: 'Photo alternative text.', + description: 'An accessible name for the logo.', + table: { + category: 'Accessibility', + }, type: { name: 'string', - required: true, + required: false, }, }, photo: { -- cgit v1.2.3