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

Test(GridLayer): fix 2 pan tests with keepBuffer=0 for graphical browsers #6202

Merged

Conversation

ghybs
Copy link
Collaborator

@ghybs ghybs commented Jun 22, 2018

Following PR #6199, this one fixes the 2 remaining tests that were failing in Firefox (see PR #5831).

These 2 tests were passing in PhantomJS because the latter is not recognized as a 3d enabled browser, hence Leaflet does not perform tiles fade-in animation, which waits at least 1 frame before calling _pruneTiles().
By adding the missing rAF, now Leaflet can prune tiles and the tests pass.

Tested in Firefox and Chromium.

ghybs added 2 commits June 22, 2018 15:57
because graphical browsers, even in non-animated pan, will still perform tile fade-in animation, which require some frames before calling _pruneTiles().
because graphical browsers, even in non-animated pan, will still perform tile fade-in animation, which require some frames before calling _pruneTiles().
@ghybs ghybs requested a review from cherniavskii June 25, 2018 07:50
Copy link
Collaborator

@cherniavskii cherniavskii left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎉

@cherniavskii cherniavskii merged commit 757e7b6 into Leaflet:master Jun 25, 2018
@ghybs ghybs deleted the testGridLayerGraphicalBrowsersPanKeepBuffer branch June 25, 2018 08:58
@ghybs ghybs mentioned this pull request Jun 25, 2018
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.

2 participants