Skip to content

Commit

Permalink
Merge pull request #36626 from rayane-djouah/replace-usages-of-isInMo…
Browse files Browse the repository at this point in the history
…dal-and-shouldShowSmallScreen-props-in-components

Fix: Login - Expensify logo is too close to the top of the screen
  • Loading branch information
puneetlath authored Feb 20, 2024
2 parents a21cfbc + ce3e99e commit 669ac33
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/pages/signin/SignInPage.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -128,7 +128,7 @@ function SignInPageInner({credentials, account, activeClients = [], preferredLoc
const styles = useThemeStyles();
const StyleUtils = useStyleUtils();
const {translate, formatPhoneNumber} = useLocalize();
const {shouldUseNarrowLayout} = useResponsiveLayout();
const {shouldUseNarrowLayout, isInModal} = useResponsiveLayout();
const safeAreaInsets = useSafeAreaInsets();
const signInPageLayoutRef = useRef<SignInPageLayoutRef>(null);
const loginFormRef = useRef<InputHandle>(null);
Expand Down Expand Up @@ -246,7 +246,7 @@ function SignInPageInner({credentials, account, activeClients = [], preferredLoc
// Bottom SafeAreaView is removed so that login screen svg displays correctly on mobile.
// The SVG should flow under the Home Indicator on iOS.
<View
style={[styles.signInPage, StyleUtils.getSafeAreaPadding({...safeAreaInsets, bottom: 0, top: shouldUseNarrowLayout ? 0 : safeAreaInsets.top}, 1)]}
style={[styles.signInPage, StyleUtils.getSafeAreaPadding({...safeAreaInsets, bottom: 0, top: isInModal ? 0 : safeAreaInsets.top}, 1)]}
testID={SignInPageInner.displayName}
>
<SignInPageLayout
Expand Down

0 comments on commit 669ac33

Please sign in to comment.