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

Several items missing in iOS 13 #164

Closed
KatieWoe opened this issue Sep 17, 2019 · 8 comments
Closed

Several items missing in iOS 13 #164

KatieWoe opened this issue Sep 17, 2019 · 8 comments
Assignees
Labels

Comments

@KatieWoe
Copy link
Contributor

For phetsims/qa#427
The icon for the first screen is missing both on the home screen and within the sim on iOS 13. It is there in iOS 12.
Image from iOS (3)

@KatieWoe KatieWoe changed the title Main icon for first screen missing in iOS 13 Several items missing in iOS 13 Sep 17, 2019
@KatieWoe
Copy link
Contributor Author

Edit: Much larger problem in the sim. The alpha particles don't show up on the second screen, and the close up on the first screen is completely gone.

@KatieWoe
Copy link
Contributor Author

Occurs in published.

@jessegreenberg
Copy link
Contributor

OK thanks @KatieWoe. It sounds like most of the protons and neutrons are missing. They are painted in an atypical way in this sim. For instance, see RutherfordNucleusNode.RutherfordNucleusIcon which uses inner class IconCanvasNode.

@jessegreenberg
Copy link
Contributor

The particles use CanvasNode and the rendering context method drawImage. Wondering if it is related to those. @KatieWoe can you please try one of these other use cases of this to see if things show up as expected?

  • Molecules in the magnifying glass in acid-base-solutions.
  • Minus charges in the wall in balloons and static electricity

@KatieWoe
Copy link
Contributor Author

Molecules in acid-base-solutions look fine to me.
Charges look fine in BASE

@jessegreenberg
Copy link
Contributor

Thanks for checking. Points to either something more specific to the sim or something else entirely.

@jessegreenberg
Copy link
Contributor

This is caused by phetsims/sun#526. Being investigated there.

@jessegreenberg jessegreenberg removed their assignment Sep 19, 2019
@jessegreenberg
Copy link
Contributor

This was patched as part of phetsims/sun#526 and can now be closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants