kylefang/react-native
1
0
Fork 0
mirror of https://github.com/zhigang1992/react-native.git synced 2026-03-29 07:48:17 +08:00
Code Issues Packages Projects Releases Wiki Activity

Fix Modal size on Android tablet

Browse Source 
Summary:
On tablets, using Display.getRotation() returned ROTATION_0 or ROTATION_180 when it was in landscape, not portrait as it does on phones. This resulted in the Modal being sized incorrectly on tablets. Using size and comparing width and height is the only way I can think of to figure out the device orientation and give the modal the correct size. With this change, all issues listed in #7708 should be resolved.

**Test plan**
Modal should correctly fill screen on Android phone and tablet in both portrait and landscape.
Closes https://github.com/facebook/react-native/pull/10159

Differential Revision: D3950369

Pulled By: andreicoman11

fbshipit-source-id: 9488c4302a76cc48e4f8a4026eb5770d40b6e3d2
This commit is contained in:
Dave Pack
2016-09-30 02:29:21 -07:00
committed by Facebook Github Bot
parent b1fdac47b5
commit 12a97e1ecd
1 changed files with 7 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
ReactAndroid/src/main/java/com/facebook/react/views/modal/ModalHostHelper.java
View File

@@ -6,7 +6,6 @@ import android.annotation.TargetApi;
import android.content.Context;
import android.graphics.Point;
import android.view.Display;
import android.view.Surface;
import android.view.WindowManager;
import com.facebook.infer.annotation.Assertions;
@@ -18,12 +17,16 @@ import com.facebook.infer.annotation.Assertions;
private static final Point MIN_POINT = new Point();
private static final Point MAX_POINT = new Point();
private static final Point SIZE_POINT = new Point();
/**
* To get the size of the screen, we use information from the WindowManager and
* default Display. We don't use DisplayMetricsHolder, or Display#getSize() because
* they return values that include the status bar. We only want the values of what
* will actually be shown on screen.
* We use Display#getSize() to determine if the screen is in portrait or landscape.
* We don't use getRotation because the 'natural' rotation will be portrait on phones
* and landscape on tablets.
* This should only be called on the native modules/shadow nodes thread.
*/
@TargetApi(16)
@@ -32,9 +35,10 @@ import com.facebook.infer.annotation.Assertions;
Display display = Assertions.assertNotNull(wm).getDefaultDisplay();
// getCurrentSizeRange will return the min and max width and height that the window can be
display.getCurrentSizeRange(MIN_POINT, MAX_POINT);
// getSize will return the dimensions of the screen in its current orientation
display.getSize(SIZE_POINT);
final int rotation = display.getRotation();
if (rotation == Surface.ROTATION_0 || rotation == Surface.ROTATION_180) {
if (SIZE_POINT.x < SIZE_POINT.y) {
// If we are vertical the width value comes from min width and height comes from max height
return new Point(MIN_POINT.x, MAX_POINT.y);
} else {