Fixed issue of incorrectly getting the height/width on Android devices.#224
Fixed issue of incorrectly getting the height/width on Android devices.#224ifsnow wants to merge 1 commit intoreact-native-modal:masterfrom
Conversation
|
Hey! |
|
@mmazzarolo I thought so. But, You can see that the logic of react-native-extra-dimensions-android and react-native is the same and RN provides Dimensions.get("screen").height == require("react-native-extra-dimensions-android").get("REAL_WINDOW_HEIGHT")I have already checked if this version works as intended on various Android devices. |
|
I can confirm it too be working on the POC branch where the modals had space at the bottom. |
|
@RichardLindhout do you mind do a quick check to see if this solution works as well? |
|
I tested this branch and I'm still seeing an error. |
|
@ifsnow I know I'm late, thanks again for submitting the PR but I'm closing it because we're still investigating which one of |
I found RN has 2 metrics(window/screen) in
Dimensions.I don't exactly know why these exist. It makes people feel confused.
react-native-extra-dimensions-androidusesscreenvalues. so, Ifscreenvalues are used internally, we do not need to use any other library and pass the props. This patched version works well on the device where the problem occurred.I hope this helps.