From abcb1dd050ae86a98e885e2cfa84d994dc882d5a Mon Sep 17 00:00:00 2001 From: Yan-Fa Li Date: Fri, 8 Nov 2024 13:38:15 -0800 Subject: [PATCH 1/2] chore: apply fixes from @precondition to pinia PR --- src/store/keycodes.js | 10 +++++----- src/store/modules/app/actions.js | 2 +- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/src/store/keycodes.js b/src/store/keycodes.js index bde6fe1c19..296135c468 100644 --- a/src/store/keycodes.js +++ b/src/store/keycodes.js @@ -59,13 +59,13 @@ function toLocaleKeycode(keycodeLUT, keycodeObject) { } /** - * Used to dynamically generate the tab data for they keycode + * Used to dynamically generate the tab data for the keycode * display. This UI is customized based on the OS keyboard layout * selected. * * @param {string} osKeyboardLayout * @param {boolean} isSteno - * @returns {Array.} */ function generateKeycodes(osKeyboardLayout, isSteno = false) { store.commit('app/setIso', !isANSI()); @@ -189,9 +189,9 @@ export const useKeycodesStore = defineStore('keycodes', { /** * @typedef {Object} KeycodeDefinition - metadata about a keycode * @property {string} name - UI display label for the keycode - * @property {string} code - QMK keycode defintion + * @property {string} code - QMK keycode definition * @property {string} [keys] - javascript keypress id. Used by keyboard handler - * @property {number} [width] - width in Key Units * 1000. e.g. 1U = 1000, 2U = 2000. + * @property {number} [width] - width in Key Units * 1000. e.g. 1U = 1000, 2U = 2000 * @property {'text'|'layer'|'container'|'layer-container'} [type] * @property {number} [layer] * @property {string} [title] - help text for hover @@ -211,7 +211,7 @@ export const useKeycodesStore = defineStore('keycodes', { /** * @typedef {Object} KeycodeStoreState * @property {Array.} keycodes - active keycodes - * @property {string} searchFilter - currently search filter + * @property {string} searchFilter - current query in keycode picker search filter * @property {SearchCounters} searchCounters - count of matching keycodes per tab * @property {boolean} steno - is steno tab active * @property {'ANSI'|'ISO/JIS'|'AppMediaMouse'|'Quantum'|'Steno'|'KeyboardSettings'} active - active tab diff --git a/src/store/modules/app/actions.js b/src/store/modules/app/actions.js index 52a6282780..26ae42af97 100644 --- a/src/store/modules/app/actions.js +++ b/src/store/modules/app/actions.js @@ -157,7 +157,7 @@ const actions = { commit('setOSKeyboardLayout', osLayout); const keycodesStore = useKeycodesStore(); - // Important to call keycodes/updatekeycodeNames *before* keymap/updateKeycodeNames. + // Important to call keycodesStore.updatekeycodeNames *before* keymap/updateKeycodeNames. keycodesStore.updateKeycodeNames(); this.commit('keymap/updateKeycodeNames'); keycodesStore.changeActive( From 3dc999fc4fe468e32b3ab28e4dd8282f593767de Mon Sep 17 00:00:00 2001 From: Yan-Fa Li Date: Fri, 8 Nov 2024 14:30:10 -0800 Subject: [PATCH 2/2] chore: missed a trailing '.' --- src/store/keycodes.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/store/keycodes.js b/src/store/keycodes.js index 296135c468..06debb098c 100644 --- a/src/store/keycodes.js +++ b/src/store/keycodes.js @@ -183,7 +183,7 @@ export const useKeycodesStore = defineStore('keycodes', { /** * @typedef {Object} WidthPlaceholder - used for spacing - * @property {number} width - width in Key Units * 1000. e.g. 1U = 1000, 2U = 2000. + * @property {number} width - width in Key Units * 1000. e.g. 1U = 1000, 2U = 2000 */ /**