Skip to content

Commit

Permalink
[BulkActions] Consolidate se23 logic and styles (#10008)
Browse files Browse the repository at this point in the history
Fixes #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)
  • Loading branch information
sophschneider authored and sam-b-rose committed Aug 15, 2023
1 parent de5d297 commit 4f86e60
Showing 1 changed file with 5 additions and 8 deletions.
13 changes: 5 additions & 8 deletions polaris-react/src/components/BulkActions/BulkActions.scss
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down

0 comments on commit 4f86e60

Please sign in to comment.