From 953ff45085564fd946cc5a38901234aa0ba49dd2 Mon Sep 17 00:00:00 2001
From: Jed Fox <git@jedfox.com>
Date: Mon, 3 Jul 2023 22:06:22 -0400
Subject: [PATCH] Fix layout of the management app with the demo bar in place
 (#1267)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The banner in #1229 pushes the bottom bar items (including version &
“change server URL” button) out of view on Netlify builds.
---
 packages/desktop-client/src/components/App.js | 63 ++++++++++---------
 upcoming-release-notes/1267.md                |  6 ++
 2 files changed, 39 insertions(+), 30 deletions(-)
 create mode 100644 upcoming-release-notes/1267.md

diff --git a/packages/desktop-client/src/components/App.js b/packages/desktop-client/src/components/App.js
index e4e963463..19e2ea5f7 100644
--- a/packages/desktop-client/src/components/App.js
+++ b/packages/desktop-client/src/components/App.js
@@ -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>
     );
diff --git a/upcoming-release-notes/1267.md b/upcoming-release-notes/1267.md
new file mode 100644
index 000000000..5229b1f7f
--- /dev/null
+++ b/upcoming-release-notes/1267.md
@@ -0,0 +1,6 @@
+---
+category: Bugfix
+authors: [j-f1]
+---
+
+Fix the “Change server” button being invisible on Netlify deploy previews
-- 
GitLab