Skip to content

Make tile loading count no longer depend on source count #4794

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

Merged
merged 1 commit into from
Feb 16, 2016

Conversation

ahocevar
Copy link
Member

@ahocevar ahocevar commented Feb 5, 2016

This pull request changes the tile queue behaviour so the number of loading tiles is no longer influenced by the number of sources that request tiles. This should give better user experience in applications with many tile layers.

Fixes #4790.

@ahocevar
Copy link
Member Author

ahocevar commented Feb 5, 2016

@marcjansen, you may want to test this in one of your applications with many tile layers.

@marcjansen
Copy link
Member

I am mostly offline, maybe @weskamm may give this a try

@fredj
Copy link
Member

fredj commented Feb 16, 2016

ping @marcjansen

@marcjansen
Copy link
Member

Sorry I won't have time to review.

@weskamm
Copy link
Contributor

weskamm commented Feb 16, 2016

i also dont have time to test this, sorry

@ahocevar
Copy link
Member Author

This is a change that only affects applications with multiple tile layers. For two layers, like in the wms-tiled.html example, I notice no negative impact on the user experience.

Anyone else up for a review?

@fredj
Copy link
Member

fredj commented Feb 16, 2016

LGTM

@ahocevar
Copy link
Member Author

Thanks @fredj.

ahocevar added a commit that referenced this pull request Feb 16, 2016
Make tile loading count no longer depend on source count
@ahocevar ahocevar merged commit bcf379c into openlayers:master Feb 16, 2016
@ahocevar ahocevar deleted the fixed-tileloading branch February 16, 2016 14:51
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.

4 participants