Closed KatieWoe closed 6 years ago
@jessegreenberg I will need your assistance for this one too I'm afraid.
there is a slight pause
@jessegreenberg @emily-phet please comment if you think this is a substantial enough bug to investigate further. @jessegreenberg can you reproduce? Have you noticed this before?
The pause is at least 3 seconds long. It's strange because normally the read out is immediate.
In addition to the delay, JAWS fails to read the aria-live alert associated with the slider. Strange that it doesn't happen with the other sliders.
Well, to be fair I am finding that JAWS often isn't reading the alerts from the sliders...
After refreshing the page, all alerts are coming through correctly.
If I comment out this bit of code which updates which dots are visible in the wire, the delay goes away, so this is performance related.
// Set the number of visible dots based on the resistivity.
var numDotsToShow = resistivityToNumberOfDots( resistivity );
dotsNode.children.forEach( function( dot, index ) {
dot.visible = index < numDotsToShow;
} );
@emily-phet would you like to spend time looking into performance improvements for this?
Possibly related to https://github.com/phetsims/a11y-research/issues/102, which outlines some performance issues with Firefox Quantum and JAWS.
I tried using renderer: canvas
on WireNode's dotsNode
, it made the delay go away! But it broke visibility of the dots when we change size of the wire.
We will need to do some other things to get it working, but I want to make sure it is worth spending the time first.
Apparently renderer shouldn't impact clipArea, the above may indicate a larger rendering issue.
@jessegreenberg, is there anyway to verify what is causing the delay (FF, JAWS, or visual renderer)?
The last message in phetsims/a11y-research#102 was that things were working better with the latest FF. I'm just wondering where the change has occurred.
@terracoda I am not sure. Without JAWS the sim runs very smoothly, so it is probably the combination of sim+AT using up resources. We could do some profiling with JAWS on to try to find where time is being spent. But if we can get this fix to work https://github.com/phetsims/resistance-in-a-wire/issues/164#issuecomment-404002454, it may not be worthwhile.
Discussed on 7/17/18 meeting -
We would like this to be fixed in scenery and hopefully we can use the renderer: canvas
option in the future to improve performance, but this issue does not need to block publication of resistance-in-a-wire with accessibility.
Lets briefly try with WebGL and see if this is still a problem before publishing with this bug.
I didn't know anything about the other rendering options, here is some doc I found, http://phetsims.github.io/scenery/doc/#node-renderer
This has been fixed with the above commit that targets Firefox. @zepumph can you make sure this gets into the release branch as part of #168?
https://github.com/phetsims/resistance-in-a-wire/commit/098297ee71a48db703e58459f3d7df9ea15f9179 was actually removed for #165 because the dots have been re-implemented with CanvasNode.
I verified that the bug was still fixed after #165, and it makes because canvas
renderer was the fix here too.
Has this issue been confirmed fixed in rc.8 @KatieWoe? If so please close.
Right, sorry. Didn't notice it during rc.8. Closing
Great thank you!
Test device: Dell Laptop Operating System: Windows 10 Browser: Firefox 61.0.1 Problem description: If you are at the bottom of the resistivity slider and use the home key to move it to the top there is a slight pause before the values are read out. This occurs when using Jaws Steps to reproduce:
For phetsims/QA/issues/137
Troubleshooting information (do not edit):