scrollIntoView can fail on android due to View collapsing

[slorber]

See facebook/react-native#3282 (comment)

the newly added _measureElement method can actually return nothing if view collapse on Android.

In such case I think we should warn the user and recommend him to use collapsed={false} to ensure measures can be taken.

cc @sebasgarcep

[slorber]

Also noticed that on Android, taking measures on mount return bad measures. It looks like adding setTimeout before measure ensures to get better result.

[sebasgarcep]

@slorber For some reason I never saw this issue in my notifications. Did you solve this issue?

[slorber]

Hi,

I actually use https://github.com/slorber/react-native-scroll-into-view and the api design prevent this issue:

https://github.com/slorber/react-native-scroll-into-view/blob/master/scrollIntoView.js#L409

[sebasgarcep]

@slorber Could you send a pull request with this change and some documentation on why it needs to work this way?

[slorber]

hi @sebasgarcep

I don't remember exactly what the problem was, but I think if we try to measure an empty view that collapse, it returns nothing. This is probably an edge case, not worth to be highlighted in main readme, but the user can find this info on this issue

      const [parentLayout, childLayout] = await Promise.all([
        this._measureElement(this._rnkasv_keyboardView),
        this._measureElement(element)
      ])

One idea would be to throw if childLayout is null/undefined here, with a link to this issue