From 832b6c4f42437138ec58b8e509f982c16a16f7f6 Mon Sep 17 00:00:00 2001 From: Sophie Schneider Date: Fri, 11 Aug 2023 12:58:15 -0400 Subject: [PATCH] [BulkActions] Consolidate se23 logic and styles (#10008) Fixes https://github.com/Shopify/polaris/issues/9921 Consolidates se23 beta styles for `BulkActions`. Compare the bulk actions shadow bevel (make sure they are the same): * This PR's [Storybook](https://5d559397bae39100201eedc1-vxezcfotko.chromatic.com/?path=/story/all-components-indextable--default) * [Production Storybook](https://storybook.polaris.shopify.com/?path=/story/all-components-indextable--default&globals=polarisSummerEditions2023:true) --- .../src/components/BulkActions/BulkActions.scss | 13 +++++-------- 1 file changed, 5 insertions(+), 8 deletions(-) diff --git a/polaris-react/src/components/BulkActions/BulkActions.scss b/polaris-react/src/components/BulkActions/BulkActions.scss index c9fcc6964b8..2bebc2813c4 100644 --- a/polaris-react/src/components/BulkActions/BulkActions.scss +++ b/polaris-react/src/components/BulkActions/BulkActions.scss @@ -52,20 +52,17 @@ $bulk-actions-button-stacking-order: ( .ButtonGroupWrapper { width: auto; justify-content: flex-start; - padding: var(--p-space-4); + padding: var(--p-space-3); background: var(--p-color-bg); border-radius: var(--p-border-radius-2); box-shadow: var(--p-shadow-md), var(--p-shadow-xl); max-width: 100%; pointer-events: auto; - #{$se23} & { - padding: var(--p-space-3); - @include shadow-bevel( - $boxShadow: var(--p-shadow-md), - $borderRadius: var(--p-border-radius-3) - ); - } + @include shadow-bevel( + $boxShadow: var(--p-shadow-md), + $borderRadius: var(--p-border-radius-3) + ); @media #{$p-breakpoints-sm-down} { // stylelint-disable-next-line selector-max-combinators, selector-max-type -- the first item of button group on small screen needs to fill the space