mirror of
https://github.com/react/react-native-devtools-frontend.git
synced 2026-10-08 20:58:24 +08:00
Currently, ShortcutRegistry allows multiple shortcuts to be stored within the same binding in a module.json, e.g. `"shortcut": "F8 Ctrl+\\"` creates two shortcuts, one on F8 and another on Ctrl+\. When multi-keypress shortcuts/chords are added in the future, they'll use the same format such that the above example would be interpreted as a single shortcut bound to the F8 Ctrl+\ sequence. In order to allow that, it's necessary to split up all of the existing shortcuts that are stored in space-delimited strings so that they'll continue to function as expected. This CL also updates the module.schema.json to disallow spaces in shortcuts, a restriction that will be removed once multi-keypress shortcuts are implemented. Custom keyboard shortcuts design doc: https://docs.google.com/document/d/1oOPSWPxCHvMoBZ0Fw9jwFZt6gP4lrsrsl8DEAp-Hy7o/edit#heading=h.2xpjzz3fl1ju Bug: 174309 Change-Id: I853f9918ad2892b2f4c4f3aec53013d7a6455f67 Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/2123807 Reviewed-by: Robert Paveza <Rob.Paveza@microsoft.com> Reviewed-by: Vidal Diazleal <vidorteg@microsoft.com> Commit-Queue: Jack Lynch <jalyn@microsoft.com>