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

Takes a while to load dates #67

Open
favianca opened this issue Apr 5, 2021 · 13 comments
Open

Takes a while to load dates #67

favianca opened this issue Apr 5, 2021 · 13 comments
Labels
enhancement New feature or request help wanted Extra attention is needed stale

Comments

@favianca
Copy link

favianca commented Apr 5, 2021

untitled
I am using hermes and polyfill, at first I was using another locale but I changed it back to en but it doesn't seem to fix it. I also thought that it might be the emulator but I have tried on real devices and the problem is still there

@brunohkbx
Copy link

I was about to open an issue for that 😅

This issue is very noticeable in lower-end devices such as iPhone 5s

Example with production bundle
ezgif com-gif-maker (4)

Dev env ( It's taking more than 2s )
Screenshot from 2021-04-02 11-06-57

Is it really necessary to load 4 months at once? wouldn't be better to render only the current one?

@RichardLindhout RichardLindhout added enhancement New feature or request help wanted Extra attention is needed labels Apr 6, 2021
@RichardLindhout
Copy link
Member

Interesting will be a nice case to improve performance since as far as I know I have tried almost every optimisation possible.
I hope React Native Fabric will improve things...

Maybe we can indeed render less on low-end devices but we need to at least render 3 months because it will be too much white when scrolling else.

@brunohkbx
Copy link

@RichardLindhout how about rendering the current month in the first render and then render the other 3 right after?

I think we could try to wrap a function responsible for rendering the other 3 months with InteractionManager.runAfterInteractions

@favianca
Copy link
Author

favianca commented Apr 7, 2021

@brunohkbx hm that is interesting but in my case I was not loading multiple months as it still load when I change to next month

@RichardLindhout
Copy link
Member

I think the rendering needs to te be faster as this will bite back when we scroll down

@RichardLindhout
Copy link
Member

Let's rewrite it in Flutter :-P

@RichardLindhout
Copy link
Member

Maybe we could detect the performance of the device and edit the amount of months loaded, because I also want fast scrolling so maybe it's a good idea to only 1 month on smaller screens

@RichardLindhout
Copy link
Member

Indeed we should not need to load 4 months at once, it also seems like the all 4 month are re-rendered when user scrolls while only new ones should reload

@RichardLindhout
Copy link
Member

RichardLindhout commented Apr 7, 2021

I've improved some things in v0.4.5, I would also recommend using Hermes on Android.

@kendallroth
Copy link

I am using Expo with Hermes (and polyfills) and still seeing very slow loading performance 🤷

@Jeandcc
Copy link

Jeandcc commented Oct 2, 2023

Is it only on my end, ,or does this continue to be an issue?

@AviadBag
Copy link

Well still an issue for me..

@RichardLindhout
Copy link
Member

RichardLindhout commented Oct 26, 2023

I think we could optimize using the native view / text directly. See callstack/react-native-paper#4157 cc: @iM-GeeKy

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed stale
Projects
None yet
Development

No branches or pull requests

7 participants