VKey assertion failure #14854

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

Originally created by @Rosefield on GitHub (Aug 12, 2021).

Originally assigned to: @lhecker on GitHub.

Windows Terminal version (or Windows build number)

dev origin/main

Other Software

I use the colemak keyboard layout primarily, but it also seems to happen with regular qwerty too.

Steps to reproduce

Launch the application, assert at line https://github.com/microsoft/terminal/pull/10917/files#diff-ac308dd6d23797c02f8be4591060500c2474856a7c3f483ac5cff219b3fc16a2R720 fails

Expected Behavior

terminal actually loads

Actual Behavior

exception thrown.

Originally created by @Rosefield on GitHub (Aug 12, 2021). Originally assigned to: @lhecker on GitHub. ### Windows Terminal version (or Windows build number) dev origin/main ### Other Software I use the colemak keyboard layout primarily, but it also seems to happen with regular qwerty too. ### Steps to reproduce Launch the application, assert at line https://github.com/microsoft/terminal/pull/10917/files#diff-ac308dd6d23797c02f8be4591060500c2474856a7c3f483ac5cff219b3fc16a2R720 fails ### Expected Behavior terminal actually loads ### Actual Behavior exception thrown.
Author
Owner

@Rosefield commented on GitHub (Aug 12, 2021):

@lhecker sorry :(

@Rosefield commented on GitHub (Aug 12, 2021): @lhecker sorry :(
Author
Owner

@DHowett commented on GitHub (Aug 12, 2021):

Ha! Never apologize for that. It's an excellent catch. 😄

@lhecker will def. need to take a look at this one.

@DHowett commented on GitHub (Aug 12, 2021): Ha! Never apologize for that. It's an excellent catch. :smile: @lhecker will def. need to take a look at this one.
Author
Owner

@lhecker commented on GitHub (Aug 12, 2021):

@Rosefield Can you post the "actions" array from your settings.json? While I know exactly what I forgot to keep in mind (not all scan-codes have virtual keys), I'm unfortunately unable to reproduce this and thus can't test if my fix is correct.

@lhecker commented on GitHub (Aug 12, 2021): @Rosefield Can you post the "actions" array from your settings.json? While I know exactly what I forgot to keep in mind (not all scan-codes have virtual keys), I'm unfortunately unable to reproduce this and thus can't test if my fix is correct.
Author
Owner

@Rosefield commented on GitHub (Aug 12, 2021):

This is my settings.json
https://gist.github.com/Rosefield/b7765da9fb8aa6aee7518b4d59d329cb

@Rosefield commented on GitHub (Aug 12, 2021): This is my settings.json https://gist.github.com/Rosefield/b7765da9fb8aa6aee7518b4d59d329cb
Author
Owner

@lhecker commented on GitHub (Aug 12, 2021):

You can fix the crash by removing this invalid action:

{
    "command": "unbound",
    "keys": ""
},

I find it a bit puzzling that we don't check for such mistakes. 😯

@lhecker commented on GitHub (Aug 12, 2021): You can fix the crash by removing this invalid action: ```json { "command": "unbound", "keys": "" }, ``` I find it a bit puzzling that we don't check for such mistakes. 😯
Author
Owner

@Rosefield commented on GitHub (Aug 12, 2021):

Interesting, I definitely didn't add that myself so there might be a bug in the settings UI that set it? It sounds like there might be a bug still, but maybe not the bug that I thought it was.

@Rosefield commented on GitHub (Aug 12, 2021): Interesting, I definitely didn't add that myself so there might be a bug in the settings UI that set it? It sounds like there might be a bug still, but maybe not the bug that I thought it was.
Author
Owner

@ghost commented on GitHub (Aug 31, 2021):

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

Handy links:

@ghost commented on GitHub (Aug 31, 2021): :tada:This issue was addressed in #10945, which has now been successfully released as `Windows Terminal Preview v1.11.2421.0`.:tada: Handy links: * [Release Notes](https://github.com/microsoft/terminal/releases/tag/v1.11.2421.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#14854