From 37cd226cd9baa475ca7440fa2dfb496db9922312 Mon Sep 17 00:00:00 2001 From: vicky-comeau Date: Fri, 4 Oct 2024 16:00:10 -0400 Subject: [PATCH] Fix: [DS-388] Scrollbar positions is now fixed --- .changeset/early-crews-sneeze.md | 5 +++++ .../components/src/overlays/Popover/src/Popover.module.css | 5 +++-- 2 files changed, 8 insertions(+), 2 deletions(-) create mode 100644 .changeset/early-crews-sneeze.md diff --git a/.changeset/early-crews-sneeze.md b/.changeset/early-crews-sneeze.md new file mode 100644 index 000000000..650c8a63c --- /dev/null +++ b/.changeset/early-crews-sneeze.md @@ -0,0 +1,5 @@ +--- +"@hopper-ui/components": patch +--- + +Removed extra padding around select and combo box popovers so that the scrollbar would be all the way to the right. diff --git a/packages/components/src/overlays/Popover/src/Popover.module.css b/packages/components/src/overlays/Popover/src/Popover.module.css index 786c4ece0..cc4fd141f 100644 --- a/packages/components/src/overlays/Popover/src/Popover.module.css +++ b/packages/components/src/overlays/Popover/src/Popover.module.css @@ -77,7 +77,7 @@ --hop-Popover-column-gap: var(--hop-space-inset-sm); --hop-Popover-row-gap: var(--hop-space-stack-sm); --hop-Popover-dialog-padding: var(--hop-space-inset-md); - --hop-Popover-dropdown-padding: var(--hop-space-inset-sm); + --hop-Popover-dropdown-padding: 0; --hop-Popover-title-font-family: var(--hop-body-sm-font-family); --hop-Popover-title-font-weight: var(--hop-body-sm-semibold-font-weight); --hop-Popover-content-font-size: var(--hop-body-sm-font-size); @@ -129,6 +129,8 @@ --footer-place-self: stretch; --grid-template: var(--hop-Popover-dropdown-grid-template); --padding: var(--hop-Popover-dropdown-padding); + + overflow: hidden; } .hop-Popover__dialog { @@ -160,7 +162,6 @@ min-inline-size: auto; max-inline-size: none; max-block-size: var(--hop-Popover-list-box-dropdown-max-height); - padding: 0; border: none; }