mirror of
https://github.com/keymanapp/keyman.git
synced 2026-09-04 22:57:40 +00:00
62796e3489
Fixes #3620. Implements the Caps Lock layer support and the double-tap gesture on the shift key to access it. The double-tap gesture has been implemented with a view to extension to support other multi-tap gestures in the future. However, for now, it is limited to supporting the Shift key, if and only if the keyboard includes a Caps layer. The reason for this v15 limitation is that multi-tap on regular keys would involve either rewinding the previous keystroke (the first tap), or forcing keyboard developers to consider 'rota' style rules in their keyboards to support the multi-tap gestures, as we need to make sure that the first tap is accepted and processed for immediate feedback. This needs more design, to avoid unnecessary complexity in the keyboards and/or the rewinding of the keystroke (even though that is conceptually supported in Keyman Engine for Web already). Basically, we don't want to constrain the way that a keyboard author may use the multi-tap gesture by hard-coding the rewind, but neither do we want to make all multi-tap gestures needlessly complex to author. The shift key (and other modifiers, potentially in future) needs special support for multi-tap as the key that is being tapped changes with the layer change. This is currently managed through recognising `K_SHIFT` in the key id. I have tried to follow the `PendingGesture` pattern for multi-tap, and the gesture itself supports a series of taps, not just a double-tap. The maximum time to complete the tap series is 125msec * number-of-taps, so for a double-tap is 250msec. The changes to support a Caps Lock layer itself were minimal; just adding the `text.KeyboardProcessor.getStateFromLayer` function and calling it during `KeyEvent` construction. The remaining changes relate to the multi-tap gesture. Minor changes: * I moved `constructNullKeyEvent` to `KeyEvent` in order to make it more accessible to other classes. * The multi-tap gesture does not have a promise to complete, so that is now an optional member of the `PendingGesture` interface. |
||
|---|---|---|
| .. | ||
| attachment-api | ||
| basic-iframe | ||
| build-visual-keyboard | ||
| caps-lock-layer-3620 | ||
| chirality | ||
| ckeditor | ||
| common | ||
| desktop-ui | ||
| empty-row | ||
| inline-osk | ||
| issue005 | ||
| issue29 | ||
| issue53 | ||
| issue62 | ||
| issue63 | ||
| issue103 | ||
| issue115 | ||
| issue116 | ||
| issue160 | ||
| issue266 | ||
| issue271 | ||
| issue382 | ||
| issue917-context-and-notany | ||
| issue920 | ||
| issue1332 | ||
| issue2924 | ||
| issue3701 | ||
| issue5455 | ||
| keyboard-errors | ||
| mnemonic | ||
| options-with-save | ||
| osk-movement | ||
| platform | ||
| prediction-mtnt | ||
| prediction-ui | ||
| promise-api | ||
| regression-tests | ||
| rotation-events | ||
| scrolling | ||
| sentry-integration | ||
| spacebar-text | ||
| start-of-sentence-3621 | ||
| test-updateLayer | ||
| commonHeader.js | ||
| index.html | ||
| lao_2008_basic-1.2.js | ||
| unminified - manual.html | ||
| unminified.html | ||
| us-1.0.js | ||