Repository navigation
refactor(kbd): retire keyb in favour of neokey - #704
Merged
Merged
Conversation
`keyb` only ever toggled the layout index (its name-based set path was broken) and is superseded by neokey, which the docs/roadmap already declare as its replacement. - neokey gains `layout <name>` (KeyboardSetLayout), `repeat`/`delay` (from the Registry-backed config) and `leds`; split into modules (output/parse/commands) with the IDS constants kept in main.rs for check-i18n bindings. - drop `userbin/keyb`, its locale catalogues and the `keyb` app id. - refresh i18n keymap, shell/docs references.
This file contains hidden or 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
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.
Qué
Retira
keyb(legacy) y dejaneokeycomo único CLI de teclado.neokeyganalayout <name>,repeat <cps>,delay <ms>yleds <lista>;showincluye repeat/delay leyendo el valor persistido (Registry) víakbd_get_repeat.main/commands/output/parse); constantes i18n enmain.rs(check-i18n valida bindings solo contra ese fichero).userbin/keyb, sus catálogos de locale y el app idkeyb; refresca keymap y docs.Por qué
keybya figuraba como legacy en docs/roadmap (ADM-NEOKEY #170) y su path de set por nombre estaba roto (usaba la clase de índice con un nombre).Verificación
neodev build --image+neodev test→ 964/964 PASS;check-i18n,check-skills,markdownlintOK.Notas
No cierra ninguna Issue. Semánticamente depende de la PR de layouts para los nombres
es-ES/en-US(#fix-kbd-layouts).