Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed the rendering delays bug when app is first opened #238

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nketia
Copy link

@nketia nketia commented Mar 24, 2021

There was about 5 seconds delay rendering the app-intro-slider when an app is first launched when tested on iOS. This delay wasn't happening in the earlier version of 2.0.1. This pull request is to fix that initial delay.

@hkar19
Copy link

hkar19 commented Mar 31, 2021

is it better to have width and height initially in state rather than on onLayout?

@nketia
Copy link
Author

nketia commented Apr 1, 2021

is it better to have width and height initially in state rather than on onLayout?

Yes. Having the width and height set in the state to window figures solves the issue.

@hkar19
Copy link

hkar19 commented Apr 5, 2021

i think due to this issue #224 , it is better to set width and height early to Dimensions.get('screen') values and keep setting width and height to state on onLayout,

as it will potentially break the goToSlide function. concerning line.

@hkar19
Copy link

hkar19 commented Apr 5, 2021

#237

hkar19 added a commit to hkar19/react-native-app-intro-slider that referenced this pull request Apr 5, 2021
@hkar19 hkar19 mentioned this pull request Apr 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants