Skip to content

Commit

Permalink
[BulkActions] Consolidate se23 logic and styles (#10008)
Browse files Browse the repository at this point in the history
### WHY are these changes introduced?

Fixes #9921

### WHAT is this pull request doing?

Consolidates se23 beta styles for `BulkActions`.

### How to 🎩

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 Aug 11, 2023
1 parent d901d92 commit 4004305
Showing 1 changed file with 4 additions and 6 deletions.
10 changes: 4 additions & 6 deletions polaris-react/src/components/BulkActions/BulkActions.scss
Original file line number Diff line number Diff line change
Expand Up @@ -59,12 +59,10 @@ $bulk-actions-button-stacking-order: (
max-width: 100%;
pointer-events: auto;

#{$se23} & {
@include shadow-bevel(
$boxShadow: var(--p-shadow-md),
$borderRadius: var(--p-border-radius-2)
);
}
@include shadow-bevel(
$boxShadow: var(--p-shadow-md),
$borderRadius: var(--p-border-radius-2)
);

@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 4004305

Please sign in to comment.