Fix binding of configparser comment chars

Neither semicolon `;` nor the number symbol `#` could be bound as they
were parsed by configparser as a comment. Fixed by allowing to bind
these via <semicolon> and <number> instead.

Author: karlch

tests/unit/gui/test_eventhandler.py

@@ -53,6 +53,7 @@ def test_temp_key_storage_clears_text(storage, qtbot):
         ),
         (Qt.Key.Key_Colon, Qt.KeyboardModifier.NoModifier, ":", ("<colon>",)),
         (Qt.Key.Key_Equal, Qt.KeyboardModifier.NoModifier, "=", ("<equal>",)),
+        (Qt.Key.Key_Semicolon, Qt.KeyboardModifier.NoModifier, ";", ("<semicolon>",)),
     ],
 )
 def test_keyevent_to_sequence(qtkey, modifier, keyname, expected):

vimiv/gui/eventhandler.py

@@ -275,18 +275,20 @@ def _get_base_keysequence(event: QKeyEvent) -> SequenceT:
         Qt.Key.Key_PageDown: "<page-down>",
         Qt.Key.Key_Delete: "<delete>",
     }
-    separator_keys = {
+    configparser_keys = {
         Qt.Key.Key_Colon: "<colon>",
         Qt.Key.Key_Equal: "<equal>",
+        Qt.Key.Key_Semicolon: "<semicolon>",
+        Qt.Key.Key_NumberSign: "<number>",
     }
     if event.key() in special_keys:
         # Parse shift here as the key does not support it otherwise
         text = special_keys[event.key()]  # type: ignore
         if event.modifiers() & Qt.KeyboardModifier.ShiftModifier:
             return "<shift>", text
         return (text,)
-    if event.key() in separator_keys:  # Required as configparser crashes otherwise
-        return (separator_keys[event.key()],)  # type: ignore
+    if event.key() in configparser_keys:  # Required as configparser crashes otherwise
+        return (configparser_keys[event.key()],)  # type: ignore
     if event.text().isprintable():
         return (event.text(),)
     return (QKeySequence(event.key()).toString().lower(),)