[Bug] Keyboard shortcuts in editor for macOS no long work in 1.34.0

[neutron-ice]

Operating System

MacOS

What's the issue you encountered?

In the Pattern Editor keyboard shortcuts, like CMD+C or CMD+V and some rest no longer work.

Downgraded to the previous version 1.33.2 – everything is okey.

How can the issue be reproduced?

When in settings assign new shortcut, e.g. press "CMD+V" instead I see "CTRL+V". When press CTRL+V instead I see "CMD+V".

But anyway nothing work in editor, irrelevant it is "cmd+v" or "ctrl+v" – work only nowhere else defined Alt+V.
Undo (CMD+Z) work like nowhere else defined "CMD+Backspace/Del".

keysbug.mp4

ImHex Version

1.34.0

ImHex Build Type

• Nightly or built from sources

Installation type

Portable

Additional context?

Apple M1, macOS 12.6

[paxcut]

There seems to be two separate errors reported. One is about the pattern editor where the mac shortcuts no longer work. The pattern editor is undergoing a transition where the shortcuts will be part of the shortcuts manager and will be editable like all the other shortcuts, but only the part that removed the old shortcut code made it to the release. Once the pr that adds the new shortcuts is merged then it should work again.
The other issue is about editing shortcuts in the hex editor and I know nothing about that.

[sisco0]

Regarding the cmd vs ctrl inconvenience,
@neutron-ice
Could you kindly check if keys are properly interpreted at other applications/websites? e.g. https://en.key-test.ru

[neutron-ice]

Could you kindly check if keys are properly interpreted at other applications/websites?

Yep, everything is properly interpreted. At same time in same environment previous version (1.33.2) works correctly, something happened with new version:

nobugs.mp4

[root670]

It seems Cmd and Ctrl have been swapped. For example undo should be Cmd-Z but only works with Ctrl-Z now. When pressing cmd or ctrl in the shortcut screen they appear swapped (i.e. press Cmd and it shows up as Ctrl)

[paxcut]

There has been a recent commit that seems to be aimed at this issue but it is missing two important details

1. The pattern editor shortcut code has been moved to be part of the shortcut manager so the changes made in the commit will not be relevant if pr feat: full syntax highlights and interactive compiler error reporting indicated using a red squiggly line under the affected symbols. #1700 is merged.
2. The main issue becomes that the shortcuts for Mac are done incorrectly for the entire Imhex and not only for the pattern editor. Searching for changes that affect mac shortcuts point to pr Support macOS keyboard shortcuts #889 but that code has been in place since Jan 2023 so there must be more recent changes that are causing the the issue as it is reported that 1.33.2 works fine.

[gkpln3]

It appears that each time i press CMD it interprets it as CTRL instead, and CTRL is interpreted as CMD.

[paxcut]

Does that swapping of keys happen everywhere on Imhex? The pattern editor and the hax editor have independent mappings for keys.In fact if you check the shortcut tab in settings you'll see all the different places where mappings could be different which of course includes the shortcut tab window itself.