Keybinding warnings are duplicated after #6635 #9651

Closed
opened 2026-01-31 02:00:13 +00:00 by claunia · 1 comment
Owner

Originally created by @zadjii-msft on GitHub (Jul 16, 2020).

Welp, I obviously didn't test that.

If you put a keybinding that raises a warning in the settings, we'll create two warnings for it, once for the keybindings, then the exact same warning for the command palette.

We should not.

Probably as easy as just not adding warnings from parsing the bindings as "commands" to the list of warnings, since the warnings have already been added.

Originally created by @zadjii-msft on GitHub (Jul 16, 2020). Welp, I obviously didn't test that. If you put a keybinding that raises a warning in the settings, we'll create _two_ warnings for it, once for the keybindings, then the _exact same warning_ for the command palette. We should _not_. Probably as easy as just not adding warnings from parsing the bindings as "commands" to the list of warnings, since the warnings have already been added.
Author
Owner

@zadjii-msft commented on GitHub (Jul 16, 2020):

Maybe it doesn't in the UI, but it certainly does in the tests.

@zadjii-msft commented on GitHub (Jul 16, 2020): Maybe it doesn't in the UI, but it certainly does in the tests.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#9651