Fix popover z-index (#3459)

Skyscanner · May 17, 2024 · f36e401 · f36e401
1 parent 6c41101
commit f36e401
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 6 deletions.
diff --git a/packages/bpk-component-popover/src/BpkPopover.module.scss b/packages/bpk-component-popover/src/BpkPopover.module.scss
@@ -25,13 +25,8 @@
 
 $arrow-size: tokens.bpk-spacing-lg();
 
-.bpk-popover-portal {
+.bpk-popover-container {
   z-index: tokens.$bpk-zindex-popover;
-
-  @include breakpoints.bpk-breakpoint-mobile {
-    margin-right: tokens.bpk-spacing-base();
-    margin-left: tokens.bpk-spacing-base();
-  }
 }
 
 .bpk-popover {

diff --git a/packages/bpk-component-popover/src/BpkPopover.tsx b/packages/bpk-component-popover/src/BpkPopover.tsx
@@ -155,6 +155,7 @@ const BpkPopover = ({
       {isOpenState && (
         <FloatingFocusManager context={context} modal={false}>
           <div
+            className={getClassName('bpk-popover--container')}
             ref={refs.setFloating}
             style={floatingStyles}
             {...getFloatingProps()}