Skip to content

Commit

Permalink
Merge pull request Expensify#25457 from bernhardoj/fix/24328-sign-in-…
Browse files Browse the repository at this point in the history
…bg-overflow

Fix able to scroll horizontally out of screen width on sign in page
  • Loading branch information
srikarparsi authored Aug 18, 2023
2 parents a7b5505 + a5c85d5 commit 488d34a
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
5 changes: 3 additions & 2 deletions src/pages/signin/SignInPageLayout/Footer.js
Original file line number Diff line number Diff line change
Expand Up @@ -157,11 +157,12 @@ function Footer(props) {
const pageFooterWrapper = [styles.footerWrapper, imageDirection, imageStyle, isVertical ? styles.pl10 : {}];
const footerColumns = [styles.footerColumnsContainer, columnDirection];
const footerColumn = isVertical ? [styles.p4] : [styles.p4, props.isMediumScreenWidth ? styles.w50 : styles.w25];
const footerWrapper = isVertical ? [StyleUtils.getBackgroundColorStyle(themeColors.signInPage), styles.overflowHidden] : [];

return (
<View style={[styles.flex1]}>
<View style={[props.isSmallScreenWidth ? StyleUtils.getBackgroundColorStyle(themeColors.signInPage) : {}]}>
{props.isSmallScreenWidth ? (
<View style={footerWrapper}>
{isVertical ? (
<View style={[styles.signInPageGradientMobile]}>
<SignInGradient height="100%" />
</View>
Expand Down
2 changes: 1 addition & 1 deletion src/pages/signin/SignInPageLayout/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -128,7 +128,7 @@ function SignInPageLayout(props) {
keyboardShouldPersistTaps="handled"
ref={scrollViewRef}
>
<View style={[styles.flex1, styles.flexColumn, StyleUtils.getMinimumHeight(Math.max(variables.signInContentMinHeight, containerHeight))]}>
<View style={[styles.flex1, styles.flexColumn, styles.overflowHidden, StyleUtils.getMinimumHeight(Math.max(variables.signInContentMinHeight, containerHeight))]}>
<BackgroundImage
isSmallScreen
pointerEvents="none"
Expand Down

0 comments on commit 488d34a

Please sign in to comment.