scroll to new position if swipe container size chages (ex: orientatio…

[trexpert]

The scrollTo method should not be called only on initial render. onLayout invoke can be caused by an orientation change or swipe container resize. So whenever we have got slides inside we should consider to call scrollTo

Is it a bugfix ?

Yes (No ticket # yet)
Bug can be tested on Android tablet (Galaxy tab A). Just make a fullscreen swiper fill with some colored boxes and move to a slideindex more then 0. Then change the orientation. The slider will stack in between 2 slides and not adjust it's scroll position.

Is it a new feature ?

No

Describe what you've done:

Just moved the scrollTo method outside the initialRender method. I think there is no need for the initialRender flag anymore

How to test it ?

See Bugfix

[Rotemy]

This is working perfectly, I just tested it on Android and iOS and it should be merged.

[Rotemy]

@trexpert
Btw, i'm working with old Android devices and for them I had to surround the scrollTo with setTimeout 0. Like that:

setTimeout(() => {
        this.scrollView.scrollTo({ ...offset, animated: false })
      }, 0);

[trexpert]

just fine

[trexpert]

);

i guess the project maintainer is not interested in pull requests.... staying open for half a year now...

[Rotemy]

@trexpert yeah.. now I'll need to do a patch in order to get this fix.

[Rotemy]

@ArrayZoneYour @trexpert you need to surround it with setTimeout 0 to make it work in slow old devices