Skip to content

Commit

Permalink
[FullscreenBar] Consolidate conditional logic (#10173)
Browse files Browse the repository at this point in the history
### WHY are these changes introduced?

Fixes #9937

### WHAT is this pull request doing?

* Consolidate se23 logic and styles for the `FullscreenBar` Component

### How to 🎩
* Compare production and this PR's chromatic storybook to make sure
styles are the same


[Production](https://storybook.polaris.shopify.com/?path=/story/all-components-fullscreenbar--all&globals=polarisSummerEditions2023:true)
[This
PR](https://5d559397bae39100201eedc1-duqojrtsgy.chromatic.com/?path=/story/all-components-fullscreenbar--all)
  • Loading branch information
sophschneider authored Aug 23, 2023
1 parent af9f264 commit b3b6c2e
Showing 1 changed file with 4 additions and 18 deletions.
22 changes: 4 additions & 18 deletions polaris-react/src/components/FullscreenBar/FullscreenBar.scss
Original file line number Diff line number Diff line change
Expand Up @@ -4,37 +4,23 @@
position: relative;
display: flex;
height: $top-bar-height;
box-shadow: var(--p-shadow-sm);
box-shadow: var(--p-shadow-xs);
background-color: var(--p-color-bg);

#{$se23} & {
box-shadow: var(--p-shadow-xs);
}
}

.BackAction {
display: flex;
flex: 0 1 auto;
align-items: center;
padding-left: var(--p-space-3);
padding-right: var(--p-space-3);
padding-left: var(--p-space-4);
padding-right: var(--p-space-4);
border-width: 0;
border-right: var(--p-border-width-1) solid var(--p-color-border-subdued);
background-color: var(--p-color-bg);
font-weight: var(--p-font-weight-medium);
cursor: pointer;

#{$se23} & {
padding-left: var(--p-space-4);
padding-right: var(--p-space-4);
}
}

.BackAction :first-child {
padding-right: var(--p-space-05);

#{$se23} & {
margin-right: var(--p-space-1_5-experimental);
padding-right: 0;
}
margin-right: var(--p-space-1_5-experimental);
}

0 comments on commit b3b6c2e

Please sign in to comment.