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

WebGL is broken on iPad #1649

Open
pixelzoom opened this issue Jul 23, 2024 · 3 comments
Open

WebGL is broken on iPad #1649

pixelzoom opened this issue Jul 23, 2024 · 3 comments

Comments

@pixelzoom
Copy link
Contributor

Using WebGL on iPad results in the sim crashing. The past 2 sims that I've worked on have both had this problem, where WebGL was enabled to take advantage of Sprites performance. See:

The workaround in both cases has been to disable WebGL on mobile Safari. For example, in FELSim.ts:

      // WebGL is typically enabled for high-performance scenery.Sprites, which are used heavily by this sim.
      // On iPadOS, the sim consistently crashed when WebGL was enabled, and we were unable to determine why.
      // Disabling WebGL uses Canvas as the fallback for Sprites. The problem was not present on iOS, but we
      // have no way to differentiate between Safari on iPadOS vs iOS, so WebGL is disabled on both platforms.
      // See https://github.com/phetsims/faradays-electromagnetic-lab/issues/182.
      webgl: !platform.mobileSafari,

This seems like a problem that should be investigated and resolved with high priority, instead of continuing to disable WebGL on iPad.

@jonathanolson
Copy link
Contributor

Noted in Slack I'd need to have an iPad to debug this. @kathy-phet or @KatieWoe let me know if there's a good way for me to get a hold of one.

@jonathanolson
Copy link
Contributor

@zepumph
Copy link
Member

zepumph commented Aug 29, 2024

@jonathanolson have you been working on this recently? If so can you give an update here? @samreid and I have continued in phetsims/density-buoyancy-common#316 and it would be good to understand if you are also in the area.

@zepumph zepumph assigned jonathanolson and unassigned kathy-phet and KatieWoe Aug 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants