Inconsistent results

[bobsmits]

With the latest patch and the fix for notched devices i get mixed results. I want the available screen height with a translucent statusbar. So that should be the actual screen height minus the softmenu bar.

Samsung s10
Has a soft menu.
ExtraDimensions.getSoftMenuBarHeight( ) -> 48 -> seems ok
ExtraDimensions.isSoftMenuBarEnabled() -> false -> not ok
ExtraDimensions.getRealWindowHeight() -> gives the device height - soft menu bar height

Samsung J5
Has no soft menu
ExtraDimensions.getSoftMenuBarHeight( ) -> 0 -> seems ok
ExtraDimensions.isSoftMenuBarEnabled() -> true -> not ok
ExtraDimensions.getRealWindowHeight() -> gives the device height

Simulator Nexus 5x
Has a soft menu
ExtraDimensions.getSoftMenuBarHeight( ) -> 0 -> not ok
ExtraDimensions.isSoftMenuBarEnabled() -> true -> seems ok
ExtraDimensions.getRealWindowHeight() -> gives the device height

I'm not a native dev otherwise i would dive in and try to fix it myself.

[gongdao123]

According to the source:

react-native-extra-dimensions-android/android/src/main/java/ca/jaysoo/extradimensions/ExtraDimensionsModule.java

Line 87 in 0d449d2

private boolean hasPermanentMenuKey() {

currently, all of the simulator's SoftMenuBarHeight will be 0, since the simulator's soft menu bar is controlled by qemu.hw.mainkeys instead of config_showNavigationBar

And I create a PR to fix it: #54

[countravioli]

@gongdao123 not sure if you would know this. I setup with Genymotion to try and test devices like Pixel3 more accurately since this PR is not part of the project yet. I was trying to test in the simulator provided by studio but with this bug, obviously that is hard. Do you think the same issue would play Genymotion emulators in addition to the regular ones?

[ManigandanRaamanathan]

still facing same issue

[ManigandanRaamanathan]

@bobsmits @gongdao123 i'm trying in samsung s10 Plus. still same issue