-
Notifications
You must be signed in to change notification settings - Fork 5
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
[Arithmetic] missing keyboard input #7
Comments
PhET has considered the requirement of letting the user enter values from an external keyboard, and we've decided that it is too complex to implement at this time. It is no longer required, and the requirement that @MDmitrieva quoted above has been removed from the original spec. The users should only be able to enter values from the on-screen keypad. Closing. |
Another user reported to phethelp:
|
Another request for this feature received 12/11:
|
If it is an iPad app would the keyboard pop up and cover the sim? Or would it be limited to external keyboards that are attached/plugged in? When we get to keyboard navigation for this sim, I wonder if it would make keyboard entry possible. because right now we don't have a design for keypad entry so it seems like that would be the time to tackle it. Tagging @jessegreenberg @terracoda and @emily-phet to consider this when we get to this sim. |
Since there is a keypad within the sim, having one pop up on iPad seems undesirable. Hopefully it is possible to support key entry on desktop and not trigger the appearance of the keypad in iOS, but this will need to be investigated as part of the effort. |
Reopening, since the issue is being reconsidered. |
By default, yes I expect it will cover the sim. Number entry is possible but not something we have worked on yet. Making sure that an on screen keyboard does not appear for mobile devices will be part of this investigation. |
Is this sim being actively worked on? If not, and no sims with a keyboard are being worked on, let's wait until one comes up to dig into this. Otherwise, it would be worth taking a little time to see if we can stop the triggering of the onscreen keyboard on mobile devices, and then discuss next steps once we have that info. @jbphet @jessegreenberg |
At this time, the a11y team is not working on any other sims with a number input that require keyboard input like this. I also do not think this sim is being actively worked on, but @jbphet please correct me if I am wrong. |
I noticed that the cursor in this sim blinks whether or not the number readout has keyboard focus, which makes it look like it always has focus. That could be something to talk about when we return to this sim for a11y. |
@jessegreenberg - you are correct, there is no active development on the main functionality of this simulation. I'll set the priority to deferred and unassign for now. |
This issue does not seem like it would be within the scope of the upcoming Arithmetic region and culture publication, so I will leave as deferred. @amanda-phet let me know if you think otherwise. |
Have any sims with numberPad (I think that's the name) been developed yet with alt input? It seems like that is the time to tackle how to better design the experience with the numberPad. |
My Solar System I know there have been various issues around keyboard input and I've commented on a few in the past... I really dislike the current input experience and would have a strong preference for moving toward default OS keyboard support! |
I think this is somewhat covered by the work needing to be done here: phetsims/scenery-phet#283 It was added as a dev priority, but I'm not sure that will get finalized in time for Arithmetic... |
@marlitas is going to check in with @jessegreenberg to see if this seems worth approaching and if so, we can set up a meeting to discuss! |
In our design discussion today, we also noted that arithmetic is using NumberKeypad, which is deprecated. It's the last sim to be using it, so it would be really great to convert to Keypad (the new solutions) so that we can delete NumberKeypad. Tracking in #200. |
Also noted during today's design discussion. The blinking cursor is probably something that was ported from Flash by MLL (search for "blinking of cursor" in EquationNode.js). If you can't use the keyboard to type numbers in the HTML5 version, then that cursor is totally misleading -- I (for example) had to be told by @amanda-phet to use the keypad. So if it turns out that typing numbers is not possible, I recommend deleting the blinking cursor, and instead highlighting the '?' in some way (outilne or background fill) to emphasize it. |
@jessegreenberg and I met about this today, and we believe that there is a straightforward solution that allows Arithmetic to use Keypad and keyboard entry without enabling alternative input. We also discussed that the blinking cursor seems helpful in those cases. |
Arithmetic now has keyboard input with the above commits. @jessegreenberg do you mind reviewing for the changes in scenery and scenery-phet? |
@KatieWoe or @Nancy-Salpepi, @jbphet brought up a question about how this might work with iPad. I don't have one to test on. Would either of you be able to check and see if this now triggers the keyboard on iPad in a way that obscures the sim? |
These changes look good to me, thanks for adding them! I tested in arithmetic.
It won't (unless there is an external BLE keyboard), KeyboardListener just listens for keydown/keyup events. The on-screen keyboard should only pop up for more native text/number inputs like @oliver-phet suggested in #7 (comment). Id love to add that to scenery someday. |
I just checked the iPad and as Jesse indicated in #7 (comment), the keyboard doesn't popup at all. |
I believe this can be closed now. Thanks everyone for their help here! |
that if the user is using a mouse and keyboard, the sim should allow number entry with the keyboard so that they don’t have to use the mouse if they don’t want.
The text was updated successfully, but these errors were encountered: