feat/macos_modifiers_fix: switch ctrl with cmd and fix key recognition

Added AA_MacDontSwapCtrlAndMeta attribute before QApplication
initialization.
Replaced isprintable check with isalnum and isascii to fix key
recognition with modifiers and prevent their change to special
symbols/letters with option key.

vimiv/app.py

@@ -6,6 +6,7 @@
 
 from vimiv.qt.gui import QIcon
 from vimiv.qt.widgets import QApplication
+from vimiv.qt.core import Qt, MacDontSwapCtrlAndMeta
 
 import vimiv
 from vimiv import api, utils
@@ -25,6 +26,8 @@ def __init__(self, *qtargs: str) -> None:
             qtargs: Arguments passed directly to the QApplication.
         """
         _logger.debug("Passing %s to qt", utils.quotedjoin(qtargs))
+        if MacDontSwapCtrlAndMeta:
+            super().setAttribute(MacDontSwapCtrlAndMeta, True)
         super().__init__([vimiv.__name__, *qtargs])
         self.setApplicationVersion(vimiv.__version__)
         self.setDesktopFileName(vimiv.__name__)

vimiv/gui/eventhandler.py

@@ -289,6 +289,6 @@ def _get_base_keysequence(event: QKeyEvent) -> SequenceT:
         return (text,)
     if event.key() in configparser_keys:  # Required as configparser crashes otherwise
         return (configparser_keys[event.key()],)  # type: ignore
-    if event.text().isprintable():
+    if event.text().isalnum() and event.text().isascii():
         return (event.text(),)
     return (QKeySequence(event.key()).toString().lower(),)

vimiv/qt/core.py

@@ -7,8 +7,10 @@
 
 if qt.USE_PYQT5:
     from PyQt5.QtCore import *
+    MacDontSwapCtrlAndMeta = Qt.AA_MacDontSwapCtrlAndMeta
 elif qt.USE_PYQT6:
     from PyQt6.QtCore import *
+    MacDontSwapCtrlAndMeta = Qt.ApplicationAttribute.AA_MacDontSwapCtrlAndMeta
 elif qt.USE_PYSIDE6:
     # TODO remove useless-suppression once we add PySide6 back to pylint toxenv
     # pylint: disable=no-name-in-module,undefined-variable,useless-suppression
@@ -17,6 +19,7 @@
 
     BoundSignal = SignalInstance
     QT_VERSION_STR = qVersion()
+    MacDontSwapCtrlAndMeta = None
 
 if qt.USE_PYQT:  # Signal aliases
     # pylint: disable=used-before-assignment