-
Notifications
You must be signed in to change notification settings - Fork 6
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
Focus should be at top when returning to home screen #695
Comments
@jessegreenberg, I'm not sure how we accomplish this, or even if we can. Is this needed for Greenhouse effect? |
Actually, a better, more natural user experience is for focus to go the button of the source screen. I thought I had this plan mapped for RaP? There might be an interaction design pattern in Binder?? - not sure.
Note having a 4-screen sim like GHE in development will give us an opportunity to investigate the sim screen interactions further. We have not done an exhaustive investigation of the Sim Screen interactions. Adding - At least I don't think I have done and exhaustive study, and I certainly haven't interviewed anyone on a sim that has more than 2 screens. |
Discussed with @terracoda, focus should return to the h1 on the home screen to be consistent with other screens. Ill take a look. |
There is specific code in the NavigationBar that is putting focus on the home screen button. // only if fired from a11y
if ( this.homeButton!.isPDOMClicking() ) {
sim.homeScreen!.view.focusHighlightedScreenButton();
} It is very old (2018), predating our decision to put focus on the 'h1' of the ScreenView (2021), so I will go ahead and remove it. |
…uld start at the top of the home screen like all ScreenViews, see #695
This was pushed to main, closing. |
@jessegreenberg @terracoda, I believe we have discussed this before.
@terracoda thinks this should match the current behavior for sim screens (focus the h1), and not focusing to the highlighted screen button. Basically we should be able to just get rid of
focusHighlightedScreenButton
.Tagging this for phetsims/ratio-and-proportion#363.
The text was updated successfully, but these errors were encountered: