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

Fix #1228 safari partially rendering #1302

Closed

Conversation

nicolaiwolko
Copy link

Problem

See issue 1228 (#1228)
Safari sometimes only partially renders the image layer.

Solution

The bug is related to the leaflet library.
Safari seems to have problems decoding too many images in main thread.
I would suggest leaflet maintainers to switch from img to canvas to avoid such decoding problems.

In the meantime setting the img.decoding attribute to "async" will fix the bug and ensures full rendering of the layer image.

Stackblitz with bugfix: https://stackblitz.com/edit/web-platform-dmw9yg?file=esri-leaflet-debug.js

@gavinr
Copy link
Contributor

gavinr commented Dec 2, 2021

Thank you for investigating and for the PR! We really appreciate your time.

In the stackblitz example, I'm noticing a "flash" of the layer when I zoom in. Here's a video (in the video it only happens once because the frame rate of the recording - it does actually happen on every zoom for me):

rec.mp4

(happening me on Windows 10, Chrome v96.0.4664.45)

Is this happening for you too?

@gavinr
Copy link
Contributor

gavinr commented Sep 29, 2022

Inactive - closing.

@gavinr gavinr closed this Sep 29, 2022
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

3 participants