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

Hands snapping to "around" Nth tick rather than "on" Nth tick mark #421

Closed
terracoda opened this issue Jan 21, 2022 · 6 comments
Closed
Assignees
Labels

Comments

@terracoda
Copy link
Contributor

terracoda commented Jan 21, 2022

This might already be on our radar as something we would address with the Voicing design, but just in case I am making an issue so we make sure we fix this for Interactive Description, too.

When using VoiceOver and a mouse in published version, I could see some very slight snapping to the tick mark line, but VoiceOver would always say "around Nth tick mark" rather than "on Nth tick mark".

I know we don't expect VoiceOver users to use the mouse, but they can, and if there is snapping behaviour, the description should also snap to that same described value.

This is important for alignment with the Voicing design. We will need/want to hear "on Nth tick mark", and not "around Nth tick mark" when a mouse user lets go of the hand near a tick mark and it snaps to the tick mark.

@terracoda
Copy link
Contributor Author

@zepumph, we can address this as we work on implementation of the indy hand sliders issues #413 through #420.

@terracoda
Copy link
Contributor Author

The snapping and the snapped-to description work fine when tick marks are enabled. The issue I described in #420 is happening because there is no snapping and "At middle" is a super tiny region - hard to hit with mouse input.

@terracoda
Copy link
Contributor Author

Actually, I can get the incorrect description during active drag. The correct snapped-to description only happens at end of drag.

@zepumph
Copy link
Member

zepumph commented Mar 2, 2022

This issue is to tackle a bug in the keyboard snapper. It has a reproducible case and needs to be fixed, see the test added in the above commit.

Actually, I can get the incorrect description during active drag. The correct snapped-to description only happens at end of drag.

That is expected under the current implementation. If that is an issue please create a new issue.

zepumph added a commit that referenced this issue Mar 2, 2022
@zepumph
Copy link
Member

zepumph commented Mar 2, 2022

Ok, I see, 3 issues here, and I discussed them with @terracoda.

  1. General snapping of keyboard, rounds to a step of .06 in certain cases (the unit test in 7a34438). @terracoda and I decided that this is not a bug. The primary design goals are to (a) make sure that you can get in-proportion for every target ratio, and (b) snap to the "grid" that is the keyboard step you are using (making sure that if you use a mouse, then keyboard steps again, it rounds to the half-tick-mark). I'll revert the commit.
  2. Description region vs in-proportion tolerance inconsistency with mouse when you haven't let go. This seems similar to Edge case and no mouse snapping to At middle region #437, where we have inconsistent/unhelpful responses until we let go (end drag) because there isn't snapping until you let go. Likely we won't solve this in RAP, but it is important to name. I'll create an issue.
  3. Bug where VO would say around when you were on while using a mouse + VO (so mouse with description?). This seems to be what @terracoda was reporting in Hands snapping to "around" Nth tick rather than "on" Nth tick mark #421 (comment). We will make this issue devoted to that bug. Over to @terracoda to see if she can reproduce this.

@terracoda
Copy link
Contributor Author

Now (on PhET test on March 2) when using VoiceOver and a mouse on MacOS 11.4, I hear object responses (descriptions about proximity to ratio) when actively dragging, and I only hear context responses - responses that contain distance-region & position-region (tick-mark-region) - at end of dragging (letting go).

This is not an issue on master, and I also checked VoiceOver on the published version, and the context response only fires at end of drag, so not a problem there either.

Item 2, above remains an issue when Voicing is being used. And we may have (unfortunately) more than the edge case described #437

Closing this issue with VoiceOver since it is not actually happening - with VoiceOver, I get the correct region in the response at end of drag as the mouse snaps to the region.

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

2 participants