-
-
Notifications
You must be signed in to change notification settings - Fork 112
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
fix(web): default subkey forwarding 🐵 #9952
Merged
jahorton
merged 3 commits into
feature-gestures
from
fix/web/gestures-and-longpress-defaults
Nov 14, 2023
Merged
fix(web): default subkey forwarding 🐵 #9952
jahorton
merged 3 commits into
feature-gestures
from
fix/web/gestures-and-longpress-defaults
Nov 14, 2023
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
keymanapp-test-bot
bot
added
has-user-test
user-test-required
User tests have not been completed
labels
Nov 7, 2023
User Test ResultsTest specification and instructions
Test Artifacts
|
mcdurdin
approved these changes
Nov 7, 2023
keymanapp-test-bot
bot
removed
the
user-test-required
User tests have not been completed
label
Nov 10, 2023
Base automatically changed from
fix/web/gestures-old-android-compat
to
feature-gestures
November 14, 2023 08:26
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I'd forgotten that default subkey selection logic is not available on the gesture-engine level, so KMW longpress handling has to override the gesture's selected-key output when a default subkey should be selected.
Technically, we could make it override only when the "select default" mode is on; that wouldn't actually be that difficult. But... it's not really necessary to toggle the
directlyEmitsKey
property, and not doing so results in somewhat simpler code.User Testing
TEST_SPECIAL_DEFAULT: if a subkey exists with an explicitly-marked 'default' subkey, it should be auto-selected when the subkey menu is first displayed.
On a mobile device (or when emulating one in a desktop browser), visit the testing page marked "Tests handling of new default-subkey feature (feat(developer): define long-press default key 🐵 #9430)."
Longpress the
d
key.When the subkey menu is displayed, verify that the fifth subkey - "default" - is selected. The text may be cropped.
Without moving the touchpoint/finger, release the touchpoint / lift your finger. This should output the text
default
.TEST_MATCHING_ID_DEFAULT: if a subkey exists with an ID perfectly matching its base key, it should be auto-selected in the absence of a specified subkey.
On a mobile device (or when emulating one in a desktop browser), visit the testing page marked "Tests handling of new default-subkey feature (feat(developer): define long-press default key 🐵 #9430)."
Longpress the
a
key.When the subkey menu is displayed, verify that the third subkey - "a" - is selected.
Without moving the touchpoint/finger, release the touchpoint / lift your finger. This should output the text
a
.TEST_NONDEFAULT_SELECTION: the user should still be able to select a subkey that's not the default, of course.
d
key.d
subkey:đ
and release.đ
is output.TEST_IOS_SPECIAL_DEFAULT: the same as the first test above, but this time within the iOS app.
Using Safari, download and then install the
default-subkey
keyboard package: default_subkey.kmpWithin the iOS app, longpress the
d
key.When the subkey menu is displayed, verify that the fifth subkey - "default" - is selected. The text may be cropped.
Without moving the touchpoint/finger, release the touchpoint / lift your finger. This should output the text
default
.