feat: Allow native div props on InlineNotification (#5070)

cultureamp · Sep 20, 2024 · 9160155 · 9160155
1 parent 218f3d0
commit 9160155
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 3 deletions.
diff --git a/.changeset/silver-cougars-explain.md b/.changeset/silver-cougars-explain.md
@@ -0,0 +1,5 @@
+---
+"@kaizen/components": minor
+---
+
+Allow native div props on InlineNotification
diff --git a/packages/components/src/Notification/InlineNotification/InlineNotification.tsx b/packages/components/src/Notification/InlineNotification/InlineNotification.tsx
@@ -1,4 +1,4 @@
-import React from "react"
+import React, { HTMLAttributes } from "react"
 import classnames from "classnames"
 import { HeadingProps } from "~components/Heading"
 import { OverrideClassName } from "~components/types/OverrideClassName"
@@ -9,7 +9,7 @@ import {
 } from "../subcomponents/GenericNotification"
 import styles from "../subcomponents/GenericNotification/GenericNotification.module.scss"
 
-export type InlineNotificationBase = OverrideClassName<{
+export type InlineNotificationBase = {
   children?: React.ReactNode
   /** @default false */
   persistent?: boolean
@@ -20,7 +20,8 @@ export type InlineNotificationBase = OverrideClassName<{
   forceMultiline?: boolean
   headingProps?: HeadingProps
   isSubtle?: boolean
-}>
+} & Omit<OverrideClassName<HTMLAttributes<HTMLDivElement>>, "style">
+// Omitted `style` above because GenericNotification has its own `style` prop
 
 export type InlineNotificationProps = InlineNotificationBase &
   (GenericNotificationType | GenericNotificationVariant)