Skip to content
Snippets Groups Projects
Unverified Commit 953ff450 authored by Jed Fox's avatar Jed Fox Committed by GitHub
Browse files

Fix layout of the management app with the demo bar in place (#1267)

The banner in #1229 pushes the bottom bar items (including version &
“change server URL” button) out of view on Netlify builds.
parent 2ebea847
No related branches found
No related tags found
No related merge requests found
......@@ -94,42 +94,45 @@ class App extends Component {
return (
<ResponsiveProvider>
<div
key={hiddenScrollbars ? 'hidden-scrollbars' : 'scrollbars'}
{...css([
{
height: '100%',
backgroundColor: '#E8ECF0',
overflow: 'hidden',
},
styles.lightScrollbar,
])}
style={{ height: '100%', display: 'flex', flexDirection: 'column' }}
>
{process.env.REACT_APP_REVIEW_ID && <DevelopmentTopBar />}
{fatalError ? (
<>
<AppBackground />
<FatalError error={fatalError} buttonText="Restart app" />
</>
) : initializing ? (
<AppBackground
initializing={initializing}
loadingText={loadingText}
/>
) : budgetId ? (
<FinancesApp />
) : (
<>
<div
key={hiddenScrollbars ? 'hidden-scrollbars' : 'scrollbars'}
{...css([
{
flexGrow: 1,
backgroundColor: '#E8ECF0',
overflow: 'hidden',
},
styles.lightScrollbar,
])}
>
{fatalError ? (
<>
<AppBackground />
<FatalError error={fatalError} buttonText="Restart app" />
</>
) : initializing ? (
<AppBackground
initializing={initializing}
loadingText={loadingText}
/>
<ManagementApp isLoading={loadingText != null} />
</>
)}
<UpdateNotification />
<MobileWebMessage />
) : budgetId ? (
<FinancesApp />
) : (
<>
<AppBackground
initializing={initializing}
loadingText={loadingText}
/>
<ManagementApp isLoading={loadingText != null} />
</>
)}
<UpdateNotification />
<MobileWebMessage />
</div>
</div>
</ResponsiveProvider>
);
......
---
category: Bugfix
authors: [j-f1]
---
Fix the “Change server” button being invisible on Netlify deploy previews
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment