phetsims / expression-exchange

"Expression Exchange" is an educational simulation in HTML5, by PhET Interactive Simulations.
GNU General Public License v3.0
2 stars 2 forks source link

Failed assertion while hovering over break apart button #14

Closed phet-steele closed 8 years ago

phet-steele commented 8 years ago

If a combined coin is clicked, there is break apart button that appears. The button behaves normally if the coin is clicked once. However! DOUBLE clicking on a combined coin and moving your mouse to hover over the button throws this error:

Uncaught Error: Assertion failed: should not be over button without hide timer running
window.assertions.assertFunction @ assert.js:23
(anonymous function) @ CoinTermNode.js?bust=1459477988755:275
inherit.emit2 @ Emitter.js?bust=1459477988755:139
inherit._notifyObservers @ Property.js?bust=1459477988755:129
inherit._setAndNotifyObservers @ Property.js?bust=1459477988755:117
inherit.set @ Property.js?bust=1459477988755:85
Object.defineProperty.set @ PropertySet.js?bust=1459477988755:141
inherit.enter @ ButtonListener.js?bust=1459477988755:91
inherit.dispatchToTargets @ Input.js?bust=1459477988755:846
inherit.dispatchEvent @ Input.js?bust=1459477988755:788
inherit.enterEvents @ Input.js?bust=1459477988755:732
inherit.branchChangeEvents @ Input.js?bust=1459477988755:723
inherit.moveEvent @ Input.js?bust=1459477988755:678
inherit.mouseMove @ Input.js?bust=1459477988755:334
inherit.run @ BatchedDOMEvent.js?bust=1459477988755:63
inherit.fireBatchedEvents @ Input.js?bust=1459477988755:158
inherit.batchEvent @ Input.js?bust=1459477988755:128
onmousemove @ Input.js?bust=1459477988755:111

This error is then succeeded by a new error once the button hides (probably a clue @jbphet);

Uncaught Error: Assertion failed: An attempt was made to remove a non-existent step listener
window.assertions.assertFunction @ assert.js:23
Timer.removeStepListener @ Timer.js?bust=1459477988755:96
callback @ Timer.js?bust=1459477988755:47
Timer.step @ Timer.js?bust=1459477988755:32
inherit.stepSimulation @ Sim.js?bust=1459477988755:730
inherit.runAnimationLoop @ Sim.js?bust=1459477988755:661

These occurred on master as of ~7:00 PM (3/31).

jbphet commented 8 years ago

Fixed, closing.