refactor(CHeader): improve and clean-up syntax

mrholek · mrholek · commit 253a3775812a · 2023-03-20T18:40:49.000+01:00
diff --git a/packages/coreui-react/src/components/header/CHeader.tsx b/packages/coreui-react/src/components/header/CHeader.tsx
@@ -19,22 +19,19 @@ export interface CHeaderProps extends HTMLAttributes<HTMLDivElement> {
 
 export const CHeader = forwardRef<HTMLDivElement, CHeaderProps>(
   ({ children, className, container, position, ...rest }, ref) => {
-    let content
-    if (container) {
-      content = (
-        <div className={`container${container !== true ? '-' + container : ''}`}>{children}</div>
-      )
-    } else {
-      content = children
-    }
-
     return (
       <div
         className={classNames('header', { [`header-${position}`]: position }, className)}
         {...rest}
         ref={ref}
       >
-        {content}
+        {container ? (
+          <div className={typeof container === 'string' ? `container-${container}` : 'container'}>
+            {children}
+          </div>
+        ) : (
+          <>{children}</>
+        )}
       </div>
     )
   },
diff --git a/packages/coreui-react/src/components/header/CHeaderToggler.tsx b/packages/coreui-react/src/components/header/CHeaderToggler.tsx
@@ -13,7 +13,7 @@ export const CHeaderToggler = forwardRef<HTMLButtonElement, CHeaderTogglerProps>
   ({ children, className, ...rest }, ref) => {
     return (
       <button type="button" className={classNames('header-toggler', className)} {...rest} ref={ref}>
-        {children ? children : <span className="header-toggler-icon"></span>}
+        {children ?? <span className="header-toggler-icon"></span>}
       </button>
     )
   },

Original file line number	Diff line number	Diff line change
`@@ -13,7 +13,7 @@ export const CHeaderToggler = forwardRef<HTMLButtonElement, CHeaderTogglerProps>`
`13`	`13`	`({ children, className, ...rest }, ref) => {`
`14`	`14`	`return (`
`15`	`15`	`<button type="button" className={classNames('header-toggler', className)} {...rest} ref={ref}>`
`16`		`- {children ? children : <span className="header-toggler-icon"></span>}`
	`16`	`+ {children ?? <span className="header-toggler-icon"></span>}`
`17`	`17`	`</button>`
`18`	`18`	`)`
`19`	`19`	`},`