Skip to content

Commit 60e2846

Browse files
Merge pull request #3 from greymistcube/prefer-edit-over-hotkey
fix(tui): fixed config editing that wasn't working as intended
2 parents 006e7fc + ad3225b commit 60e2846

1 file changed

Lines changed: 7 additions & 7 deletions

File tree

src/tui/update.ts

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -775,19 +775,19 @@ function handleGenericKeyPress(model: AppModel, msg: AppMsg): [AppModel, Command
775775
const configModel = screenState.model as import('./types.js').ConfigScreenModel;
776776
let configMsg: import('./types.js').ConfigMsg | null = null;
777777

778-
if (key.name === 'tab' || key.name === 'h' || key.name === 'l') {
779-
configMsg = { type: 'CONFIG_TOGGLE_FOCUS' };
780-
} else if (key.name === 'left') {
781-
configMsg = { type: 'CONFIG_FOCUS_LIST' };
782-
} else if (key.name === 'right') {
783-
configMsg = { type: 'CONFIG_FOCUS_EDITOR' };
784-
} else if (configModel.isEditing) {
778+
if (configModel.isEditing) {
785779
if (key.name === 'escape') configMsg = { type: 'CONFIG_EDIT_CANCEL' };
786780
else if (key.name === 'return' || key.name === 'enter')
787781
configMsg = { type: 'CONFIG_EDIT_SAVE' };
788782
else if (key.name === 'backspace') configMsg = { type: 'CONFIG_EDIT_BACKSPACE' };
789783
else if (key.sequence.length === 1 && !key.ctrl && !key.meta)
790784
configMsg = { type: 'CONFIG_EDIT_INPUT', char: key.sequence };
785+
} else if (key.name === 'tab' || key.name === 'h' || key.name === 'l') {
786+
configMsg = { type: 'CONFIG_TOGGLE_FOCUS' };
787+
} else if (key.name === 'left') {
788+
configMsg = { type: 'CONFIG_FOCUS_LIST' };
789+
} else if (key.name === 'right') {
790+
configMsg = { type: 'CONFIG_FOCUS_EDITOR' };
791791
} else {
792792
if (key.name === 'j' || key.name === 'down') configMsg = { type: 'CONFIG_OPTION_DOWN' };
793793
else if (key.name === 'k' || key.name === 'up') configMsg = { type: 'CONFIG_OPTION_UP' };

0 commit comments

Comments
 (0)