Merge pull request #6670 from Expensify/Rory-FixiPadProLayout

[CP Stg] Fix iPad pro SignInPageLayout
Expensify · Dec 9, 2021 · 7708155 · 7708155
2 parents 9efb4d5 + 07306f3
commit 7708155
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 2 deletions.
diff --git a/src/components/withWindowDimensions.js b/src/components/withWindowDimensions.js
@@ -71,7 +71,7 @@ export default function (WrappedComponent) {
         onDimensionChange(newDimensions) {
             const {window} = newDimensions;
             const isSmallScreenWidth = window.width <= variables.mobileResponsiveWidthBreakpoint;
-            const isMediumScreenWidth = !isSmallScreenWidth && window.width <= variables.mediumScreenResponsiveWidthBreakpoint;
+            const isMediumScreenWidth = !isSmallScreenWidth && window.width <= variables.tabletResponsiveWidthBreakpoint;
             this.setState({
                 windowHeight: window.height,
                 windowWidth: window.width,

diff --git a/src/pages/signin/SignInPage.js b/src/pages/signin/SignInPage.js
@@ -85,7 +85,6 @@ class SignInPage extends Component {
                 <SignInPageLayout
                     welcomeText={welcomeText}
                     shouldShowWelcomeText={showLoginForm || showPasswordForm || !showResendValidationLinkForm}
-                    shouldShowWelcomeScreenshot={showLoginForm}
                 >
                     {/* LoginForm and PasswordForm must use the isVisible prop. This keeps them mounted, but visually hidden
                     so that password managers can access the values. Conditionally rendering these components will break this feature. */}