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

Disappearance of median brackets is inconsistent #106

Closed
Nancy-Salpepi opened this issue Mar 23, 2022 · 9 comments
Closed

Disappearance of median brackets is inconsistent #106

Nancy-Salpepi opened this issue Mar 23, 2022 · 9 comments

Comments

@Nancy-Salpepi
Copy link

Test device
Dell

Operating System
Windows 10

Browser
Chrome

Problem description
For phetsims/qa#789

After the data is sorted, if I move soccer balls so that the median value doesn't change but the cards are out of order, the median brackets do not disappear. However, when a move a card, without changing the median value, the brackets do disappear.
The brackets do disappear if dragging the soccer ball causes the median value to change.

Steps to reproduce

  1. On the median screen, kick several balls
  2. Check the Median checkbox in the accordion panel
  3. Sort the data manually (or check and then uncheck the Sort Data checkbox)
  4. Drag a soccer ball so that its card is now out of order but the median value stays the same--Median brackets are still there
  5. Move a card--brackets disappear

Visuals

medianbrackets.mp4
Troubleshooting information: !!!!! DO NOT EDIT !!!!! Name: ‪Center and Variability‬ URL: https://phet-dev.colorado.edu/html/center-and-variability/1.0.0-rc.1/phet/center-and-variability_all_phet.html Version: 1.0.0-rc.1 2022-03-21 16:46:53 UTC Features missing: applicationcache, applicationcache, touch Flags: pixelRatioScaling User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/99.0.4844.83 Safari/537.36 Language: en-US Window: 1431x633 Pixel Ratio: 2/1 WebGL: WebGL 1.0 (OpenGL ES 2.0 Chromium) GLSL: WebGL GLSL ES 1.0 (OpenGL ES GLSL ES 1.0 Chromium) Vendor: WebKit (WebKit WebGL) Vertex: attribs: 16 varying: 31 uniform: 1024 Texture: size: 16384 imageUnits: 16 (vertex: 16, combined: 32) Max viewport: 16384x16384 OES_texture_float: true Dependencies JSON: {}
@Nancy-Salpepi Nancy-Salpepi added the type:bug Something isn't working label Mar 23, 2022
@Nancy-Salpepi
Copy link
Author

It can look really odd in some cases:
Screen Shot 2022-03-23 at 3 11 26 PM

@amanda-phet
Copy link
Contributor

Definitely a problem! If you drag a ball and the value changes, then the median bars should only stay there if the data remains sorted.

@Nancy-Salpepi
Copy link
Author

I also noticed that if the ball that is moved is linked to the middle sorted card, then the median bars never disappear--even if the median value changes.

brackets2.mp4

@Nancy-Salpepi
Copy link
Author

In the State wrapper--the median bars disappear in the downstream wrapper (but remain in the upstream when they shouldn't).

bracketsstate.mp4

@chrisklus
Copy link
Contributor

Thanks @Nancy-Salpepi! Marking for the April milestone.

@chrisklus
Copy link
Contributor

@samreid and I committed a fix above. @Nancy-Salpepi could you please test on phettest?

@chrisklus
Copy link
Contributor

If it seems fixed, please leave open and relabel as status:ready-to-cherry-pick, thanks!

@chrisklus chrisklus assigned Nancy-Salpepi and unassigned samreid and chrisklus Mar 28, 2022
@Nancy-Salpepi
Copy link
Author

Looks good!

@samreid
Copy link
Member

samreid commented Apr 6, 2022

@chrisklus and I confirmed this is fixed in in #122, closing.

@samreid samreid closed this as completed Apr 6, 2022
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

4 participants