Skip to content

Commit

Permalink
fix: ui banner covering sidebar (argoproj#11101)
Browse files Browse the repository at this point in the history
  • Loading branch information
saumeya authored Nov 25, 2022
1 parent 9b6992a commit ba8931b
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
2 changes: 1 addition & 1 deletion ui/src/app/ui-banner/ui-banner.scss
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
.ui-banner {
background: $argo-color-gray-5;
position: fixed;
width: 100%;
width: -webkit-fill-available;
z-index: 10;
text-align: center;
color: $argo-color-teal-8;
Expand Down
3 changes: 2 additions & 1 deletion ui/src/app/ui-banner/ui-banner.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,7 @@ export const Banner = (props: React.Props<any>) => {
position: string;
}) => {
const heightOfBanner = permanent ? '28px' : '70px';
const leftOffset = prefs.hideSidebar ? '60px' : '230px';
let show = false;
if (!content || content === '' || content === null) {
if (prefs.hideBannerContent) {
Expand All @@ -68,7 +69,7 @@ export const Banner = (props: React.Props<any>) => {
}
return (
<React.Fragment>
<div className={combinedBannerClassName} style={{visibility: show ? 'visible' : 'hidden', height: heightOfBanner}}>
<div className={combinedBannerClassName} style={{visibility: show ? 'visible' : 'hidden', height: heightOfBanner, left: leftOffset}}>
<div className='ui-banner-text' style={{maxHeight: permanent ? '25px' : '50px'}}>
{url !== undefined ? (
<a href={url} target='_blank' rel='noopener noreferrer'>
Expand Down

0 comments on commit ba8931b

Please sign in to comment.