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

Added Fallback for Offscreen Canvas #120

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

terrible-terry
Copy link

OffScreencanvas isn't supported by all browsers, this uses OffScreencanvas when available and provides a fallback when its not available.

@vasturiano
Copy link
Owner

@terrible-terry thanks for reaching out.

By looking at the MDN docs, OffscreenCanvas appears to have a pretty good support coverage among all modern browsers. May I ask which browser did you come across that didn't support it?

@terrible-terry
Copy link
Author

IOS Mobile 15.7.1 on Safari

@vasturiano
Copy link
Owner

Thanks @terrible-terry.

It seems it is supported in iOS Safari since v16.4, which was released in March 2023. My recommendation would be to update the browser to a more recent version.

@terrible-terry
Copy link
Author

I have a small percentage of users who operate older IOS devices that cannot be upgraded to a compatible version. Fairly old devices in my opinion but this is just the solution I went with rather than tell them they need to use another browser or buy a new device.

@vasturiano
Copy link
Owner

@terrible-terry you can perhaps try to apply a polyfill for OffScreenCanvas directly in your app. That will probably fix the issue for those users.

The problem with supporting adhoc older browser conditions is that it can be just the tip of the iceberg, and there might be many more other unsupported features that also need attention. So you need to draw the line somewhere. Hope you understand.

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