Skip to content
This repository has been archived by the owner on Mar 3, 2023. It is now read-only.

speed start time about 80ms #4273

Closed
wants to merge 3 commits into from
Closed

Conversation

yongkangchen
Copy link

cause first of require 'react-atom-fork' cost about 80ms

cause first of require 'react-atom-fork' cost about 80ms
@kevinsawicki
Copy link
Contributor

This will only speed things up when no editors are opened right?

Then when the first editor is opened that 80ms require time will occur?

@yongkangchen
Copy link
Author

Yes. It speed window load time, and window shown faster is a kind of user experience, make user feel faster.

@bthusby
Copy link

bthusby commented Dec 30, 2014

All speed improvement is needed! Who can resolve the merge conflicts? Is this the same fix as mentioned in another pull request? #4269

@yongkangchen
Copy link
Author

@bthusby see #4269 and #4293, so this pull request is expired.

@bthusby
Copy link

bthusby commented Dec 31, 2014

Ok, just noticed it was still open :) Thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants