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

Sim iFrame shift when tab changes focus in a11y-view html #687

Closed
mbarlow12 opened this issue Jul 4, 2018 · 11 comments
Closed

Sim iFrame shift when tab changes focus in a11y-view html #687

mbarlow12 opened this issue Jul 4, 2018 · 11 comments
Assignees
Labels

Comments

@mbarlow12
Copy link
Contributor

I just released a new one-off version of Faraday's Law and noticed that when you tab around the a11y-view, the sim jumps up with each press until the position below.

image

Not sure about who's the responsible dev for this, so I tagged you both, @zepumph @jessegreenberg.

@zepumph
Copy link
Member

zepumph commented Jul 4, 2018

I see this as a potential for the version you were finding the bug in,
https://phet-dev.colorado.edu/html/faradays-law/1.4.0-jump.2/phet/faradays-law_a11y_view.html

I was not able to reproduce this problem. By your brief description above, I imagine that it was easy and noticeable when it was happening. I also couldn't see it in master.

Could you please do the following things on multiple browsers if possible as maybe it is a weird browser thing:

  1. reproduce it again from the jump version.
  2. is it reproducible from master?
  3. If possible, is it reproducible from a previous jump one-off?

Also what is your OS? Are there any errors in the console?

Good Luck!

(p.s. the jumping is so cool! Good work)

@zepumph zepumph assigned mbarlow12 and unassigned zepumph Jul 4, 2018
@jessegreenberg
Copy link
Contributor

Weird! I am also not able to reproduce in master or in https://phet-dev.colorado.edu/html/faradays-law/1.4.0-jump.2/phet/faradays-law_a11y_view.html, where this bug was also noted in phetsims/faradays-law#109.

The focusable items are shifted and stacked vertically along the side of the simulation, it looks like the browser is trying to scroll down to those items when they receive focus.

@jessegreenberg jessegreenberg removed their assignment Jul 5, 2018
@zepumph
Copy link
Member

zepumph commented Jul 5, 2018

@mbarlow12
Copy link
Contributor Author

Hmm - this might be specific to FF on Arch. I'll investigate further.

@jessegreenberg
Copy link
Contributor

Might just be Firefox, we can see it again here, outside of the a11y view:
phetsims/resistance-in-a-wire#160

@jessegreenberg
Copy link
Contributor

@mbarlow12 thanks for pointing this out, this was indeed a bug on all Firefox. I think it is fixed in master, can you please verify?

@mbarlow12
Copy link
Contributor Author

I noticed that it is related to PDOM content. When I remove all a11y content from a Node/AccessibleSectionNode/etc... the bug no longer occurs.

@jessegreenberg
Copy link
Contributor

That makes sense since the PDOM would be short enough to fit in the Display bounds.

@mbarlow12
Copy link
Contributor Author

I'm seeing it Ohm's Law and RIAW in and out of the a11y view.

@jessegreenberg
Copy link
Contributor

Here is the issue related to this, that introduced the regression: phetsims/scenery#822

@jessegreenberg
Copy link
Contributor

On slack @mbarlow12 said

I hadn't realized it was scenery to pull. Yes, fixed.

My fault for not indicating the fix was located in scenery in #687 (comment). But this issue can be closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants