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

Blue background overlapping with other parts parts of screen with wire movement #489

Open
etwa4650 opened this issue Apr 26, 2019 · 11 comments

Comments

@etwa4650
Copy link

etwa4650 commented Apr 26, 2019

Test Device

JORDAN

Operating System

MAC OS 10.14.4

Browser

Firefox

Problem Description

for phetsims/qa#311

While moving one end of a wire, the blue background will appear over parts the black banner at the bottom and the toolbox on the right hand side.

Seems like the effects might be more dramatic with more wires connected/added.
Was able to duplicate in both the "Intro" and "Lab" simulations.

This also occurs in the published version.

Screen recording in the link below.

Steps to Reproduce

Add a wire vertically on the left hand side of screen.
Add a wire horizontally at the bottom of the screen, close to the bottom edge.
Drag and stretch the right end of the bottom wire around.

Note: I have been able to replicate this with just a single horizontal wire at the bottom, but it is more easily reproduced with the vertical wire.

Visuals

https://drive.google.com/file/d/116UdA4XaiKHMGhf4_u8MpAs8Y6SAU3xz/view?usp=sharing

@samreid
Copy link
Member

samreid commented Apr 26, 2019

Great discovery @etwa4650! @jonathanolson have you seen anything like this or have any ideas about it?

@samreid
Copy link
Member

samreid commented Apr 29, 2019

I am unable to reproduce this problem on Mac OS 10.14.4 running Firefox 66.0.3 (64-bit). What firefox version was this seen on? Is there another QA machine that can be used to try to try to reproduce this behavior?

@samreid samreid assigned etwa4650 and KatieWoe and unassigned samreid Apr 29, 2019
@KatieWoe
Copy link
Contributor

@samreid I'm seeing it on 66.0.3, though it seems less pronounced for me than in the video. @amanda-phet checked on her machine and we couldn't reproduce it there.

@KatieWoe
Copy link
Contributor

KatieWoe commented May 6, 2019

Looks like it may be related to phetsims/area-model-common#175, if that helps

@etwa4650
Copy link
Author

etwa4650 commented Jul 22, 2019

Also seen on Windows 10 in phetsims/qa#369

Although it seems much harder to replicate.

@samreid
Copy link
Member

samreid commented Jul 25, 2019

@ariel-phet would you like investigation into this issue before publication of CCK DC and DC Virtual Lab with the fuse component? It was the only issue pointed out during RC.1. If you would like investigation into this, I believe @jonathanolson may be best versed in problems with Firefox repaint regions.

@kathy-phet
Copy link

Does this exist in the published version? If yes, then I would vote for publishing as is, if not, then investigate what change to scenery may be causing this issue? ... but just defer, not close.

@jonathanolson
Copy link
Contributor

I'm also curious about whether WebGL was used for that error (as it seems similar to some instances in WebGL with invalid pixel data)?

@samreid
Copy link
Member

samreid commented Jul 25, 2019

Does this exist in the published version? If yes, then I would vote for publishing as is, if not, then investigate what change to scenery may be causing this issue? ... but just defer, not close.

It was stated in the initial issue description that:

This also occurs in the published version.

@etwa4650 or @KatieWoe can you additionally test the sim with ?webgl=false to see whether that suppresses the problem?

@ariel-phet
Copy link

@samreid a few points:

  1. We are not supporting firefox and Mac anymore, so that is not a concern
  2. I can reproduce on FF and Win 10 (in both dev and published version)
  3. ?webgl=false does NOT suppress the problem
  4. Pretty easy to trigger with even a single wire

@ariel-phet
Copy link

Since this exists in the published version, will defer for now.

@ariel-phet ariel-phet removed their assignment Jul 25, 2019
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

6 participants