-
Notifications
You must be signed in to change notification settings - Fork 8
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
replace LaserNode with scenery-phet.LaserPointerNode #339
Comments
@ariel-phet this issue could be given to another developer if desired--not too much bending-light knowledge required. Can you help prioritize and assign? The main point is to use the new scenery-phet LaserPointerNode @pixelzoom developed for Models of the Hydrogen atom in Bending Light instead of custom raster images. |
@samreid since bending light is published, this seems fairly lowly priority, agreed? |
Yes, it would not significantly change the user experience, it is mainly an internal maintenance issue. |
Ok, going to clear assignee for the moment, will revisit when appropriate. This sim is actually a very good candidate for phet-io work at some point, to see if we can deal with a complicated sim. I am guessing the work could be wrapped into that effort. |
Thanks to the changes in the referenced issues, this has been completed. @pixelzoom do you have time for a quick review? If not, we may need to leave this for a "review right before publication" sort of thing. |
Changes and behavior in bending-light look good, so I'll go ahead and close this. But note the regression in phetsims/scenery-phet#660 (comment), which does not appear to affect bending-light, since its LaserPointerNode uses the default |
... as recommended in phetsims/models-of-the-hydrogen-atom#2.
When
options.hasKnob
, you'll need to add the knob as a decoration, as you've done for the prisms that can be rotated (see PrismNode).Remember to delete related images and assets.
The text was updated successfully, but these errors were encountered: