Merge #186: keymap: Keyboard Shortcuts dialog, named shortcut sets and the shortcut editor

# Conflicts:
#	crates/keymap/src/lib.rs

.github/workflows/keymap.yml

@@ -3,9 +3,9 @@ name: Keymap
 on:
   push:
     branches: [main]
-    paths: ["crates/keymap/**", "crates/settings/**", "Cargo.toml", "Cargo.lock", ".github/workflows/keymap.yml"]
+    paths: ["crates/keymap/**", "crates/settings/**", "crates/ui/**", "crates/i18n/**", "Cargo.toml", "Cargo.lock", ".github/workflows/keymap.yml"]
   pull_request:
-    paths: ["crates/keymap/**", "crates/settings/**", "Cargo.toml", "Cargo.lock", ".github/workflows/keymap.yml"]
+    paths: ["crates/keymap/**", "crates/settings/**", "crates/ui/**", "crates/i18n/**", "Cargo.toml", "Cargo.lock", ".github/workflows/keymap.yml"]
   workflow_dispatch:
 
 permissions:
@@ -32,12 +32,14 @@ jobs:
           key: craft-keymap-linux
       - run: cargo fmt -p craft-keymap --check
       - run: cargo clippy -p craft-keymap --all-targets --no-default-features --locked -- -D warnings
-      - run: cargo clippy -p craft-keymap --all-targets --all-features --locked -- -D warnings
-      # Doctests run once, in the all-features step.
+      # `snapshot` covers `ui` and `egui` and compiles the dialog screenshots, which need a GPU
+      # to run, so the test steps leave them out.
+      - run: cargo clippy -p craft-keymap --all-targets --features snapshot --locked -- -D warnings
+      # Doctests run once, in the `ui` step.
       - name: Test core
         run: cargo test -p craft-keymap --no-default-features --lib --tests --locked
-      - name: Test egui dispatch
-        run: cargo test -p craft-keymap --all-features --locked
+      - name: Test egui dispatch and the shortcuts dialog
+        run: cargo test -p craft-keymap --features ui --locked
   native:
     timeout-minutes: 15
     name: Keymap (${{ matrix.os }})
@@ -54,7 +56,7 @@ jobs:
       - uses: Swatinem/rust-cache@d1527c8f67ef9af357290475d4af4b0fab8f12e0 # v2.9.3
         with:
           key: craft-keymap-${{ matrix.os }}
-      - run: cargo test -p craft-keymap --all-features --locked
+      - run: cargo test -p craft-keymap --features ui --locked
   wasm:
     timeout-minutes: 10
     name: Keymap WASM
@@ -69,4 +71,4 @@ jobs:
       - uses: Swatinem/rust-cache@d1527c8f67ef9af357290475d4af4b0fab8f12e0 # v2.9.3
         with:
           key: craft-keymap-wasm
-      - run: cargo check -p craft-keymap --all-features --locked --target wasm32-unknown-unknown
+      - run: cargo check -p craft-keymap --features ui --locked --target wasm32-unknown-unknown