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

Implement asynchronous loading #2047

Open
1 of 2 tasks
rajvaibhavdubey opened this issue Apr 4, 2019 · 3 comments
Open
1 of 2 tasks

Implement asynchronous loading #2047

rajvaibhavdubey opened this issue Apr 4, 2019 · 3 comments

Comments

@rajvaibhavdubey
Copy link
Member

I'm submitting a ...

  • bug report
  • feature request

Description :

Loading asynchronously will prevent any delay in pageload because the page will attempt to load all elements simultaneously,

Current behaviour:

Currently the pages are loaded synchronously

Would you like to work on the issue?

Yes

@rajvaibhavdubey
Copy link
Member Author

@yashLadha @gabru-md @ParthS007 Discussion needed.

@abishekvashok
Copy link
Member

Async loading of all elements is not a good practice. Images are loaded asynchroneously, large images therefore appear slowly on slow connections. Many simulataneous connections to load Dom can also ruin the webpage experience. Therefore I propose we close this.

@yashLadha
Copy link
Member

It is not a good practice to transfer images through network though, instead, thumbnails and SVG would be a better approach for rendering images. Secondly initiating several parallel loads at the same time, so this problem is solvable by maintaining a thread pool in the browser and limiting the number of threads. A queue can be maintained which takes care of the large number and the images should be thumbnails or SVG for better complying with the logic.

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

No branches or pull requests

4 participants