phetsims / concentration

"Concentration" is an educational simulation in HTML5, by PhET Interactive Simulations.
http://phet.colorado.edu/en/simulation/concentration
GNU General Public License v3.0
1 stars 7 forks source link

multitouch oddness with shaker on Win8 Surface + Firefox #19

Closed orejolaphet closed 10 years ago

orejolaphet commented 10 years ago

With Solid Solute selected, multiple touch pinch motion causes image and text for the shaker to change size, change position, and juxtaposition of the words.

concentration bug

Name: Concentration URL: http://www.colorado.edu/physics/phet/dev/html/concentration/1.0.0-rc.2/concentration_en.html Version: 1.0.0-rc.2 Features missing: flexbox Flags: pixelRatioScaling User Agent: Mozilla/5.0 (Windows NT 6.2; WOW64; rv:24.0) Gecko/20100101 Firefox/24.0 Language: en-US Window: 1280x595 Pixel Ratio: 1.5/1

pixelzoom commented 10 years ago

Please provide screenshots.

pixelzoom commented 10 years ago

This ticket originally contained 2 unrelated issues. Moved the performance issue to #23.

orejolaphet commented 10 years ago

Screenshot has been appended.

pixelzoom commented 10 years ago

When text gets disconnected from the shaker (as shown in the screenshot), it sometimes indicates that the sim has crashed.

pixelzoom commented 10 years ago

@orejolaphet: I don't have access to a Surface, so can't reproduce or investigate this. Can you please try this again with the Web Console open to see if anything prints to the console? It may be different on Surface, but you can typically open the Web Console by going to the Firefox menu (upper-left), then selection Web Developer > Web Console. Thanks.

orejolaphet commented 10 years ago

concentration bug 2

[11:12:47.606] GET http://www.colorado.edu/physics/phet/dev/html/concentration/1.0.0-rc.2/concentration_en.html [HTTP/1.1 200 OK 261ms]

[11:12:47.609] The character encoding declaration of the HTML document was not found when prescanning the first 1024 bytes of the file. When viewed in a differently-configured browser, this page will reload automatically. The encoding declaration needs to be moved to be within the first 1024 bytes of the file. @ http://www.colorado.edu/physics/phet/dev/html/concentration/1.0.0-rc.2/concentration_en.html:60

[11:12:47.714] GET http://www.google-analytics.com/ga.js [HTTP/1.1 304 Not Modified 8ms]

[11:12:47.861] Error in parsing value for 'display'. Declaration dropped. @ http://www.colorado.edu/physics/phet/dev/html/concentration/1.0.0-rc.2/concentration_en.html

[11:12:47.861] Error in parsing value for 'left'. Declaration dropped. @ http://www.colorado.edu/physics/phet/dev/html/concentration/1.0.0-rc.2/concentration_en.html

[11:12:47.861] Error in parsing value for 'right'. Declaration dropped. @ http://www.colorado.edu/physics/phet/dev/html/concentration/1.0.0-rc.2/concentration_en.html

[11:12:47.907] Unknown property 'box-sizing'. Declaration dropped. @ http://www.colorado.edu/physics/phet/dev/html/concentration/1.0.0-rc.2/concentration_en.html

[11:12:48.008] Expected end of value but found '!'. Error in parsing value for 'padding'. Declaration dropped. @ http://www.colorado.edu/physics/phet/dev/html/concentration/1.0.0-rc.2/concentration_en.html

[11:12:48.008] Expected end of value but found '!'. Error in parsing value for 'margin'. Declaration dropped. @ http://www.colorado.edu/physics/phet/dev/html/concentration/1.0.0-rc.2/concentration_en.html

[11:12:48.420] Use of getPreventDefault() is deprecated. Use defaultPrevented instead. @ http://www.colorado.edu/physics/phet/dev/html/concentration/1.0.0-rc.2/concentration_en.html:70

@pixelzoom I'm not exactly sure what it is you are looking for in the Web Console so I attached a screen shot along with what was printed inside.

pixelzoom commented 10 years ago

Nothing of interest in the console output, this stuff is all unrelated to what the sim is doing. (But might be worth having @jonathanolson skim this to see if there's anything general.)

jonathanolson commented 10 years ago

Probably not related, but the character encoding warning should go away (I made a chipper commit to shift the copyright info below the encoding).

Two points I'd like to know more about:

Potential bug areas:

orejolaphet commented 10 years ago

@jonathanolson The issue only happens with Firefox on the Surface. Also the issue isn't present on an iPad. To answer your question about size / proportion / orientation of the text is that it seems correlated to opposite direction of motion for the two fingers. When you pinch and bring your fingers together the shaker gets smaller in scale and moves to the left behind the text. When you move your fingers apart the shaker increases in scale and moves to the right of the text. Nothing happens if you have the two fingers move with the same motion. I tested the issue again on the latest versions of both Concentration and Beer's Law Lab

pixelzoom commented 10 years ago

Moved this issue to https://github.com/phetsims/scenery/issues/189