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

Additions to alternative input - number keys and jump key #62

Closed
2 tasks done
emily-phet opened this issue Jun 8, 2020 · 7 comments
Closed
2 tasks done

Additions to alternative input - number keys and jump key #62

emily-phet opened this issue Jun 8, 2020 · 7 comments
Assignees
Labels
type:bug Something isn't working

Comments

@emily-phet
Copy link

emily-phet commented Jun 8, 2020

To the existing alternative input mode where both hands can be controlled simultaneously (W/S and arrow keys), please add the ability for:

  • Number keys (0-9) to jump both hands to that number gridline
  • “j” key to jumps both hands to lowest currently occupied position

This would be the same for both screens.

Noting here - we will also need to confirm that this interaction technically works well with screen readers we support (presumably so, in role=application mode), if some sort of button "pickup" is needed to make use more intuitive for non-visual experience, and design visual cues to indicate available actions upon initial (or all) first encounter.

Also - it was brought up last week that the number 0 could jump hands to 0 or to 10, let's start with it jumping hands to 0 position (or lowest position). Also, gridlines position can change in the second screen, for right now let's focus only on providing access to the 0-9 grid positions with the number keys.

@zepumph
Copy link
Member

zepumph commented Jun 8, 2020

When the base unit changes for the grid, do you want the behavior of the number keys to change? It would be easiest if the numbers were treated more like percents, where 1 went to 10% of the ratio height. Do you instead want to always align these numbers to the grid number?

@zepumph
Copy link
Member

zepumph commented Jun 9, 2020

@emily-phet and I talked today, and we do want the numbers to map to the visual grid line numbers, not to percent.

@zepumph
Copy link
Member

zepumph commented Jun 10, 2020

Implemented above. @emily-phet please review.

@zepumph zepumph assigned emily-phet and unassigned zepumph Jun 10, 2020
zepumph added a commit that referenced this issue Jun 10, 2020
@emily-phet
Copy link
Author

@zepumph The basic behavior looks great to me.
It seems a bit buggy though:

  • I had a few times where the sound kept playing after I had moved the hands.
  • The hands don't always look visually aligned. For example, on startup, go straight to using the number keys to jump the hands. They seem a bit off in horizontal placement.

Also - let's have the orange cueing arrows disappear if you use any of the keypresses to move the hands (up/down arrow keys, W/S, number keys, or j key).

@zepumph
Copy link
Member

zepumph commented Jun 22, 2020

I had a few times where the sound kept playing after I had moved the hands.

Right now, we only have a rough way of triggering sound to play with alternative input. The sound plays when there is focus on either hand, or on the composite ratio. You won't get any sound when fitness is 0, but you will get sound otherwise. It would most likely be good to open a new issue to discuss this further if the current way isn't right.

The hands don't always look visually aligned. For example, on startup, go straight to using the number keys to jump the hands. They seem a bit off in horizontal placement.

They seem aligned to me. I think there is a bit of a optical illusion when in the middle of the space with no grid lines, and I feel like they are off, but then I show the grid lines and they are right on. I wonder if what you are witnessing is #53. To check this see if the bottom "framing" rectangles are aligned, if they are offset, then refresh the sim. Sorry I haven't gotten to that just yet.

@zepumph zepumph removed their assignment Jun 22, 2020
@zepumph
Copy link
Member

zepumph commented Jul 10, 2020

I have sometimes gotten a reentrant error when pressing the 0 key. It is a bit hard to reproduce, but I think I will want to investigate. Reopening.

@zepumph zepumph reopened this Jul 10, 2020
@emily-phet emily-phet assigned zepumph and unassigned emily-phet Jul 14, 2020
@zepumph
Copy link
Member

zepumph commented Jul 21, 2020

Fixed above. Closing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants