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

Render HTML Loading #313

Closed
rymesaint opened this issue Jun 4, 2020 · 2 comments
Closed

Render HTML Loading #313

rymesaint opened this issue Jun 4, 2020 · 2 comments
Labels
more-info-needed More information is needed to resolve the issue. Will be closed if no response is received.

Comments

@rymesaint
Copy link

I have some trouble when our customer getting an image from html because their connection is so slow, and cannot load the image on html they think that our app has bug on it. So is there any future update for having html image loading so they know that the image is being loaded.

@ryan-berger
Copy link
Collaborator

@rymesaint I do know that some image loading can be slow, however, this may not be a defect of flutter_html, rather a large image or a poor internet connection.

Currently, there is a PR open that should allow you to choose an image provider, and if I end up liking it and finding it is good to go, you may be able to download the images ahead of time, so that your html can render it locally.

If this is entirely a poor connection or a large image issue, I recommend that you compress the image so that it will decrease the load time.

If none of this applies, please reply with an example of an image that is being slow, along with the expected behavior, and I'll see what I can do.

@ryan-berger ryan-berger added the more-info-needed More information is needed to resolve the issue. Will be closed if no response is received. label Jun 11, 2020
@erickok
Copy link
Collaborator

erickok commented Feb 8, 2021

We are now showing a loading spinner when the image is loading. More advanced loading states can be added later, if still there is a need for it, after #505 has been merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
more-info-needed More information is needed to resolve the issue. Will be closed if no response is received.
Projects
None yet
Development

No branches or pull requests

3 participants