Ignore default shortcuts for Vi/Emacs

[hluk]

Overrides some default shortcuts in the Vi/Emacs modes.

Enables Vi/Emacs navigation in menus.

Enables Ctrl+[ in Vi and Ctrl+G in Emacs to work in many other places as Esc key (for example, to hide menus, dialogs).

User can override shortcuts, but not some reserved ones in specific cases (mainly, if the item list or a menu has focus).

This also disables rerunning failed tests cases when run locally (not in a CI).

Fixes #3012

[hluk]

@janmejay Hi, Can you help review and test this?

I've enabled Vi mode for my normal use to test it - it seems to work well.

[janmejay]

Tested Emacs keys. Works great. There is a minor confusing element though, the keyboard-shortcuts list displays incorrect binding.

Haven't reviewed the code yet, plan to do it over the weekend.

[janmejay]

@hluk how do you run tests locally? I see scary things like

# Clean up old configuration.
rm -rf ~/.config/copyq.test

in test-linux.sh.

I see Dockerfile, is that the way to go?

[hluk]

@janmejay Thanks for testing!

Tested Emacs keys. Works great. There is a minor confusing element though, the keyboard-shortcuts list displays incorrect binding.

Yeah, that can be confusing; or the fact that user cannot set some shortcuts. These shortcuts are also shown in menus.

The easiest fix could be to update ShortcutButton::checkAmbiguousShortcuts to also check all navigation styles and add some indication to the shortcut buttons that a shortcut conflicts with a specific style.

[hluk]

how do you run tests locally?

The full command to build the test version of CopyQ and run specific tests:

cmake -G Ninja -DCMAKE_BUILD_TYPE=DEBUG -DCMAKE_INSTALL_PREFIX=$PWD/install \
  -DWITH_TESTS=1 -DWITH_QT6=1 -DPEDANTIC=ON -DCMAKE_CXX_FLAGS="-ggdb" \
  "$COPYQ_SOURCE_CODE_DIR"

ninja install

xnest.sh ./copyq tests navigationDefault navigationVi navigationEmacs

Here is the xnest.sh script. It starts Xephyr with Openbox to spin up a nested X11 environment for testing.

BTW, you don't have to remove the old test configuration ~/.config/copyq.test, that should be handled by the tests.