You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I was testing the zoom magnification feature using the ?supportZoom=true in BAM. The latest BAM RC doesn't work with this query parameter. More specifically, pinch to zoom and Cmd + doesn't work on my MacBook.
@ariel-phet mentioned that this shouldn't block 1.0 and most likely will get included in this sim's next maintenance release. Assigning to @jessegreenberg to take an initial look with any assistance I can provide.
The text was updated successfully, but these errors were encountered:
Regarding the query parameter, it looks like SHAs for build-a-molecule were taken while the param was still called zoom (instead of supportsZoom). When zoom is used, there is some magnification but it isn't working very well.
I checked out the scenery SHA for 1.0.0-rc.2, and it is from June 19, 2020. The published molecules-and-light scenery SHA with magnification is from July 21. So it looks like code related to zoom at this release was taken at a transient or not-working state.
Figuring out how to fix the old code could be done, but the scenery SHA for this release is > 2 months old now, and there have been many changes since then related to the zoom feature. It would be much easier (at least for zoom) to include in a republication off of master.
shouldn't block 1.0 and most likely will get included in this sim's next maintenance release.
@Denz1994 will this release be a maintenance release patched into 1.0 or a minor release off of master?
I was testing the zoom magnification feature using the
?supportZoom=true
in BAM. The latest BAM RC doesn't work with this query parameter. More specifically, pinch to zoom andCmd +
doesn't work on my MacBook.Here is the link that was tested.
@ariel-phet mentioned that this shouldn't block 1.0 and most likely will get included in this sim's next maintenance release. Assigning to @jessegreenberg to take an initial look with any assistance I can provide.
The text was updated successfully, but these errors were encountered: