[COR-481][BpkDrawer] Removing header border (#3670)

* Removing header border * Restore story * Restoring className --------- Co-authored-by: tristan-grayford <[email protected]> Co-authored-by: Amber Shand <[email protected]>
Skyscanner · Nov 14, 2024 · 5d0a925 · 5d0a925
1 parent 60def25
commit 5d0a925
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 6 deletions.
diff --git a/packages/bpk-component-drawer/src/BpkDrawer.tsx b/packages/bpk-component-drawer/src/BpkDrawer.tsx
@@ -89,7 +89,8 @@ const BpkDrawer = ({
           dialogRef={dialogRef}
           closeLabel={closeLabel || ""}
           closeText={closeText}
-          additionalClassName={className}
+          // eslint-disable-next-line @skyscanner/rules/forbid-component-props
+          className={className}
           contentClassName={contentClassName}
           getApplicationElement={getApplicationElement}
           hideTitle={hideTitle}

diff --git a/packages/bpk-component-drawer/src/BpkDrawerContent.module.scss b/packages/bpk-component-drawer/src/BpkDrawerContent.module.scss
@@ -80,8 +80,6 @@
     justify-content: space-between;
     align-items: center;
     flex: 0 0;
-
-    @include borders.bpk-border-bottom-sm(tokens.$bpk-line-day);
   }
 
   &__heading {

diff --git a/packages/bpk-component-drawer/src/BpkDrawerContent.tsx b/packages/bpk-component-drawer/src/BpkDrawerContent.tsx
@@ -16,7 +16,7 @@
  * limitations under the License.
  */
 
-import type { ReactNode } from 'react';
+import type { ReactNode, RefObject } from 'react';
 
 import { Transition } from 'react-transition-group';
 
@@ -34,12 +34,12 @@ const getClassName = cssModules(STYLES);
 
 type Props = {
   children: ReactNode,
-  dialogRef: () => React.RefObject<HTMLElement>,
+  dialogRef: () => RefObject<HTMLElement>,
   onCloseAnimationComplete: () => void,
   onClose: () => void
   id: string,
   title: string,
-  className?: string,
+  className?: string | null,
   contentClassName?: string,
   closeLabel?: string,
   closeText?: string,