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

Measuring tape can flicker #804

Open
samreid opened this issue Mar 22, 2023 · 3 comments
Open

Measuring tape can flicker #804

samreid opened this issue Mar 22, 2023 · 3 comments

Comments

@samreid
Copy link
Member

samreid commented Mar 22, 2023

During phetsims/my-solar-system#86 @zepumph and @AgustinVallejo and I observed the measuring tape flickering when the distance measurement was 0:

Kapture 2023-03-22 at 11 45 32

@zepumph zepumph self-assigned this Mar 22, 2023
@zepumph zepumph removed their assignment Mar 23, 2023
@AgustinVallejo AgustinVallejo transferred this issue from phetsims/my-solar-system Mar 24, 2023
@AgustinVallejo
Copy link
Contributor

Originally issue in MSS. But with @arouinfar we reproduced it in Projectile Motion.

@arouinfar
Copy link

@AgustinVallejo @DianaTavares and I reviewed this issue and determined it is not specific to My Solar System, so it's likely an issue with MeasuringTapeNode.

To reproduce, you must manually drag the tip until it reaches a measurement of 0 units. Then, grab the measuring tape body and drag it around the play area. The flicker is not usually as severe as what is shown in #804 (comment).

This is difficult to reproduce, largely because it is difficult to zero out the measuring tape by hand. Notably, the flicker doesn't seem reproducible if the measuring tape is zeroed out by dragging it to the edge of its bounds. So it's possible the problem arises when the length is very small, not zero.

I would categorize this as a low-severity issue, given the difficulty to reproduce and the general uselessness of measuring a distance of zero. However, I'm going to assign this to the responsible dev of scenery-phet @pixelzoom in case there is a clear/easy solution.

@pixelzoom I completely defer to your judgment here.

@pixelzoom
Copy link
Contributor

Unassigning until someone has a need for this.

@pixelzoom pixelzoom removed their assignment Mar 28, 2023
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