Skip to content

Commit

Permalink
Merge pull request #595 from dulithsenanayake/alpha-banner-display-is…
Browse files Browse the repository at this point in the history
…sue-fix

Fix UI issues when enable banners
  • Loading branch information
dulithsenanayake authored Mar 12, 2024
2 parents 035221d + 0f07043 commit da595ac
Showing 1 changed file with 4 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -134,6 +134,7 @@ const Root = styled('div')((
position: 'relative',
minHeight: theme.custom.banner.active ? `calc(100vh - ${64 + footerHeight}px)` : `calc(100vh - ${footerHeight}px)`,
marginLeft: -4,
marginTop: theme.custom.banner.active ? 0 : '64px',
},
[`& .${classes.push}`]: {
height: 50,
Expand Down Expand Up @@ -192,7 +193,7 @@ const Root = styled('div')((
position: 'fixed',
width: '100%',
boxSizing: 'border-box',
zIndex: 1,
zIndex: 1000,
},
[`& .${classes.listRoot}`]: {
padding: 0,
Expand Down Expand Up @@ -503,7 +504,7 @@ class LayoutLegacy extends React.Component {
position='fixed'
className={classes.appBar}
id='appBar'
style={{ top: active ? this.state.bannerHeight + 'px' : 'unset' }}
style={{ top: active ? this.state.bannerHeight + 'px' : 0 }}
>
<Toolbar className={classes.toolbar} id='toolBar'>
<Hidden mdUp>
Expand Down Expand Up @@ -686,7 +687,7 @@ class LayoutLegacy extends React.Component {
</Toolbar>
</AppBar>
<main>
<div className={classes.contentWrapper} style={{ paddingTop: active ? 'unset' : '64px' }}>{children}</div>
<div className={classes.contentWrapper}>{children}</div>
</main>
{footerActive && <div className={classes.push} />}
</div>
Expand Down

0 comments on commit da595ac

Please sign in to comment.