More detailed error on keybinding errors #15203

Closed
opened 2026-01-31 04:31:21 +00:00 by claunia · 8 comments
Owner

Originally created by @joca-bt on GitHub (Sep 14, 2021).

After updating I got an error message saving my settings:

error message

It would be helpful if the error message showed either the actions or the keybindings causing the error. Otherwise the user has to guess.

Originally created by @joca-bt on GitHub (Sep 14, 2021). After updating I got an error message saving my settings: <img width="368" alt="error message" src="https://user-images.githubusercontent.com/1297942/133312489-2bc42d00-ce0c-4a71-a132-ff8f53e096a7.png"> It would be helpful if the error message showed either the actions or the keybindings causing the error. Otherwise the user has to guess.
Author
Owner

@zadjii-msft commented on GitHub (Sep 14, 2021):

After updating

Curious, usually I wouldn't expect this to happen on an update. Could you share your settings.json and Terminal version? I'm curious myself what param became required.

@zadjii-msft commented on GitHub (Sep 14, 2021): > After updating Curious, usually I wouldn't expect this to happen on an update. Could you share your `settings.json` and Terminal version? I'm curious myself what param became required.
Author
Owner

@joca-bt commented on GitHub (Sep 14, 2021):

Version: 1.10.2383.0

I only noticed when updating to 1.10 but I believe it started happening after updating to 1.09. I originally defined the keybindings using 1.08 or 1.07, I don't remember which one.

The offending keybindings:

        { "command": { "action": "findMatch", "direction": "next" }, "keys": "ctrl+shift+s" },
        { "command": { "action": "findMatch", "direction": "prev" }, "keys": "ctrl+shift+r" },
@joca-bt commented on GitHub (Sep 14, 2021): Version: 1.10.2383.0 I only noticed when updating to 1.10 but I believe it started happening after updating to 1.09. I originally defined the keybindings using 1.08 or 1.07, I don't remember which one. The offending keybindings: ``` { "command": { "action": "findMatch", "direction": "next" }, "keys": "ctrl+shift+s" }, { "command": { "action": "findMatch", "direction": "prev" }, "keys": "ctrl+shift+r" }, ```
Author
Owner

@joca-bt commented on GitHub (Sep 14, 2021):

By "when updating" I mean "I updated the terminal, restarted it, and then saved my current settings to check if everything was alright and got that message".

@joca-bt commented on GitHub (Sep 14, 2021): By "when updating" I mean "I updated the terminal, restarted it, and then saved my current settings to check if everything was alright and got that message".
Author
Owner

@zadjii-msft commented on GitHub (Sep 14, 2021):

Weird. I just copy pasted those two into a fresh 1.10 and saved the settings, then rebooted the terminal, and the warning didn't appear (either on hot-reload or reboot).

@zadjii-msft commented on GitHub (Sep 14, 2021): Weird. I just copy pasted those two into a fresh 1.10 and saved the settings, then rebooted the terminal, and the warning didn't appear (either on hot-reload or reboot).
Author
Owner

@joca-bt commented on GitHub (Sep 14, 2021):

Close all terminals, replace your settings.json with the code from below, launch a new terminal, and then save settings.

{
    "$schema": "https://aka.ms/terminal-profiles-schema/",

    "defaultProfile": "{0caa0dad-35be-5f56-a8ff-afceeeaa6101}",

    "profiles":
    {
        "list":
        [
            {
                "guid": "{0caa0dad-35be-5f56-a8ff-afceeeaa6101}",
                "name": "Command Prompt"
            }
        ]
    },

    "actions":
    [
        { "command": "find", "keys": "ctrl+f" },
        { "command": { "action": "findMatch", "direction": "next" }, "keys": "ctrl+shift+s" },
        { "command": { "action": "findMatch", "direction": "prev" }, "keys": "ctrl+shift+r" }
    ]
}
@joca-bt commented on GitHub (Sep 14, 2021): Close all terminals, replace your `settings.json` with the code from below, launch a new terminal, and then save settings. ``` { "$schema": "https://aka.ms/terminal-profiles-schema/", "defaultProfile": "{0caa0dad-35be-5f56-a8ff-afceeeaa6101}", "profiles": { "list": [ { "guid": "{0caa0dad-35be-5f56-a8ff-afceeeaa6101}", "name": "Command Prompt" } ] }, "actions": [ { "command": "find", "keys": "ctrl+f" }, { "command": { "action": "findMatch", "direction": "next" }, "keys": "ctrl+shift+s" }, { "command": { "action": "findMatch", "direction": "prev" }, "keys": "ctrl+shift+r" } ] } ```
Author
Owner

@zadjii-msft commented on GitHub (Sep 14, 2021):

and then save settings.

YEP that'll do it

image

we're not re-serializing findMatch args correctly. Thanks!

@zadjii-msft commented on GitHub (Sep 14, 2021): > and then save settings. YEP that'll do it ![image](https://user-images.githubusercontent.com/18356694/133323912-f50729c6-a979-45f1-9c82-0638b48d5ef6.png) we're not re-serializing `findMatch` args correctly. Thanks!
Author
Owner

@ghost commented on GitHub (Oct 4, 2021):

:tada:This issue was addressed in #11233, which has now been successfully released as Windows Terminal v1.10.2714.0.🎉

Handy links:

@ghost commented on GitHub (Oct 4, 2021): :tada:This issue was addressed in #11233, which has now been successfully released as `Windows Terminal v1.10.2714.0`.:tada: Handy links: * [Release Notes](https://github.com/microsoft/terminal/releases/tag/v1.10.2714.0) * [Store Download](https://www.microsoft.com/store/apps/9n8g5rfz9xk3?cid=storebadge&ocid=badge)
Author
Owner

@ghost commented on GitHub (Oct 4, 2021):

:tada:This issue was addressed in #11233, which has now been successfully released as Windows Terminal Preview v1.11.2731.0.🎉

Handy links:

@ghost commented on GitHub (Oct 4, 2021): :tada:This issue was addressed in #11233, which has now been successfully released as `Windows Terminal Preview v1.11.2731.0`.:tada: Handy links: * [Release Notes](https://github.com/microsoft/terminal/releases/tag/v1.11.2731.0) * [Store Download](https://www.microsoft.com/store/apps/9n8g5rfz9xk3?cid=storebadge&ocid=badge)
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#15203