Merge pull request #9129 from Expensify/jules-fixMinHeightRegression

Fix min hight constant typo
Expensify · May 23, 2022 · a0c8daf · a0c8daf
2 parents 4aadc67 + 92ea2d9
commit a0c8daf
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/pages/signin/SignInPageLayout/index.js b/src/pages/signin/SignInPageLayout/index.js
@@ -62,7 +62,7 @@ const SignInPageLayout = (props) => {
         return content;
     }
 
-    if (props.isMediumScreenWidth && props.windowHeight >= variables.minHeigthToShowGraphics) {
+    if (props.isMediumScreenWidth && props.windowHeight >= variables.minHeightToShowGraphics) {
         return (
             <View style={[styles.dFlex, styles.signInPageInner, styles.flexColumnReverse, styles.justifyContentBetween]}>
                 {graphicLayout}

diff --git a/src/styles/variables.js b/src/styles/variables.js
@@ -46,5 +46,5 @@ export default {
     tooltipzIndex: 10050,
     gutterWidth: 16,
     popoverMenuShadow: '0px 4px 12px 0px rgba(0, 0, 0, 0.06)',
-    minHeightToShowGraphics: 854, // below this height UI was broken on login form layout as there isn't enough height to show forma and graphics.
+    minHeightToShowGraphics: 854, // Login form layout breaks below this height due to insufficient space to show the form and graphics
 };