From f861e6a269ba9f62700776d3cd13b644a9e836d4 Mon Sep 17 00:00:00 2001 From: Armand Philippot Date: Wed, 20 Sep 2023 16:38:54 +0200 Subject: refactor: use named export for everything except pages Next expect a default export for pages so only those components should use default exports. Everything else should use named exports to reduce the number of import statements. --- src/components/atoms/layout/section.tsx | 25 +++++++++++-------------- 1 file changed, 11 insertions(+), 14 deletions(-) (limited to 'src/components/atoms/layout/section.tsx') diff --git a/src/components/atoms/layout/section.tsx b/src/components/atoms/layout/section.tsx index cb727ff..107e80a 100644 --- a/src/components/atoms/layout/section.tsx +++ b/src/components/atoms/layout/section.tsx @@ -1,18 +1,17 @@ -import { FC, ReactNode } from 'react'; -import Heading from '../headings/heading'; +import { FC, HTMLAttributes, ReactNode } from 'react'; +import { Heading } from '../headings'; import styles from './section.module.scss'; export type SectionVariant = 'dark' | 'light'; -export type SectionProps = { - /** - * Set additional classnames to the section element. - */ - className?: string; +export type SectionProps = Omit< + HTMLAttributes, + 'children' | 'content' +> & { /** * The section content. */ - content: ReactNode; + content: ReactNode | ReactNode[]; /** * The section title. */ @@ -32,20 +31,20 @@ export type SectionProps = { * * Render a section element. */ -const Section: FC = ({ +export const Section: FC = ({ className = '', content, title, variant = 'dark', withBorder = true, + ...props }) => { const borderClass = withBorder ? styles[`wrapper--borders`] : ''; const variantClass = styles[`wrapper--${variant}`]; + const sectionClass = `${styles.wrapper} ${borderClass} ${variantClass} ${className}`; return ( -
+
{title} @@ -53,5 +52,3 @@ const Section: FC = ({
); }; - -export default Section; -- cgit v1.2.3