@Nerixyz @Mm2PL see changes

I don't have a windows machine to test with but this works on my end

I don't have a windows machine to test with but this works on my end

This doesn't change anything on Windows or Linux, so it's fine. I don't know about common macOS key binds, so I can't comment on the chosen bind.

@pajlada

However, I think I would prefer to not set a hotkey in this specific scenario for macOS, rather than adding a macOS-specific hotkey.

I agree that it is a bit silly to have one specific default hotkey be different for one specific subset of users. Still, I would argue that the given solution is best.

The only other "solution" I can think of at the moment that would address each of this PR's reviews would be to set the following attribute, which modifies Qt's default behavior of swapping Control with Command on macOS:

QCoreApplication::setAttribute(Qt::AA_MacDontSwapCtrlAndMeta);

However, this is a terrible solution, mainly because this affects existing configurations. It also presents the same exact issue that this PR aims to solve, as Control+E is also a macOS system keybind.

Another possible solution was my original commit, that simply changed the hotkey for all systems, but I agreed with the other reviewers that a macOS-specific issue shouldn't affect non-macOS behavior. I also felt that replacing Ctrl+H altogether could possibly lead to this same exact issue re-occurring in the distant future.

macOS users can still add a custom hotkey to the "toggle local r9k" action as they see fit.

Of course. They can do this whether the given solution is implemented or not. The issue that this PR aims to solve, though, is that macOS users are forced to configure this manually should they want to utilize this feature.

If this was a more used feature I would agree to consider a macOS binding for it, but adding it adds complexity to all future hotkeys that I want to avoid unless necessary