Skip to content

Commit

Permalink
Merge pull request #5160 from Expensify/revert-5032-fix/emojipicker-l…
Browse files Browse the repository at this point in the history
…anscape-modal-size
  • Loading branch information
roryabraham authored Sep 9, 2021
2 parents bc035f5 + 32f1fc5 commit 16e5463
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 3 deletions.
1 change: 0 additions & 1 deletion src/components/withWindowDimensions.js
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,6 @@ export default function (WrappedComponent) {
const {window} = newDimensions;
const isSmallScreenWidth = window.width <= variables.mobileResponsiveWidthBreakpoint;
const isMediumScreenWidth = !isSmallScreenWidth && window.width <= variables.mediumScreenResponsiveWidthBreakpoint;

this.setState({
windowHeight: window.height,
windowWidth: window.width,
Expand Down
3 changes: 1 addition & 2 deletions src/styles/getModalStyles.js
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ import variables from './variables';
import themeColors from './themes/default';

export default (type, windowDimensions, popoverAnchorPosition = {}) => {
const {isSmallScreenWidth, windowWidth, windowHeight} = windowDimensions;
const {isSmallScreenWidth, windowWidth} = windowDimensions;

let modalStyle = {
margin: 0,
Expand Down Expand Up @@ -131,7 +131,6 @@ export default (type, windowDimensions, popoverAnchorPosition = {}) => {
paddingTop: 12,
justifyContent: 'center',
overflow: 'hidden',
...windowWidth >= windowHeight ? {top: 40} : {},
};

shouldAddBottomSafeAreaPadding = true;
Expand Down

0 comments on commit 16e5463

Please sign in to comment.