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

Cancel the pending image requests #223

Closed
wants to merge 1 commit into from

Conversation

Anenth
Copy link

@Anenth Anenth commented Mar 23, 2019

When the component gets unmounted the pending request should also get canceled

Before
before

After
after

When the component gets unmounted the pending request should also get canceled
@mbrevda
Copy link
Owner

mbrevda commented Mar 23, 2019

When the component is unmounted the img element is destroyed. Is there anything else that can be done to cancel the request?

@Anenth
Copy link
Author

Anenth commented Mar 24, 2019

@mbrevda, I gave that a try, but it was not canceling the pending request.
Do you find any issues with this change?

@mbrevda mbrevda closed this in 35c155d Mar 24, 2019
@mbrevda
Copy link
Owner

mbrevda commented Mar 24, 2019

My bad - was on mobile and didn't realize this was a PR. Thanks so much for pointing this out! For the record, I have confirmed that the current behavior of deleting the prompt does not cancel the current download.

@Anenth
Copy link
Author

Anenth commented Mar 25, 2019

@mbrevda Awesome thanks!

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.

None yet

2 participants