diff --git a/AGENTS.md b/AGENTS.md index aab3b1b..4afe1a9 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -16,11 +16,6 @@ Nutzerorientierte Einführung: [`README.md`](README.md). - `versapad_data.py` — Decoding für Anzeige: JSON laden, HID-Keycode/ Consumer-Usage/Modifier → lesbarer Text, Grid-Geometrie (`index = spalte*5+reihe`), `hid_key_choices()`/`consumer_choices()`. - Seit 2026-08-28 zusätzlich `tk_event_to_hid()` (Tk-Tastendruck → - HID-Keycode+Modifier, siehe Tastendruck-Erkennung unten) und - `macro_slot_choices()`/`macro_slot_from_choice()` für die Slot-Auswahl. - `action_label()`/`annotate_profile()` nehmen die Makrotabelle optional - entgegen und zeigen dann statt „Makro (Slot 7)" die echte Tastenfolge. `app_dir()` liefert das Verzeichnis für die eigene Config (`versapad_combined.DEFAULT_PATH`) — bei der `.exe` der Installations- ordner, sonst der Projektordner, siehe Installierbarkeit-Notiz unten. @@ -59,21 +54,12 @@ Nutzerorientierte Einführung: [`README.md`](README.md). `read_profile_names()` liest nur die Namen (kein Board-Zugriff, fuer Tab-Beschriftungen im Nur-Lese-Modus, siehe Installierbarkeit-Notiz). -- `versapad_keylayout.py` (seit 2026-08-29) — passive Abfragen ans aktive - Windows-Tastaturlayout: `hid_for_vk()` (Virtual-Key → Scan-Code → HID über - `MapVirtualKeyW`) und `key_name()` (Beschriftung über `GetKeyNameTextW`). - Optional: auf Nicht-Windows/ohne ctypes bleibt `AVAILABLE` False und - `versapad_data` faellt auf seine US-Naeherung zurueck. Zur Abgrenzung - gegen die WinAPI-Verbote siehe Domaenenregeln. - **UI:** - `desktop_viewer.py` — Tkinter, flaches Design, drei unabhängige Modi (Nur lesen / Live-Sync / Programmiermodus, siehe Kritische Domänenregeln), Tray-Icon statt Taskleisten-Minimierung, Info-Button mit MCP-Doku. - `action_dialog.py` — Bearbeiten-Dialoge (`ActionEditDialog`, - `MacroStepsDialog`) für den Programmiermodus. Gemeinsame Basis - `_ModalDialog` (Positionierung über dem Elternfenster, Enter/Escape, - Verteilung der Tastenevents) und `_KeyCapture` (Tastendruck-Erkennung). + `MacroStepsDialog`) für den Programmiermodus. **MCP-Server:** - `versapad_mcp_server.py` — registriert als projektgebundener MCP-Server @@ -107,8 +93,9 @@ Nutzerorientierte Einführung: [`README.md`](README.md). Fehler hier live aufgedeckt. Vollständige Modulübersicht mit Zeilenreferenzen bei Bedarf direkt im Code -nachschlagen. Menschenlesbare Referenzdoku (Architektur, Datenmodell, -Protokoll) liegt in `docs/`, siehe „Dokumentation und Verifikation" unten. +nachschlagen — die Dateien sind klein genug, dass eine separate +`docs/current-architecture.md` hier keinen Mehrwert hätte (siehe +Dokumentation und Verifikation unten für die Größeneinschätzung). ## Kritische Domänenregeln @@ -127,18 +114,6 @@ Protokoll) liegt in `docs/`, siehe „Dokumentation und Verifikation" unten. (`profile_names`) — die Firmware-Structs haben keinen Platz für einen String (Header exakt 32B, jedes Profil exakt 236B, alles verplant). Sie landen nie aufs Board, egal welcher Schreibpfad benutzt wird. -- **Notizen** (freier Text je Button/Encoder-Aktion, seit 2026-08-15) sind - aus demselben Grund rein lokal: leben im `"note"`-Feld JEDER Action - (`{"type","data","note"}`), nicht in einer separaten Struktur. `to_binary`/ - `pack_config` ignorieren das Feld beim Schreiben (liest nur type/data), - `from_binary` liefert frisch vom Board immer `note=""` (Board kennt keine - Notizen) — `versapad_combined.merge_notes(neu, alt)` kopiert bestehende - Notizen nach jedem `load_from_board()`/`fetch_from_board()` zurück, sonst - gingen sie bei jedem Board-Refresh verloren. Action-Typ wechseln - (`set_button_key` etc.) darf die Notiz NICHT loeschen (baut die neue - Action ueber `_replace_action()`/`dlg.action["note"]` mit der alten Notiz), - nur `set_button_note()`/`set_encoder_note()`/das Notiz-Feld im - Programmiermodus-Dialog aendern sie gezielt. - Der COM-Port ist exklusiv. Live-Sync und Programmiermodus schalten sich gegenseitig aus (ein `VersaPadLink` kann nicht von zwei Konsumenten gleichzeitig genutzt werden); VersaGUI läuft als Tray-App dauerhaft im @@ -151,28 +126,6 @@ Protokoll) liegt in `docs/`, siehe „Dokumentation und Verifikation" unten. vorhanden, und fällt sonst auf die klassischen `versapad_config{1,2,3}.json` zurück — beide Ansichten müssen dieselbe Quelle zeigen, sonst wirkt eine Bearbeitung "verschwunden". -- **Config-Pfad darf NIE im Installationsverzeichnis liegen (2026-08-15, - Datenverlust-Bug):** `build_and_deploy.ps1` räumt sein Zielverzeichnis vor - jedem Deploy komplett ab (`Remove-Item $localDir -Recurse -Force`). Nachdem - `app_dir()` am selben Tag auf genau dieses Verzeichnis gezeigt hatte, hat - **jeder Rebuild die Nutzer-Config mitgelöscht**; die App hat sie danach - kommentarlos leer vom Board neu aufgebaut (`load_or_fetch()` → - `fetch_from_board()`), wodurch alle Notizen weg waren. Besonders tückisch: - Board-Bindings und LEDs sahen danach völlig normal aus, nur die rein - lokalen Felder (Notizen, Profilnamen) fehlten -- der Schaden ist also - unsichtbar, wenn man nur aufs Grid schaut. Fix: `app_dir()` liefert jetzt - `%APPDATA%\VersaPadViewer` (Roaming), getrennt vom Installationsordner in - `%LOCALAPPDATA%`; zusätzlich rettet `build_and_deploy.ps1` vorgefundene - `versapad_config*.json` aus dem Zielverzeichnis über den Deploy hinweg - (für Altinstallationen). **Regel für künftige Änderungen: Programm- und - Datenverzeichnis nie zusammenlegen, egal wie praktisch "alles an einem - Ort" klingt.** -- **Nur EIN Config-Pfad, unabhängig vom Startweg (2026-08-15):** `app_dir()` - hat bewusst KEINEN `sys.frozen`-Zweig mehr. Vorher sah die gebaute `.exe` - eine andere Datei als der aus dem Quellcode gestartete MCP-Server, was zu - zwei auseinanderlaufenden Configs führte (die `.exe` zeigte ein anderes - Profil 0 als der MCP-Server meldete). Wer hier wieder nach Startweg - unterscheidet, baut denselben Bug erneut ein. - **Installierbarkeit verbessert 2026-08-14:** Drei Probleme beim Weitergeben an andere Leute behoben. (1) `build_and_deploy.ps1` starb bei fehlenden Paketen (pyinstaller/pystray/pillow) kommentarlos, v.a. bei @@ -187,10 +140,7 @@ Protokoll) liegt in `docs/`, siehe „Dokumentation und Verifikation" unten. Funktion) liefert bei der gebauten `.exe` deren Installationsordner (`sys.executable`-Verzeichnis), sonst den Projektordner (`__file__`- Verzeichnis) -- `DEFAULT_PATH` hängt jetzt daran, landet also immer neben - der laufenden Installation. **Diese app_dir()-Variante ist seit - 2026-08-15 überholt und war aktiv schädlich, siehe oben "Config-Pfad - darf nie im Installationsverzeichnis liegen".** - `versapad_data.CONFIG_PATHS` (Lese-Interop + der laufenden Installation. `versapad_data.CONFIG_PATHS` (Lese-Interop mit der C#-VersaGUI) bleibt bewusst auf dem Desktop, siehe oben. (3) Fehlte die Config UND war kein Board erreichbar, blockierte das Tool mit einer Fehlermeldung statt zu starten. Fix: `load_or_fetch()` legt jetzt bei @@ -204,18 +154,6 @@ Protokoll) liegt in `docs/`, siehe „Dokumentation und Verifikation" unten. Namen jetzt immer aus der kombinierten JSON (`combined["profile_names"]` bzw. `versapad_combined.read_profile_names()`), eine einzige Quelle der Wahrheit für alle Frontends. -- **Herkunft der drei Punkte oben + Tab-Umbenennen-Fix:** ursprünglich per - Cherry-Pick aus Julian Appels `dev/jappel`-Branch auf `main` übernommen - (der sich zeitgleich mit dem eigenen COM-Port-Fix entwickelt hatte, kein - Fork-Sync-Automatismus -- manuell gegengelesen). Auf `main` bewusst NICHT - übernommen wurde damals der `dist/VersaPadViewer`-Build-Zielpfad (statt - `%LOCALAPPDATA%\VersaPadViewer`) sowie `.mcp.json` und die `docs/*.md`- - Referenzdateien. **2026-08-15: `main` wurde zurück in `dev/jappel` - gemerged**, damit landen beide Linien wieder in einem Branch -- inklusive - `.mcp.json`, `docs/*.md` und dem `%LOCALAPPDATA%`-Zielpfad (der hat sich - im Merge durchgesetzt, siehe Config-Pfad-Bullets oben). Der Notes-Feature- - Teil dieser Historie ist der Grund, warum `docs/data-model.md` das - `"note"`-Feld nachträglich braucht. - **Bug behoben 2026-08-08:** `server.py` (`vp.load_profile()`) und `desktop_viewer.py` (`_current_profile_view()`) lasen im Nur-Lese-Modus hart von den Desktop-JSONs -- fehlten sie (z.B. User loescht sie), gab es @@ -242,135 +180,12 @@ Protokoll) liegt in `docs/`, siehe „Dokumentation und Verifikation" unten. `vcomb.DEFAULT_PATH`, fällt nur bei fehlender/kaputter Datei auf `default_combined()` zurück. Bei jedem "komisches Layout"-Report hier immer ALLE 3 Profile prüfen, nicht nur das gemeldete. -- **WinAPI: was verboten bleibt und was nicht.** Verboten sind - **Fenstermanipulation am eigenen Fenster** (`SetWindowLongW`, - `ShowWindow`) und **globale Eingabehooks** (`SetWindowsHookEx`) — genau - diese Kombination hat laut Speicher-Notiz „keine Selbstversteck- - Fenstertricks" in einem anderen Projekt Bitdefender-Fehlalarme ausgeloest. - Erlaubt und seit 2026-08-29 in Benutzung sind **passive Layout-Abfragen** - (`MapVirtualKeyW`, `GetKeyNameTextW` in `versapad_keylayout.py`): sie - lesen nur die Tastaturbelegung, fassen weder Fenster noch Prozesse noch - den Eingabestrom an; die offizielle VersaGUI (C#) benutzt - `GetKeyNameText()` fuer denselben Zweck. Die Grenze verlaeuft also nicht - bei „ctypes", sondern bei „greift ins System ein". -- **Normales Fenster mit Titelleiste (seit 2026-08-29).** Von 2026-08-15 - bis dahin lief das Fenster randlos (`overrideredirect(True)`) — das kostet - unter Windows zwingend den Taskleisten-Eintrag. Als der ausdruecklich - gebraucht wurde, gab es nur drei Wege: Titelleiste zurueck, - `SetWindowLongW`+WS_EX_APPWINDOW (siehe Verbot oben) oder ein - unsichtbares Proxy-Fenster. Der User hat sich fuer die Titelleiste - entschieden. Damit sind ersatzlos entfallen: ziehbare Kopfzeile - (`_start_move`/`_on_move`), Groessen-Anfasser unten rechts - (`_start_resize`/`_on_resize`, ersetzt durch `self.minsize()`), die - eigenen `✕`/`—`-Knoepfe und der ``-Handler, der Minimieren ins Tray - umgeleitet hat. **Wer das Fenster wieder randlos machen will, nimmt dem - User den Taskleisten-Eintrag weg** — nicht ohne Ruecksprache. -- Schliessen (`✕`) legt weiterhin ins Tray statt zu beenden (wie die - offizielle VersaGUI, das Programm laeuft im Hintergrund weiter), - Minimieren geht jetzt aber ganz normal in die Taskleiste. `Escape` - minimiert ebenfalls, statt wie frueher ins Tray zu legen — mit - Taskleisten-Eintrag waere „verschwindet spurlos" die unangenehmere - Ueberraschung. -- Fensterhöhe muss zum Karteninhalt passen: seit die Karten eine Notizzeile - haben (`CARD_H` 84 → 114) braucht das Fenster ~960px, sonst liegen - Encoder-Bereich und Fußzeile unterhalb des Rands. Wer `CARD_H` ändert, - muss `geometry()` mit anpassen. -- **Dialogposition wird gegen das ELTERNFENSTER begrenzt, nie gegen - `winfo_screenwidth()` (2026-08-29):** Tk meldet dort nur den - Hauptbildschirm. Die erste Fassung hat damit geklemmt — lag das - Hauptfenster auf einem zweiten Monitor, zog genau diese Begrenzung den - Dialog zurueck an den Rand des ersten. Passt der Dialog nicht ins - Elternfenster (der Makro-Schritte-Dialog ist breiter als der - Action-Dialog), wird er an dessen linker oberer Ecke ausgerichtet statt - zentriert. -- **Dialoge öffnen über dem Hauptfenster, nicht in der Bildschirmecke - (2026-08-28):** `_ModalDialog` baut sich `withdraw()`n auf, positioniert - sich in `run()` per `_center_on_parent()` und wird erst dann - `deiconify()`t. Ohne das legt Tk jeden Toplevel bei `+0+0` an — bei 20 - Tasten hintereinander wandert der Blick jedes Mal in die linke obere Ecke. - Das `withdraw()` gehört zwingend dazu, sonst blitzt der Dialog dort auf, - bevor er springt. -- **Verschachtelte Dialoge geben den Grab zurück:** `MacroStepsDialog` läuft - im `ActionEditDialog`. `_finish()` ruft `grab_release()`, was den Grab des - *aufrufenden* Dialogs mit wegnimmt — `run()` setzt ihn deshalb am Ende - wieder, wenn der Parent ein `_ModalDialog` ist. -- **Panel-Höhe im `ActionEditDialog` ist fix** (`grid_propagate(False)`): - sonst springt die Fenstergröße bei jedem Typwechsel (Keine/Taste/Makro/…) - und OK/Abbrechen wandern unter dem Mauszeiger weg. -- **Kopieren/Einfügen zwischen Tasten** (Rechtsklickmenü bzw. Strg+C/Strg+V - auf der Karte unter dem Mauszeiger) benutzt eine reine In-Memory-Ablage - (`self._clip`), **nicht** die System-Zwischenablage — dort liegen - Action-Dicts, kein Text. Immer `copy.deepcopy()`, sonst teilen sich zwei - Tasten dasselbe Dict und eine spätere Bearbeitung ändert beide. „Leeren" - behält die Notiz (gleiche Regel wie beim Typwechsel im Dialog, siehe - Notizen-Bullet oben). Eine kopierte LED-Farbe auf einen Encoder - einzufügen ist wirkungslos (Encoder haben keine eigene LED) — das ist - Absicht, kein Fehler. -- Die Toolbar des Programmiermodus ist bei der Standardbreite (790px) fast - voll — zusätzliche Hinweistexte dort kurz halten, sonst werden sie rechts - abgeschnitten. -- **Tastendruck-Erkennung (seit 2026-08-28, auf expliziten User-Wunsch):** - Tasten lassen sich im Bearbeiten-Dialog per „⌨ Taste drücken" erfassen, - Makro-Schritte zusätzlich als Folge am Stück („⏺ Folge aufnehmen"). - Umgesetzt **ausschließlich über Tk-Fenster-Events** (``/ - `` auf dem Dialog-Toplevel, ausgewertet in - `versapad_data.tk_event_to_hid()`) — **weiterhin kein WinAPI-Hook** - (`SetWindowsHookEx` o.ä.), aus demselben AV-Fehlalarm-Grund wie bei den - Fensterverstecktricks. Wer hier auf einen globalen Hook „aufrüstet", - baut genau dieses Risiko ein. Konsequenzen, die so bleiben müssen: - - Erkannt wird nur, was das fokussierte Fenster erreicht — Win+L, - Strg+Alt+Entf und andere vom System abgefangene Kombinationen nicht. - - **Welche Taste gemeint ist, wird ueber die PHYSISCHE POSITION - aufgeloest, nie ueber das erzeugte Zeichen** (2026-08-29, nach - Fehlermeldung aus der Praxis). HID-Keycodes *sind* Positionen: das Board - sendet eine Position, erst Windows macht daraus ein Zeichen. Die erste - Fassung ging ueber keysym/Zeichen und war auf deutschem Layout - entsprechend kaputt — Y und Z landeten vertauscht auf dem Board, ÄÖÜ - und #/+ waren gar nicht erfassbar. Reihenfolge jetzt: (1) benannte - Tasten ueber den keysym, (2) Zeichentasten ueber - `versapad_keylayout.hid_for_vk()`, (3) die alte Naeherung nur noch als - Fallback ohne WinAPI. **Nicht auf "Zeichen auswerten" zurueckbauen** — - das ist genau der Fehler, der hier behoben wurde. - - Schritt (1) ist keine Bequemlichkeit, sondern noetig: `MapVirtualKeyW` - liefert fuer die Pfeiltasten denselben Scan-Code wie fuer ihre - Numpad-Zwillinge (gemessen: VK_LEFT und VK_NUMPAD4 beide 0x4B, das - E0-Praefix von `MAPVK_VK_TO_VSC_EX` bleibt dort aus). Ueber den - Scan-Code allein waeren Pfeiltasten nicht von Numpad-Tasten zu - unterscheiden. - - Das Dropdown bleibt daneben stehen (Korrekturmöglichkeit), es ersetzt - die Erkennung nicht und wird von ihr nicht ersetzt. - - Modifier werden doppelt ermittelt (selbst mitgeführte Press/Release-Bits - *plus* `event.state`), weil ein KeyRelease bei Fokuswechsel verloren - gehen kann. - - `_KeyCapture` hängt an einem `tk.Label`, nicht an einem `tk.Button`: - Buttons reagieren per Klassen-Binding selbst auf Leertaste/Enter und - würden die laufende Aufnahme mit ihrem eigenen Klick beantworten. - - Makro-Schritte filtern das Win-Bit weg (`allow_win=False`), passend zur - Firmware-Regel oben. -- **Tastenbeschriftungen kommen vom aktiven Layout** (seit 2026-08-29). - `versapad_data.hid_key_name()` fragt fuer Zeichentasten - `GetKeyNameTextW` (deutsch: HID 0x1C → „Z", 0x34 → „ä"); fuer alles - andere bleiben die gepflegten deutschen Namen aus `_SPECIAL_KEYS` - („Enter", „Bild↑", „Num5") — die lesen sich besser als das, was Windows - liefert („EINGABE", „4 (ZEHNERTASTATUR)"). Ohne Layout-Abfrage - (Nicht-Windows) faellt alles auf die alte US-Naeherung zurueck. - Konsequenzen, die man kennen muss: - - Bestehende Belegungen aendern ihre **Anzeige**, nicht ihre Daten. Wer - frueher im Dropdown „Z" gewaehlt hat, bekam 0x1D — das steht so in der - Config und zeigt jetzt wahrheitsgemaess „Y", weil es auf dieser Tastatur - ein Y tippt. Das ist keine Regression, sondern der sichtbar gewordene - Altfehler. - - Namen sind Schluessel (Dropdown, `hid_key_code_for_name()`) und muessen - eindeutig bleiben. Es gibt echte Kollisionen: auf deutschem Layout heisst - HID 0x31 schlicht „#" — den Namen trug bisher HID 0x32. Der Layoutname - gewinnt, der verdraengte US-Name wird als „# (US-Layout)" gekennzeichnet - statt verworfen, damit die Taste ansprechbar bleibt. - - `hid_key_code_for_name()` akzeptiert weiterhin beide Schreibweisen, bei - Kollision gewinnt das Layout. Fuer den MCP-Server heisst das: - `set_button_key(key="Z")` trifft die Taste, die auf dieser Tastatur ein - Z tippt (0x1C), nicht mehr die US-Position 0x1D. - - Ein Layoutwechsel zur Laufzeit wird nicht bemerkt (Namen werden einmal - ermittelt und behalten). +- HID-Tasten-Auswahl im Programmiermodus ist ein Dropdown, kein + Tastendruck-Capture (bewusst — kein WinAPI-Hook, um keinen AV-Fehlalarm + wie bei den Fensterverstecktricks in anderen Projekten zu riskieren). +- Zeichentasten-Labels zeigen eine US-Layout-Näherung, nicht das tatsächlich + aktive Windows-Tastaturlayout (die echte VersaGUI löst das über + `GetKeyNameText()`, das bilden wir ohne WinAPI-Call nicht nach). - Tk-Aufrufe (`self.after()`, Widget-Konfiguration) NIE direkt aus einem Fremdthread (Serial-Thread, pystray-Thread) — hat in einer früheren Version einen stillen Absturz verursacht. Threads legen Ergebnisse nur in @@ -415,28 +230,6 @@ Byte-Identität), nicht nur gegen Beispieldaten. Vor jedem Schreibvorgang aufs Board: erst mit unveränderten Daten testen (Identity-Write), bevor echte Änderungen geschrieben werden. -**Änderungen an der Config immer aus der Sicht prüfen, die die laufende App -hat (2026-08-15 gelernt):** Claudes Bash- und PowerShell-Werkzeuge sehen -unter `C:\Users\chris\AppData\...` teilweise *unterschiedliche* Dateien — -derselbe Pfad lieferte gleichzeitig 29826 Bytes (Bash/MCP-Server) und -28455 Bytes (PowerShell/`.exe`). Konkret heißt das: **Config-Schreibvorgänge -über den versapad-MCP-Server (`save_local()`) erreichen die installierte -`.exe` nicht zuverlässig.** Das hat eine Fehlersuche über viele Runden -verschleppt, weil "Notizen sind in der Datei" (Bash) und "App zeigt keine -Notizen" gleichzeitig stimmten. Vorgehen: -- Config-Dateien, die die installierte App lesen soll, über **PowerShell** - schreiben/prüfen (`Get-Item`, dort ausgeführtes `python`), nicht über Bash. -- Bei "Änderung wirkt nicht"-Symptomen als Erstes Dateigröße/mtime aus - *beiden* Sichten vergleichen, bevor Code-Ursachen gesucht werden. -- `Z:\Git\...` (SMB-Share) ist von diesem Effekt nicht betroffen — Quellcode - und Build verhalten sich normal. - -Verlässlich zum Ziel führt bei solchen Widersprüchen ein temporäres -Diagnose-Modul, das im gebauten Bundle mitläuft und `app_dir()`, -`DEFAULT_PATH`, `os.stat()` und den tatsächlich geladenen JSON-Inhalt in eine -Datei schreibt — damit war die Ursache in einem Durchlauf sichtbar, nachdem -Vermutungen mehrfach danebenlagen. - ## Naming Code-Identifier englisch (Python-Konvention: `pack_config`, `read_macros`, @@ -453,6 +246,8 @@ selbst vorgegeben (`SAction.data`), dort beibehalten statt umzubenennen. explizit vom User abgelehnt ("lass uns weg"), Live-Sync bleibt read-only - Profilnamen aufs Board schreiben — technisch unmöglich (kein Platz im Firmware-Struct), bleibt lokal +- Tastendruck-Capture statt Dropdown im Programmiermodus — bewusst + vermieden (WinAPI-Hook-Risiko) - Hintergrund-Thread für "Vom Board laden"/"Zum Board übertragen" — laufen aktuell synchron im UI-Thread (kurzzeitiges Einfrieren möglich) - Vorgefertigte `.exe` im Repo/als Release-Asset — bewusst nicht committet @@ -480,17 +275,6 @@ Prüfungen vor einem Commit an Binärformat/Protokoll: ```bash python -m py_compile *.py ``` - -Für UI-Änderungen (Dialoge, Tastendruck-Erkennung, Kopieren/Einfügen) hat -sich zusätzlich bewährt, ein Wegwerf-Skript im Scratchpad zu fahren, das die -Dialoge ohne Board aufbaut, Tk-Events als kleine Fake-Event-Objekte -(`keysym`/`keycode`/`state`) durchreicht und das Ergebnis-Dict prüft — die -komplette Capture- und Copy/Paste-Logik ist so ohne Klicken verifizierbar. -Wichtig dabei: `versapad_combined.DEFAULT_PATH` vorher auf eine Temp-Datei -umbiegen, sonst schreibt `_autosave_combined()` in die echte Nutzer-Config. -Für Screenshots gilt: Tk rechnet in logischen Pixeln, `ImageGrab` liefert -physische — bei aktiver Windows-Skalierung (hier 125%) sonst ein zu kleiner -Ausschnitt, der wie ein Layout-Fehler aussieht. Danach ein Live-Testskript gegen ein angeschlossenes Board laufen lassen (read → unpack → pack → Bytevergleich, siehe Existing-Codebase-Regel) — es gibt keine automatisierten Unit-Tests dafür, die Verifikation läuft diff --git a/README.md b/README.md index 0c84f10..c6af12c 100644 --- a/README.md +++ b/README.md @@ -12,9 +12,9 @@ echtes Board getestet (Read-Modify-Write ist byte-identisch zum Original, inklusive CRC). Nicht vorhanden: automatisierte Tests (Verifikation läuft manuell gegen ein angeschlossenes Board), eine vorgefertigte `.exe` zum Download (siehe unten, warum), und Mehrbenutzer-/Netzwerkbetrieb. Die eigene -Config-Datei (`versapad_config_all.json`) liegt automatisch unter -`%APPDATA%\VersaPadViewer\` (siehe „Aufbau" unten) und wird bei Bedarf -automatisch neu angelegt — kein manuelles Pfad-Anpassen mehr nötig. Nur die *optionale* +Config-Datei (`versapad_config_all.json`) liegt automatisch neben der +Installation (siehe „Aufbau" unten) und wird bei Bedarf automatisch neu +angelegt — kein manuelles Pfad-Anpassen mehr nötig. Nur die *optionale* Lese-Interop mit den JSON-Exports der offiziellen VersaGUI (`versapad_data.CONFIG_PATHS`) ist noch für eine bestimmte Windows-Maschine hartkodiert (OneDrive-Desktop) — für einen anderen Rechner dort anpassen, @@ -30,47 +30,13 @@ falls gewünscht. und schaltet die Ansicht automatisch mit - **Programmiermodus** — Zellen anklicken und bearbeiten (Taste, Medientaste, Makro, Profilwechsel, LED-Farbe/Animation), direkt aufs Board schreiben - oder als Datei speichern. Der Dialog öffnet über dem Hauptfenster, - Enter bestätigt, Escape bricht ab -- **Tastendruck-Erkennung** — statt die Taste im Dropdown zu suchen, - „⌨ Taste drücken" klicken und die gewünschte Kombination einfach - drücken (Modifier inklusive). Erkannt wird die *physische* Taste, nicht - das Zeichen — Y/Z, ÄÖÜ, `#`, `+` und `ß` landen also richtig auf dem - Board, auch auf deutschem Layout. Läuft über das Dialogfenster, nicht - über einen System-Hook: vom System abgefangene Kombinationen (Win+L, - Strg+Alt+Entf) kommen nicht an -- **Layoutrichtige Tastennamen** — Beschriftungen kommen vom aktiven - Windows-Layout (`Strg+Z` heißt auf deutscher Tastatur auch `Strg+Z`, und - `ä`/`ö`/`ü` heißen so). Bestehende Belegungen ändern dadurch ihre - Anzeige, nicht ihre Funktion + oder als Datei speichern - **Makro-Editor** — bis zu 8 Schritte pro Slot, liest/schreibt die echte - Makro-Tabelle vom Board. Schritte einzeln erfassen oder die ganze Folge - am Stück aufnehmen („⏺ Folge aufnehmen"). Die Slot-Auswahl listet alle 32 - Slots samt Inhalt, statt sie einzeln durchklicken zu müssen -- **Makros im Grid lesbar** — eine Makro-Belegung zeigt die tatsächliche - Tastenfolge (`Makro 3: Strg+C → Strg+V`) statt nur der Slot-Nummer; das - gilt auch in der Browser-Ansicht und in den MCP-Antworten -- **Farb-Schnellwahl** — zwölf Grundfarben direkt in der LED-Zeile des - Dialogs, der System-Farbdialog nur noch für den Rest („mehr…") -- **Kopieren/Einfügen zwischen Tasten** — Rechtsklick auf eine Karte im - Programmiermodus: Belegung und/oder Farbe kopieren und auf andere Tasten - anwenden, oder die Belegung leeren. Strg+C/Strg+V wirken auf die Karte - unter dem Mauszeiger -- **Notizen** — freier Text pro Button/Encoder-Aktion, was sie tatsächlich - tut (z.B. "Speichern in Fusion 360"), zusätzlich zur automatischen - Beschriftung ("Strg+S"). Rein lokal wie Profilnamen, geht nie aufs Board, - bleibt beim Tastenwechsel und beim "Vom Board laden" erhalten + Makro-Tabelle vom Board - **MCP-Server** — lässt eine KI (Claude o.ä.) die Belegung direkt per Tool-Aufruf ändern, ohne Klicks in der GUI (siehe unten) - **Tray-Icon** — minimiert/schließt ins Tray statt in die Taskleiste, wie die offizielle VersaGUI -- **Normales Fenster mit Taskleisten-Eintrag** — Titelleiste, Alt+Tab, - Aero-Snap und Größe ändern am Rahmen funktionieren nativ. `✕` beendet - nicht, sondern legt ins Tray (wie die offizielle VersaGUI — das Programm - läuft im Hintergrund weiter); Minimieren geht normal in die Taskleiste. -- **Tastenkürzel im Hauptfenster** — `Strg+1/2/3` Profil wechseln, `F2` - Profil umbenennen, `Strg+E` Programmiermodus an/aus, `Strg+C`/`Strg+V` - Taste unter dem Mauszeiger kopieren/einfügen, `Esc` minimieren. ## Voraussetzungen @@ -109,7 +75,7 @@ Maschine/Python-Version unterschiedlich). Selbst bauen: Das Skript installiert/aktualisiert selbst alle nötigen Pakete aus `requirements.txt` (kein manuelles `pip install` vorher nötig), baut dann mit PyInstaller (`--onedir --windowed`, eigenes Icon) und kopiert das -Ergebnis nach `%LOCALAPPDATA%\VersaPadViewer\VersaPadViewer.exe`. +Ergebnis nach `dist\VersaPadViewer\VersaPadViewer.exe` im Projektordner. Bricht ein Schritt ab (fehlendes Python, PyInstaller-Fehler, ...), zeigt das Skript eine klare Fehlermeldung und wartet auf einen Tastendruck, statt sich bei Doppelklick im Explorer kommentarlos zu schließen (`-NoPause` @@ -127,8 +93,7 @@ automatisch zuerst nach `%TEMP%` und baut nur dort. | Datei | Zweck | |---|---| -| `versapad_data.py` | Decoding für die Anzeige: JSON laden, HID-Keycodes/Consumer-IDs/Modifier → lesbarer Text, Tastendruck → HID-Keycode | -| `versapad_keylayout.py` | Abfragen ans aktive Windows-Tastaturlayout: physische Tastenposition und Tastenname (optional, nur Windows) | +| `versapad_data.py` | Decoding für die Anzeige: JSON laden, HID-Keycodes/Consumer-IDs/Modifier → lesbarer Text | | `versapad_protocol.py` | Binäres NVM-Layout des Boards (740B Config + 512B Makros), CRC16 — pack/unpack | | `versapad_serial.py` | Serial-Client: liest/schreibt Config + Makros per 8-Byte-Paket-Protokoll | | `versapad_combined.py` | Ein-Datei-Format für alle 3 Profile + Makros + lokale Profilnamen | @@ -142,20 +107,14 @@ VersaMCU-Firmware-Quellen übernommen und gegen ein echtes Board validiert (Read → unpack → pack ist bytegenau identisch zum Original, inklusive CRC). Die eigene Config-Datei (`versapad_config_all.json` — alle 3 Profile + -Makros + lokale Profilnamen + Notizen, siehe `versapad_combined.py`) liegt -unter `%APPDATA%\VersaPadViewer\` (`versapad_data.app_dir()`), also -**getrennt vom Installationsordner** und unabhängig davon, ob die `.exe` -oder der Quellcode gestartet wurde. Beides ist Absicht: das Build-Skript -räumt sein Zielverzeichnis vor jedem Deploy komplett ab (läge die Config -dort, würde jeder Rebuild sie löschen), und ein vom Startweg abhängiger -Pfad hatte zu zwei auseinanderlaufenden Configs geführt. Fehlt die Datei -(z.B. frische Installation), wird sie automatisch angelegt — per Serial vom +Makros + lokale Profilnamen, siehe `versapad_combined.py`) liegt neben der +Installation: bei der gebauten `.exe` im selben Ordner, beim Start aus dem +Quellcode im Projektordner (`versapad_data.app_dir()`). Fehlt sie (z.B. +frische Installation), wird sie automatisch angelegt — per Serial vom Board, falls eins angeschlossen ist, sonst als leere Default-Config. Profilnamen (`profile_names`) stehen direkt in dieser Datei und lassen sich per Doppelklick auf einen Tab (in jedem Modus — Nur-Lesen, Live-Sync oder -Programmiermodus) oder `rename_profile()` (MCP) ändern. Notizen ebenso — -im Bearbeiten-Dialog des Programmiermodus oder per -`set_button_note()`/`set_encoder_note()` (MCP). +Programmiermodus) oder `rename_profile()` (MCP) ändern. Die klassischen `versapad_config1/2/3.json` sind kein von diesem Tool geschriebenes Format, sondern optionale Lese-Interop mit einem JSON-Export @@ -189,13 +148,10 @@ rechts im Fenster, oder direkt in `versapad_mcp_server.py`. gleichzeitig mit der offiziellen VersaGUI laufen (die läuft dauerhaft als Tray-App weiter, auch wenn nur ihr Konfigurationsfenster geschlossen wird — für Parallelbetrieb muss sie über ihr Tray-Menü beendet werden) -- Die Tastendruck-Erkennung läuft bewusst über das Dialogfenster statt über - einen globalen WinAPI-Hook — vom System abgefangene Kombinationen (Win+L, - Strg+Alt+Entf) erreichen das Fenster nie und lassen sich so nicht erfassen -- Ein Wechsel des Tastaturlayouts im laufenden Programm wird nicht bemerkt - (die Tastennamen werden einmal beim ersten Zugriff ermittelt) — Neustart - hilft. Unter Nicht-Windows fällt die Beschriftung auf eine - US-Layout-Näherung zurück +- HID-Tasten-Auswahl im Programmiermodus ist ein Dropdown, kein + Tastendruck-Capture (bewusst, um keinen WinAPI-Hook zu brauchen) +- Zeichentasten-Labels zeigen eine US-Layout-Näherung, nicht das tatsächlich + aktive Tastatur-Layout - Unsignierte `.exe` — kann von Antivirus/Smart App Control blockiert werden; `--onedir` (statt `--onefile`) verringert das Risiko, verhindert es aber nicht diff --git a/action_dialog.py b/action_dialog.py index 511c091..09a9f0d 100644 --- a/action_dialog.py +++ b/action_dialog.py @@ -1,24 +1,12 @@ """ Modale Bearbeiten-Dialoge fuer den Programmiermodus -- Pendant zu -VersaGUI/src/ActionDialog.cs, in Tkinter. +VersaGUI/src/ActionDialog.cs, aber in Tkinter und ohne Tastendruck-Capture +(kein WinAPI-Hook -- stattdessen Tasten-Auswahl per Dropdown, siehe +versapad_data.hid_key_choices()). ActionEditDialog: Action-Typ + Daten + optional LED (nur MX-Buttons). MacroStepsDialog: bis zu 8 Schritte (Taste + Strg/Shift/Alt), passend zur echten GUI ("8 Step-Buttons + je Strg/Shift/Alt-Checkboxen", kein Win). - -Tastendruck-Erkennung (_KeyCapture): Tasten lassen sich statt per Dropdown -auch einfach druecken. Das laeuft ueber ganz normale Tk-Fokus-Events des -Dialogfensters -- KEIN globaler WinAPI-Tastaturhook (siehe Domaenenregel -"keine Selbstversteck-/Hook-Fenstertricks"). Konsequenz: erkannt wird nur, -was das fokussierte Fenster erreicht -- Win+L, Strg+Alt+Entf und aehnliche -vom System abgefangene Kombinationen also nicht. - -Welche physische Taste gemeint ist, loest versapad_data.tk_event_to_hid() -ueber den Scan-Code des aktiven Layouts auf (versapad_keylayout), nicht -ueber das erzeugte Zeichen -- sonst landet auf deutschem Layout jedes Y auf -der Z-Taste des Boards und AeOeUe/#/+ sind gar nicht erfassbar. Das Dropdown -daneben zeigt dieselbe Taste unter ihrem Layout-Namen und bleibt als -Korrekturmoeglichkeit stehen. """ import tkinter as tk from tkinter import colorchooser, ttk @@ -30,7 +18,6 @@ BG2 = "#14161b" TEXT = "#e8e8ec" TEXT_DIM = "#8a8d98" ACCENT = "#3a6ff0" -CAPTURE_BG = "#c04a2a" TYPE_CHOICES = [ ("None", "Keine"), @@ -47,100 +34,8 @@ PROFILE_SWITCH_CHOICES = [ ("Profil 3", 2), ] -# Schnellzugriff direkt in der LED-Zeile -- der Umweg ueber den -# System-Farbdialog lohnt sich fuer die paar Standardfarben nicht. -# ("aus" ist Schwarz: LED bleibt dunkel, die Firmware kennt kein -# separates Enable-Flag.) -QUICK_COLORS = [ - ("aus", (0, 0, 0)), - ("weiß", (255, 255, 255)), - ("rot", (255, 0, 0)), - ("orange", (255, 80, 0)), - ("gelb", (255, 200, 0)), - ("grün", (0, 255, 0)), - ("türkis", (0, 255, 160)), - ("cyan", (0, 200, 255)), - ("blau", (0, 64, 255)), - ("violett", (128, 0, 255)), - ("magenta", (255, 0, 200)), - ("rosa", (255, 128, 128)), -] - -MOD_CHECKBOXES = (("Strg", 0x01), ("Shift", 0x02), ("Alt", 0x04), ("Win", 0x08)) -MACRO_MOD_CHECKBOXES = MOD_CHECKBOXES[:3] # Firmware kennt in Makros kein Win - - -class _KeyCapture: - """Macht ein Label zum "Taste drücken"-Schalter: solange er aktiv ist, - wandert jeder Tastendruck im Dialog nicht ins Widget, sondern durch - versapad_data.tk_event_to_hid() in ein (keycode, modifier)-Paar. - - on_result(keycode, modifier) wird im Tk-Main-Thread aufgerufen. - repeat=True laesst die Aufnahme nach einem Treffer weiterlaufen (fuer - "Folge aufnehmen" im Makro-Dialog), sonst endet sie nach dem ersten. - allow_win=False filtert das Win-Bit weg (Makro-Schritte). - """ - - def __init__(self, dialog, label, on_result, idle_text, allow_win=True, repeat=False): - self.dialog = dialog - self.label = label - self.on_result = on_result - self.idle_text = idle_text - self.allow_win = allow_win - self.repeat = repeat - self.active = False - self._held = 0 - label.configure(text=idle_text, cursor="hand2") - label.bind("", lambda e: self.toggle()) - - def toggle(self): - self.stop() if self.active else self.start() - - def start(self): - self.active = True - self._held = 0 - self.dialog.begin_capture(self) - self.label.configure(text="… jetzt drücken (Klick = Stopp)", bg=CAPTURE_BG, fg="#fff") - self.label.focus_set() - - def stop(self): - if not self.active: - return - self.active = False - self._held = 0 - self.dialog.end_capture(self) - self.label.configure(text=self.idle_text, bg=BG, fg=TEXT) - - def handle_key(self, event, pressed): - """Vom Dialog aufgerufen, solange diese Aufnahme aktiv ist. - Gibt immer "break" zurueck -- waehrend der Aufnahme darf kein - Tastendruck als Text im Notizfeld oder als Dialog-Shortcut landen.""" - mod = vp.TK_MODIFIER_KEYSYMS.get(event.keysym) - if mod is not None: - # Modifier selbst sind nie das Ziel, sie sammeln nur Bits -- - # ein KeyRelease sieht der Dialog nicht immer (Fokuswechsel), - # das state-Feld im Treffer-Event faengt das ab. - self._held = (self._held | mod) if pressed else (self._held & ~mod) - return "break" - if not pressed: - return "break" - hit = vp.tk_event_to_hid(event.keysym, event.keycode, event.state, self._held) - if hit is None: - return "break" # nicht zuordenbar -> einfach weiter warten - keycode, modifier = hit - if not self.allow_win: - modifier &= ~0x08 - if not self.repeat: - self.stop() - self.on_result(keycode, modifier) - return "break" - class _ModalDialog(tk.Toplevel): - """Basis: oeffnet mittig ueber dem aufrufenden Fenster (nicht in der - Bildschirmecke), Enter = OK, Escape = Abbrechen, und verteilt - Tastendruecke an eine ggf. laufende _KeyCapture.""" - def __init__(self, parent, title): super().__init__(parent) self.title(title) @@ -148,116 +43,18 @@ class _ModalDialog(tk.Toplevel): self.transient(parent) self.resizable(False, False) self.cancelled = True - self._active_capture = None - # Erst unsichtbar aufbauen, in run() positionieren und dann zeigen -- - # sonst blitzt der Dialog kurz in der linken oberen Bildschirmecke auf. - self.withdraw() - self.bind("", self._on_key_press) - self.bind("", self._on_key_release) - self.protocol("WM_DELETE_WINDOW", lambda: self._finish(True)) - - # ── Tastendruck-Aufnahme ────────────────────────────────────────── - - def begin_capture(self, capture): - if self._active_capture is not None and self._active_capture is not capture: - self._active_capture.stop() - self._active_capture = capture - - def end_capture(self, capture): - if self._active_capture is capture: - self._active_capture = None - - def _on_key_press(self, event): - if self._active_capture is not None: - return self._active_capture.handle_key(event, pressed=True) - if event.keysym in ("Return", "KP_Enter"): - self._on_ok() - return "break" - if event.keysym == "Escape": - self._finish(True) - return "break" - return None - - def _on_key_release(self, event): - if self._active_capture is not None: - return self._active_capture.handle_key(event, pressed=False) - return None - - def _on_ok(self): - """Von Enter und vom OK-Knopf -- Unterklassen ueberschreiben das.""" - self._finish(False) def _finish(self, cancelled): - if self._active_capture is not None: - self._active_capture.stop() self.cancelled = cancelled self.grab_release() self.destroy() - # ── Positionierung + Ablauf ─────────────────────────────────────── - - def _center_on_parent(self): - """Mittig ueber dem Elternfenster. Ohne das oeffnet Tk jeden Toplevel - bei +0+0, also bei jeder bearbeiteten Taste erneut in der - Bildschirmecke -- weit weg vom Fenster, in dem gerade geklickt wurde. - - Begrenzt wird bewusst gegen das ELTERNFENSTER, nicht gegen - `winfo_screenwidth()`: Tk meldet dort nur den Hauptbildschirm. Lag - das Hauptfenster auf einem zweiten Monitor, hat genau diese - Begrenzung den Dialog wieder auf den Rand des ersten Monitors - gezogen. Passt der Dialog nicht ins Elternfenster (der - Makro-Schritte-Dialog ist breiter als der Action-Dialog), wird er an - dessen linker oberer Ecke ausgerichtet statt zentriert -- so bleibt - er in jedem Fall auf dem Monitor, auf dem gearbeitet wird.""" - parent = self.master - width, height = self.winfo_reqwidth(), self.winfo_reqheight() - try: - px, py = parent.winfo_rootx(), parent.winfo_rooty() - pw, ph = parent.winfo_width(), parent.winfo_height() - except tk.TclError: - px = py = 0 - pw, ph = self.winfo_screenwidth(), self.winfo_screenheight() - x = px + max(0, (pw - width) // 2) - y = py + max(0, (ph - height) // 3) # etwas oberhalb der Mitte wirkt ruhiger - self.geometry(f"+{x}+{y}") - def run(self): self.update_idletasks() - self._center_on_parent() - self.deiconify() self.grab_set() - self.focus_force() self.wait_window(self) - # Ein verschachtelter Dialog (Makro-Schritte aus dem Action-Dialog) - # nimmt beim Schliessen den Grab mit -- ohne Rueckgabe waere der - # aufrufende Dialog danach nicht mehr modal. - parent = self.master - if isinstance(parent, _ModalDialog) and parent.winfo_exists(): - parent.grab_set() - parent.focus_force() return not self.cancelled - # ── gemeinsame kleine Bausteine ─────────────────────────────────── - - def _capture_label(self, parent, on_result, idle_text="⌨ Taste drücken", - allow_win=True, repeat=False): - """Bewusst ein Label statt tk.Button: Buttons reagieren per - Klassen-Binding selbst auf Leertaste/Enter und wuerden die laufende - Aufnahme mit ihrem eigenen Klick beantworten.""" - label = tk.Label(parent, bg=BG, fg=TEXT, font=("Segoe UI", 9), - padx=10, pady=3, relief="flat") - _KeyCapture(self, label, on_result, idle_text, allow_win=allow_win, repeat=repeat) - return label - - def _button_row(self, row, columnspan, ok_text="OK"): - btns = tk.Frame(self, bg=BG2) - btns.grid(row=row, column=0, columnspan=columnspan, pady=14) - tk.Button(btns, text=f"{ok_text} (Enter)", command=self._on_ok, bg=ACCENT, fg="#fff", - activebackground=ACCENT, relief="flat", padx=16).pack(side="left", padx=4) - tk.Button(btns, text="Abbrechen (Esc)", command=lambda: self._finish(True), bg=BG, - fg=TEXT, activebackground=BG, relief="flat", padx=16).pack(side="left", padx=4) - return btns - class MacroStepsDialog(_ModalDialog): """Ergebnis in self.steps nach run()==True.""" @@ -271,81 +68,43 @@ class MacroStepsDialog(_ModalDialog): self._code_by_name = {name: code for code, name in key_choices} names = ["(leer)"] + [name for _, name in key_choices] - head = tk.Frame(self, bg=BG2) - head.grid(row=0, column=0, columnspan=6, sticky="w", padx=12, pady=(12, 6)) - tk.Label(head, text="Bis zu 8 Schritte, Ausführung stoppt beim ersten leeren.", - bg=BG2, fg=TEXT_DIM, font=("Segoe UI", 8)).pack(anchor="w") - tk.Label(head, text="Einzeln erfassen mit ⌨ je Zeile, oder die ganze Folge am Stück " - "aufnehmen. Win ist in Makros nicht möglich (Firmware).", - bg=BG2, fg=TEXT_DIM, font=("Segoe UI", 8)).pack(anchor="w") - - tools = tk.Frame(self, bg=BG2) - tools.grid(row=1, column=0, columnspan=6, sticky="w", padx=12, pady=(0, 6)) - self._record_label = tk.Label(tools, bg=BG, fg=TEXT, font=("Segoe UI", 9), - padx=10, pady=3) - self._record = _KeyCapture(self, self._record_label, self._on_record_step, - "⏺ Folge aufnehmen", allow_win=False, repeat=True) - self._record_label.pack(side="left") - tk.Button(tools, text="Alle leeren", command=self._clear_all, bg=BG, fg=TEXT, - activebackground=BG, relief="flat", padx=10).pack(side="left", padx=6) - self._record_hint = tk.Label(tools, text="", bg=BG2, fg=TEXT_DIM, font=("Segoe UI", 8)) - self._record_hint.pack(side="left", padx=6) + tk.Label(self, text="Bis zu 8 Schritte, Ausführung stoppt beim ersten leeren.", + bg=BG2, fg=TEXT_DIM, font=("Segoe UI", 8)).grid( + row=0, column=0, columnspan=5, sticky="w", padx=12, pady=(12, 6)) self._key_vars = [] self._mod_vars = [] for i in range(8): - r = i + 2 + r = i + 1 tk.Label(self, text=f"{i + 1}.", bg=BG2, fg=TEXT_DIM, - font=("Segoe UI", 9)).grid(row=r, column=0, padx=(12, 4), pady=2, sticky="e") + font=("Segoe UI", 9)).grid(row=r, column=0, padx=(12, 4), pady=2, sticky="e") key_var = tk.StringVar(value="(leer)") - ttk.Combobox(self, textvariable=key_var, values=names, width=16, - state="readonly").grid(row=r, column=1, padx=4, pady=2) + combo = ttk.Combobox(self, textvariable=key_var, values=names, width=16, state="readonly") + combo.grid(row=r, column=1, padx=4, pady=2) self._key_vars.append(key_var) mods = {} - for j, (label, _bit) in enumerate(MACRO_MOD_CHECKBOXES): + for j, label in enumerate(("Strg", "Shift", "Alt")): v = tk.BooleanVar(value=False) tk.Checkbutton(self, text=label, variable=v, bg=BG2, fg=TEXT, - selectcolor=BG, activebackground=BG2, activeforeground=TEXT, - font=("Segoe UI", 8)).grid(row=r, column=2 + j, padx=2, pady=2, - sticky="w") + selectcolor=BG, activebackground=BG2, activeforeground=TEXT, + font=("Segoe UI", 8)).grid(row=r, column=2 + j, padx=2, pady=2, sticky="w") mods[label] = v self._mod_vars.append(mods) - self._capture_label( - self, lambda code, mod, idx=i: self._apply_step(idx, code, mod), - idle_text="⌨", allow_win=False, - ).grid(row=r, column=5, padx=(6, 12), pady=2) - for i, step in enumerate(steps[:8]): - self._apply_step(i, step["keycode"], step["modifier"]) + self._key_vars[i].set(self._name_by_code.get(step["keycode"], "(leer)")) + self._mod_vars[i]["Strg"].set(bool(step["modifier"] & 0x01)) + self._mod_vars[i]["Shift"].set(bool(step["modifier"] & 0x02)) + self._mod_vars[i]["Alt"].set(bool(step["modifier"] & 0x04)) - self._button_row(row=10, columnspan=6) - - def _apply_step(self, index, keycode, modifier): - self._key_vars[index].set(self._name_by_code.get(keycode, "(leer)")) - for label, bit in MACRO_MOD_CHECKBOXES: - self._mod_vars[index][label].set(bool(modifier & bit)) - - def _clear_all(self): - for i in range(8): - self._apply_step(i, 0, 0) - self._record_hint.configure(text="") - - def _on_record_step(self, keycode, modifier): - """Aufnahmemodus: jeder Tastendruck fuellt den naechsten freien - Schritt. Nach dem 8. stoppt die Aufnahme selbst (mehr Schritte - kennt SMacroTable nicht).""" - free = next((i for i in range(8) if self._key_vars[i].get() == "(leer)"), None) - if free is None: - self._record.stop() - self._record_hint.configure(text="alle 8 Schritte belegt") - return - self._apply_step(free, keycode, modifier) - self._record_hint.configure(text=f"Schritt {free + 1} aufgenommen") - if free == 7: - self._record.stop() + btns = tk.Frame(self, bg=BG2) + btns.grid(row=9, column=0, columnspan=5, pady=12) + tk.Button(btns, text="OK", command=self._on_ok, bg=ACCENT, fg="#fff", + activebackground=ACCENT, relief="flat", padx=16).pack(side="left", padx=4) + tk.Button(btns, text="Abbrechen", command=lambda: self._finish(True), bg=BG, + fg=TEXT, activebackground=BG, relief="flat", padx=16).pack(side="left", padx=4) def _on_ok(self): steps = [] @@ -353,8 +112,11 @@ class MacroStepsDialog(_ModalDialog): name = self._key_vars[i].get() if name == "(leer)": break # Firmware: keycode=0 beendet die Sequenz -> Rest ignorieren - modifier = sum(bit for label, bit in MACRO_MOD_CHECKBOXES - if self._mod_vars[i][label].get()) + modifier = ( + (0x01 if self._mod_vars[i]["Strg"].get() else 0) | + (0x02 if self._mod_vars[i]["Shift"].get() else 0) | + (0x04 if self._mod_vars[i]["Alt"].get() else 0) + ) steps.append({"keycode": self._code_by_name[name], "modifier": modifier}) self.steps = steps self._finish(False) @@ -362,9 +124,7 @@ class MacroStepsDialog(_ModalDialog): class ActionEditDialog(_ModalDialog): """Ergebnis in self.action / self.led nach run()==True. led bleibt None - wenn led_in None war (Encoder -- keine eigene Farbe). self.action enthaelt - immer ein "note"-Feld (freie Notiz, was die Aktion tut -- rein lokal wie - Profilnamen, geht nie aufs Board).""" + wenn led_in None war (Encoder -- keine eigene Farbe).""" def __init__(self, parent, title, action, led, macros): super().__init__(parent, title) @@ -374,30 +134,21 @@ class ActionEditDialog(_ModalDialog): self.led = None row = 0 - note_row = tk.Frame(self, bg=BG2) - note_row.grid(row=row, column=0, columnspan=3, sticky="w", padx=12, pady=(12, 4)); row += 1 - tk.Label(note_row, text="Notiz:", bg=BG2, fg=TEXT_DIM, font=("Segoe UI", 9)).pack(side="left") - self._note_var = tk.StringVar(value=action.get("note", "")) - tk.Entry(note_row, textvariable=self._note_var, width=52, bg=BG, fg=TEXT, - insertbackground=TEXT, relief="flat").pack(side="left", padx=6) - tk.Label(self, text="Aktion", bg=BG2, fg=TEXT_DIM, font=("Segoe UI", 9, "bold")).grid( - row=row, column=0, sticky="w", padx=12, pady=(4, 4)); row += 1 + row=row, column=0, sticky="w", padx=12, pady=(12, 4)); row += 1 self._type_var = tk.StringVar(value=action["type"]) type_frame = tk.Frame(self, bg=BG2) type_frame.grid(row=row, column=0, columnspan=3, sticky="w", padx=12); row += 1 for value, label in TYPE_CHOICES: tk.Radiobutton(type_frame, text=label, variable=self._type_var, value=value, - command=self._on_type_change, bg=BG2, fg=TEXT, selectcolor=BG, - activebackground=BG2, activeforeground=TEXT, - font=("Segoe UI", 9)).pack(side="left", padx=(0, 8)) + command=self._on_type_change, bg=BG2, fg=TEXT, selectcolor=BG, + activebackground=BG2, activeforeground=TEXT, + font=("Segoe UI", 9)).pack(side="left", padx=(0, 8)) - # Feste Groesse: sonst springt die Fensterhoehe (und damit die - # Position von OK/Abbrechen) bei jedem Typwechsel. - self._panel = tk.Frame(self, bg=BG2, width=600, height=90) - self._panel.grid(row=row, column=0, columnspan=3, sticky="ew", padx=12, pady=8) - self._panel.grid_propagate(False) + self._panel_row = row + self._panel = tk.Frame(self, bg=BG2) + self._panel.grid(row=row, column=0, columnspan=3, sticky="w", padx=12, pady=8) row += 1 # HidKey-Panel @@ -414,16 +165,17 @@ class ActionEditDialog(_ModalDialog): self._consumer_var = tk.StringVar() # Macro-Panel - self._macro_slot = action["data"] if action["type"] == "Macro" else 0 - self._macro_choice_var = tk.StringVar() + self._macro_slot_var = tk.IntVar(value=action["data"] if action["type"] == "Macro" else 0) + self._macro_preview_var = tk.StringVar() # ProfileSwitch-Panel self._profile_switch_var = tk.StringVar(value=PROFILE_SWITCH_CHOICES[0][0]) if action["type"] == "HidKey": keycode = action["data"] & 0xFF + modifier = (action["data"] >> 8) & 0xFF self._hidkey_key_var.set(self._key_name_by_code.get(keycode, "(leer)")) - self._hidkey_mods_init = (action["data"] >> 8) & 0xFF + self._hidkey_mods_init = modifier else: self._hidkey_key_var.set("(leer)") self._hidkey_mods_init = 0 @@ -445,67 +197,45 @@ class ActionEditDialog(_ModalDialog): self._led_color = (self._led["r"], self._led["g"], self._led["b"]) if self._led else (80, 40, 0) if self._led is not None: - row = self._build_led_panel(row) + led_frame = tk.Frame(self, bg=BG2) + led_frame.grid(row=row, column=0, columnspan=3, sticky="w", padx=12, pady=(4, 8)); row += 1 + tk.Label(led_frame, text="LED", bg=BG2, fg=TEXT_DIM, font=("Segoe UI", 9, "bold")).pack(anchor="w") + + color_row = tk.Frame(led_frame, bg=BG2) + color_row.pack(anchor="w", pady=4) + self._swatch = tk.Label(color_row, text=" ", bg=self._hex(), relief="flat", width=4) + self._swatch.pack(side="left") + tk.Button(color_row, text="Farbe wählen...", command=self._pick_color, bg=BG, + fg=TEXT, activebackground=BG, relief="flat").pack(side="left", padx=8) + + anim_row = tk.Frame(led_frame, bg=BG2) + anim_row.pack(anchor="w", pady=4) + tk.Label(anim_row, text="Animation:", bg=BG2, fg=TEXT_DIM, font=("Segoe UI", 9)).pack(side="left") + ttk.Combobox(anim_row, textvariable=self._anim_var, values=list(vp.ANIM_LABELS.keys()), + width=12, state="readonly").pack(side="left", padx=6) + tk.Label(anim_row, text="Periode (ms):", bg=BG2, fg=TEXT_DIM, font=("Segoe UI", 9)).pack(side="left", padx=(12, 0)) + tk.Spinbox(anim_row, from_=2, to=10000, increment=100, textvariable=self._period_var, + width=7).pack(side="left", padx=6) + + btns = tk.Frame(self, bg=BG2) + btns.grid(row=row, column=0, columnspan=3, pady=14) + tk.Button(btns, text="OK", command=self._on_ok, bg=ACCENT, fg="#fff", + activebackground=ACCENT, relief="flat", padx=16).pack(side="left", padx=4) + tk.Button(btns, text="Abbrechen", command=lambda: self._finish(True), bg=BG, + fg=TEXT, activebackground=BG, relief="flat", padx=16).pack(side="left", padx=4) - self._button_row(row=row, columnspan=3) self._on_type_change() - # ── LED ─────────────────────────────────────────────────────────── - - def _build_led_panel(self, row): - led_frame = tk.Frame(self, bg=BG2) - led_frame.grid(row=row, column=0, columnspan=3, sticky="w", padx=12, pady=(4, 8)) - tk.Label(led_frame, text="LED", bg=BG2, fg=TEXT_DIM, - font=("Segoe UI", 9, "bold")).pack(anchor="w") - - color_row = tk.Frame(led_frame, bg=BG2) - color_row.pack(anchor="w", pady=4) - self._swatch = tk.Label(color_row, text=" ", bg=self._hex(), relief="flat", width=4) - self._swatch.pack(side="left") - self._hex_label = tk.Label(color_row, text=self._hex(), bg=BG2, fg=TEXT_DIM, - font=("Consolas", 9), width=9) - self._hex_label.pack(side="left", padx=(6, 6)) - for name, rgb in QUICK_COLORS: - chip = tk.Label(color_row, bg="#%02x%02x%02x" % rgb, width=2, height=1, - cursor="hand2", relief="flat", - highlightbackground="#3a3d47", highlightthickness=1) - chip.pack(side="left", padx=1) - chip.bind("", lambda e, c=rgb: self._set_color(c)) - # Tk kennt keine Tooltips -- der Farbname landet in der Zeile darunter. - chip.bind("", lambda e, n=name: self._color_hint.configure(text=n)) - chip.bind("", lambda e: self._color_hint.configure(text="")) - tk.Button(color_row, text="mehr…", command=self._pick_color, bg=BG, fg=TEXT, - activebackground=BG, relief="flat", padx=8).pack(side="left", padx=(8, 0)) - self._color_hint = tk.Label(led_frame, text="", bg=BG2, fg=TEXT_DIM, font=("Segoe UI", 8)) - self._color_hint.pack(anchor="w") - - anim_row = tk.Frame(led_frame, bg=BG2) - anim_row.pack(anchor="w", pady=4) - tk.Label(anim_row, text="Animation:", bg=BG2, fg=TEXT_DIM, - font=("Segoe UI", 9)).pack(side="left") - ttk.Combobox(anim_row, textvariable=self._anim_var, values=list(vp.ANIM_LABELS.keys()), - width=12, state="readonly").pack(side="left", padx=6) - tk.Label(anim_row, text="Periode (ms):", bg=BG2, fg=TEXT_DIM, - font=("Segoe UI", 9)).pack(side="left", padx=(12, 0)) - tk.Spinbox(anim_row, from_=2, to=10000, increment=100, textvariable=self._period_var, - width=7).pack(side="left", padx=6) - return row + 1 - def _hex(self): r, g, b = self._led_color return f"#{r:02x}{g:02x}{b:02x}" - def _set_color(self, rgb): - self._led_color = tuple(rgb) - self._swatch.configure(bg=self._hex()) - self._hex_label.configure(text=self._hex()) - def _pick_color(self): - result = colorchooser.askcolor(color=self._hex(), title="LED-Farbe", parent=self) + result = colorchooser.askcolor(color=self._hex(), title="LED-Farbe") if result and result[0]: - self._set_color(tuple(int(c) for c in result[0])) - - # ── Action-Panels ───────────────────────────────────────────────── + r, g, b = (int(c) for c in result[0]) + self._led_color = (r, g, b) + self._swatch.configure(bg=self._hex()) def _on_type_change(self): for w in self._panel.winfo_children(): @@ -513,109 +243,83 @@ class ActionEditDialog(_ModalDialog): t = self._type_var.get() if t == "HidKey": - self._build_hidkey_panel() + row1 = tk.Frame(self._panel, bg=BG2); row1.pack(anchor="w", pady=2) + self._hidkey_mods = {} + for label, bit in (("Strg", 0x01), ("Shift", 0x02), ("Alt", 0x04), ("Win", 0x08)): + v = tk.BooleanVar(value=bool(self._hidkey_mods_init & bit)) + tk.Checkbutton(row1, text=label, variable=v, bg=BG2, fg=TEXT, selectcolor=BG, + activebackground=BG2, activeforeground=TEXT, + font=("Segoe UI", 9)).pack(side="left", padx=(0, 6)) + self._hidkey_mods[bit] = v + row2 = tk.Frame(self._panel, bg=BG2); row2.pack(anchor="w", pady=4) + tk.Label(row2, text="Taste:", bg=BG2, fg=TEXT_DIM, font=("Segoe UI", 9)).pack(side="left") + names = ["(leer)"] + [n for _, n in vp.hid_key_choices()] + ttk.Combobox(row2, textvariable=self._hidkey_key_var, values=names, + width=18, state="readonly").pack(side="left", padx=6) + elif t == "HidConsumer": row1 = tk.Frame(self._panel, bg=BG2); row1.pack(anchor="w", pady=2) - tk.Label(row1, text="Medienaktion:", bg=BG2, fg=TEXT_DIM, - font=("Segoe UI", 9)).pack(side="left") - ttk.Combobox(row1, textvariable=self._consumer_var, - values=[n for _, n in vp.consumer_choices()], - width=20, state="readonly").pack(side="left", padx=6) + tk.Label(row1, text="Medienaktion:", bg=BG2, fg=TEXT_DIM, font=("Segoe UI", 9)).pack(side="left") + names = [n for _, n in vp.consumer_choices()] + ttk.Combobox(row1, textvariable=self._consumer_var, values=names, + width=20, state="readonly").pack(side="left", padx=6) + elif t == "Macro": - self._build_macro_panel() + row1 = tk.Frame(self._panel, bg=BG2); row1.pack(anchor="w", pady=2) + tk.Label(row1, text="Slot (0-31):", bg=BG2, fg=TEXT_DIM, font=("Segoe UI", 9)).pack(side="left") + tk.Spinbox(row1, from_=0, to=31, textvariable=self._macro_slot_var, width=5, + command=self._refresh_macro_preview).pack(side="left", padx=6) + tk.Button(row1, text="Schritte bearbeiten...", command=self._edit_macro_steps, + bg=BG, fg=TEXT, activebackground=BG, relief="flat").pack(side="left", padx=8) + row2 = tk.Frame(self._panel, bg=BG2); row2.pack(anchor="w", pady=(4, 0)) + tk.Label(row2, textvariable=self._macro_preview_var, bg=BG2, fg=TEXT_DIM, + font=("Segoe UI", 8), wraplength=340, justify="left").pack(anchor="w") + self._refresh_macro_preview() + elif t == "ProfileSwitch": row1 = tk.Frame(self._panel, bg=BG2); row1.pack(anchor="w", pady=2) tk.Label(row1, text="Ziel:", bg=BG2, fg=TEXT_DIM, font=("Segoe UI", 9)).pack(side="left") ttk.Combobox(row1, textvariable=self._profile_switch_var, - values=[n for n, _ in PROFILE_SWITCH_CHOICES], - width=22, state="readonly").pack(side="left", padx=6) + values=[n for n, _ in PROFILE_SWITCH_CHOICES], + width=22, state="readonly").pack(side="left", padx=6) self.update_idletasks() - def _build_hidkey_panel(self): - row1 = tk.Frame(self._panel, bg=BG2); row1.pack(anchor="w", pady=2) - self._hidkey_mods = {} - for label, bit in MOD_CHECKBOXES: - v = tk.BooleanVar(value=bool(self._hidkey_mods_init & bit)) - tk.Checkbutton(row1, text=label, variable=v, bg=BG2, fg=TEXT, selectcolor=BG, - activebackground=BG2, activeforeground=TEXT, - font=("Segoe UI", 9)).pack(side="left", padx=(0, 6)) - self._hidkey_mods[bit] = v - - row2 = tk.Frame(self._panel, bg=BG2); row2.pack(anchor="w", pady=4) - tk.Label(row2, text="Taste:", bg=BG2, fg=TEXT_DIM, font=("Segoe UI", 9)).pack(side="left") - names = ["(leer)"] + [n for _, n in vp.hid_key_choices()] - ttk.Combobox(row2, textvariable=self._hidkey_key_var, values=names, - width=18, state="readonly").pack(side="left", padx=6) - self._capture_label(row2, self._apply_captured_key).pack(side="left", padx=(8, 0)) - - tk.Label(self._panel, text="Erkennung läuft über das Dialogfenster, nicht über einen " - "System-Hook: Win+L o.ä. fängt Windows selbst ab.", - bg=BG2, fg=TEXT_DIM, font=("Segoe UI", 8)).pack(anchor="w") - - def _apply_captured_key(self, keycode, modifier): - self._hidkey_key_var.set(self._key_name_by_code.get(keycode, "(leer)")) - self._hidkey_mods_init = modifier - for bit, var in self._hidkey_mods.items(): - var.set(bool(modifier & bit)) - - def _build_macro_panel(self): - row1 = tk.Frame(self._panel, bg=BG2); row1.pack(anchor="w", pady=2) - tk.Label(row1, text="Slot:", bg=BG2, fg=TEXT_DIM, font=("Segoe UI", 9)).pack(side="left") - # Dropdown listet alle 32 Slots MIT Inhalt -- vorher musste man sich - # per Spinbox durch die Tabelle klicken, um ein Makro wiederzufinden. - self._macro_combo = ttk.Combobox(row1, textvariable=self._macro_choice_var, - values=vp.macro_slot_choices(self._macros), - width=52, state="readonly") - self._macro_combo.pack(side="left", padx=6) - self._macro_combo.bind("<>", self._on_macro_slot_selected) - tk.Button(row1, text="Schritte bearbeiten…", command=self._edit_macro_steps, - bg=BG, fg=TEXT, activebackground=BG, relief="flat", - padx=8).pack(side="left", padx=8) - - tk.Label(self._panel, text="Slots sind global (nicht pro Profil) – dasselbe Makro auf " - "zwei Tasten meint denselben Slot.", - bg=BG2, fg=TEXT_DIM, font=("Segoe UI", 8)).pack(anchor="w", pady=(6, 0)) - self._refresh_macro_choices() - - def _on_macro_slot_selected(self, _event=None): - self._macro_slot = vp.macro_slot_from_choice(self._macro_choice_var.get()) - - def _refresh_macro_choices(self): - choices = vp.macro_slot_choices(self._macros) - self._macro_combo.configure(values=choices) - self._macro_choice_var.set(choices[self._macro_slot]) + def _refresh_macro_preview(self): + slot = self._macro_slot_var.get() + steps = self._macros[slot] if 0 <= slot < len(self._macros) else [] + self._macro_preview_var.set(f"Slot {slot}: {vp.macro_slot_label(steps)}") def _edit_macro_steps(self): - slot = self._macro_slot + slot = self._macro_slot_var.get() current = self._macros[slot] if 0 <= slot < len(self._macros) else [] dlg = MacroStepsDialog(self, current) if dlg.run(): while len(self._macros) <= slot: self._macros.append([]) self._macros[slot] = dlg.steps - self._refresh_macro_choices() - - # ── Ergebnis ────────────────────────────────────────────────────── + self._refresh_macro_preview() def _on_ok(self): t = self._type_var.get() - if t == "HidKey": - keycode = self._key_code_by_name.get(self._hidkey_key_var.get(), 0) + if t == "None": + action = {"type": "None", "data": 0} + elif t == "HidKey": + name = self._hidkey_key_var.get() + keycode = self._key_code_by_name.get(name, 0) modifier = sum(bit for bit, v in self._hidkey_mods.items() if v.get()) action = {"type": "HidKey", "data": (modifier << 8) | keycode} elif t == "HidConsumer": action = {"type": "HidConsumer", "data": self._cons_id_by_name[self._consumer_var.get()]} elif t == "Macro": - action = {"type": "Macro", "data": self._macro_slot} + action = {"type": "Macro", "data": self._macro_slot_var.get()} elif t == "ProfileSwitch": name = self._profile_switch_var.get() - action = {"type": "ProfileSwitch", - "data": next(v for n, v in PROFILE_SWITCH_CHOICES if n == name)} + data = next(v for n, v in PROFILE_SWITCH_CHOICES if n == name) + action = {"type": "ProfileSwitch", "data": data} else: action = {"type": "None", "data": 0} - action["note"] = self._note_var.get() self.action = action if self._led is not None: r, g, b = self._led_color diff --git a/build_and_deploy.ps1 b/build_and_deploy.ps1 index 0accd2c..e0617e0 100644 --- a/build_and_deploy.ps1 +++ b/build_and_deploy.ps1 @@ -1,8 +1,5 @@ # Baut VersaPadViewer.exe (PyInstaller --onedir) und kopiert das Ergebnis -# nach %LOCALAPPDATA%\VersaPadViewer (nicht $projectDir\dist wie im -# jappel-Upstream -- hier bereits die tatsaechlich installierte/genutzte -# Instanz, siehe Speicher-Notiz "VersaPad Viewer Tool"; ein Pfadwechsel -# wuerde bestehende Verknuepfungen/Autostart-Eintraege brechen). +# nach $projectDir\dist\VersaPadViewer. # # WICHTIG: Sowohl Bauen als auch Laufen muessen lokal passieren, NICHT auf # dem Netzlaufwerk (Z:\Git\...): @@ -30,7 +27,7 @@ param( $ErrorActionPreference = "Stop" $projectDir = $PSScriptRoot $buildSrc = "$env:TEMP\versapad_build_src" -$localDir = "$env:LOCALAPPDATA\VersaPadViewer" +$localDir = "$projectDir\dist\VersaPadViewer" $requirementsFile = "$projectDir\requirements.txt" function Assert-LastExitCode([string]$step) { @@ -83,29 +80,10 @@ try { throw "PyInstaller hat keine VersaPadViewer.exe erzeugt, obwohl der Exit-Code 0 war -- Build-Ausgabe oben pruefen." } - # Zielverzeichnis abraeumen, aber NIE Nutzerdaten mitloeschen: die Config - # liegt zwar inzwischen woanders (Roaming-AppData, siehe - # versapad_data.app_dir()), aeltere Installationen haben sie aber noch - # hier liegen -- ohne diese Sicherung loescht jeder Rebuild sie mit - # (am 2026-08-15 genau so passiert, alle Notizen weg). - $userData = Get-ChildItem $localDir -Filter "versapad_config*.json" -File -ErrorAction SilentlyContinue - $rescued = @() - foreach ($f in $userData) { - $tmp = Join-Path $env:TEMP $f.Name - Copy-Item $f.FullName $tmp -Force - $rescued += @{ Tmp = $tmp; Name = $f.Name } - Write-Host "Nutzerdatei gesichert: $($f.Name)" -ForegroundColor Yellow - } - if (Test-Path $localDir) { Remove-Item $localDir -Recurse -Force } Copy-Item "$buildSrc\dist\VersaPadViewer" -Destination $localDir -Recurse - - foreach ($r in $rescued) { - Copy-Item $r.Tmp (Join-Path $localDir $r.Name) -Force - Remove-Item $r.Tmp -Force - } Remove-Item $buildSrc -Recurse -Force Write-Host "" diff --git a/desktop_viewer.py b/desktop_viewer.py index 6909ed5..81b1805 100644 --- a/desktop_viewer.py +++ b/desktop_viewer.py @@ -19,7 +19,6 @@ automatisch ausgeschaltet, damit sich beide nicht um den Port streiten). Start: python desktop_viewer.py """ -import copy import os import queue import sys @@ -55,11 +54,7 @@ OK_GREEN = "#3ecf6e" WARN_RED = "#e0895a" POLL_MS = 1500 -CARD_W, CARD_H = 150, 114 -# Untergrenze fuers Verkleinern (self.minsize) -- knapp unter der Groesse, -# die das 4x5-Grid + Encoder mindestens brauchen, damit nichts -# abgeschnitten wird. -MIN_W, MIN_H = 700, 500 +CARD_W, CARD_H = 150, 84 SERIAL_POLL_S = 1.5 SERIAL_IDLE_S = 3.0 @@ -102,11 +97,9 @@ Buttons setzen (profile 0-2, index 0-19): set_button_profile_switch(profile, index, target) set_button_none(profile, index) set_button_led(profile, index, r, g, b, anim, period_ms) - set_button_note(profile, index, note) freie Notiz, was der Button tut Encoder setzen (index 0-3, field 'sw'/'cw'/'ccw'): set_encoder_key / _consumer / _macro / _profile_switch / _none(...) - set_encoder_note(profile, index, field, note) Makro: set_macro(slot, steps) steps=[{"key":"Z","modifiers":["Strg"]}, ...] @@ -129,23 +122,13 @@ class VersaPadViewer(tk.Tk): super().__init__() self.title("VersaPad Steuermatrix") self.configure(bg=BG) - # Hoehe muss Kopfzeile + Tabs + 5 Kartenreihen + Encoder + Fusszeile - # fassen -- seit die Karten eine Notizzeile haben (CARD_H 84 -> 114) - # reichten die alten 760px nicht mehr: der Encoder-Bereich lag - # unterhalb des Fensterrands. Wer CARD_H aendert, muss hier mit. - self.geometry("790x960") + self.geometry("760x760") self.profile = 0 self._mtimes = {} self.combined = None # kombinierter Programmiermodus-State, erst bei Bedarf befuellt self._link = vs.VersaPadLink() self._closing = False - # Zwischenablage fuer "Belegung/Farbe auf andere Taste uebertragen" - # (Rechtsklickmenue bzw. Strg+C/V auf der Karte unter dem Mauszeiger). - # Bewusst NUR im Speicher, nicht die System-Zwischenablage: hier - # liegen Action-Dicts, kein Text. - self._clip = None - self._hover = None self.live_sync = tk.BooleanVar(value=False) self.editing = tk.BooleanVar(value=False) self._serial_results = queue.Queue() @@ -166,79 +149,23 @@ class VersaPadViewer(tk.Tk): ) self._tray_icon.run_detached() - # Normales Fenster MIT Windows-Titelleiste. Bis 2026-08-28 lief es - # randlos (`overrideredirect(True)`) -- das kostet unter Windows aber - # zwingend den Taskleisten-Eintrag, und der wurde ausdruecklich - # gebraucht. Nachruesten liesse er sich nur per `SetWindowLongW` - # (WS_EX_APPWINDOW) -- genau der Aufruf, der laut Speicher-Notiz - # "keine Selbstversteck-Fenstertricks" schon AV-Fehlalarme - # ausgeloest hat und deshalb nicht in Frage kommt. Mit der echten - # Titelleiste kommen Taskleiste, Alt+Tab, Aero-Snap, Ziehen und - # Groessenaendern am Rahmen nativ zurueck; die Eigenbau-Loesungen - # dafuer (ziehbare Kopfzeile, Anfasser unten rechts, eigene - # ✕/—-Knoepfe) sind damit ersatzlos entfallen. - self.minsize(MIN_W, MIN_H) + header = tk.Frame(self, bg=BG) + header.pack(fill="x", padx=20, pady=(18, 4)) + tk.Label(header, text="VersaPad Steuermatrix", bg=BG, fg=TEXT, + font=("Segoe UI", 15, "bold")).pack(anchor="w") + self.header_sub = tk.Label(header, text="pollt Config-JSONs alle 1.5s", bg=BG, + fg=TEXT_DIM, font=("Segoe UI", 9)) + self.header_sub.pack(anchor="w") - self.toggles_row = header = tk.Frame(self, bg=BG) - header.pack(fill="x", padx=20, pady=(10, 6)) - tk.Label(header, text="VersaPad", bg=BG, fg=TEXT, - font=("Segoe UI", 11, "bold")).pack(side="left", anchor="w", padx=(0, 16)) - - info_btn = tk.Label(header, text="ⓘ", bg=BG, fg=TEXT_DIM, font=("Segoe UI", 12), - cursor="hand2", padx=6) - info_btn.pack(side="right", padx=(0, 8)) + info_btn = tk.Label(header, text="ⓘ", bg=BG, fg=TEXT_DIM, font=("Segoe UI", 14), + cursor="hand2") + info_btn.place(relx=1.0, x=0, y=-4, anchor="ne") info_btn.bind("", lambda e: self._show_mcp_info()) info_btn.bind("", lambda e: info_btn.configure(fg=ACCENT)) info_btn.bind("", lambda e: info_btn.configure(fg=TEXT_DIM)) - # Modus-Umschalter direkt neben der Ueberschrift statt in eigener - # Zeile -- spart eine komplette Zeile Fensterhoehe. - self.sync_check = tk.Checkbutton( - header, text="Live-Sync", variable=self.live_sync, - command=self._on_toggle_sync, bg=BG, fg=TEXT, selectcolor=CARD_BG, - activebackground=BG, activeforeground=TEXT, font=("Segoe UI", 8)) - self.sync_check.pack(side="left") - self.sync_status = tk.Label(header, text="aus", bg=BG, fg=TEXT_DIM, font=("Segoe UI", 8)) - self.sync_status.pack(side="left", padx=(4, 14)) - - self.edit_check = tk.Checkbutton( - header, text="Programmiermodus", variable=self.editing, - command=self._on_toggle_editing, bg=BG, fg=TEXT, selectcolor=CARD_BG, - activebackground=BG, activeforeground=TEXT, font=("Segoe UI", 8)) - self.edit_check.pack(side="left", padx=(0, 14)) - - self.always_on_top = tk.BooleanVar(value=False) - self.topmost_check = tk.Checkbutton( - header, text="Immer im Vordergrund", variable=self.always_on_top, - command=self._on_toggle_topmost, bg=BG, fg=TEXT, selectcolor=CARD_BG, - activebackground=BG, activeforeground=TEXT, font=("Segoe UI", 8)) - self.topmost_check.pack(side="left") - - # Schmale Toolbar-Zeile fuers Board-I/O -- nur sichtbar im - # Programmiermodus (siehe _on_toggle_editing), direkt unter den - # Checkboxen statt weit unten zwischen Tabs und Matrix. - self.prog_row = tk.Frame(self, bg=BG) - for text, cmd in ( - ("Vom Board laden", self._load_from_board), - ("Zum Board übertragen", self._write_to_board), - ("Datei laden...", self._load_file_dialog), - ("Datei speichern...", self._save_file_dialog), - ): - tk.Button(self.prog_row, text=text, command=cmd, bg=CARD_BG, fg=TEXT, - activebackground=ACCENT, activeforeground="#fff", relief="flat", - padx=8, pady=2, font=("Segoe UI", 8)).pack(side="left", padx=(0, 6)) - self.prog_status = tk.Label(self.prog_row, text="", bg=BG, fg=TEXT_DIM, font=("Segoe UI", 8)) - self.prog_status.pack(side="left", padx=(8, 0)) - # Kurz halten: die Toolbar ist bei der Standardbreite (790px) schon - # fast voll, laengerer Text wird rechts abgeschnitten. - tk.Label(self.prog_row, text="Rechtsklick = kopieren/einfügen", - bg=BG, fg=TEXT_DIM, font=("Segoe UI", 8)).pack(side="right") - # prog_row wird erst bei aktivem Programmiermodus gepackt (siehe _on_toggle_editing) - - # Profil-Tabs direkt ueber der Steuermatrix, nicht mehr oben am - # Fensterkopf -- naeher an dem, was sie auswaehlen. self.tabs = tk.Frame(self, bg=BG) - self.tabs.pack(fill="x", padx=20, pady=(8, 8)) + self.tabs.pack(fill="x", padx=20, pady=(12, 10)) self.tab_buttons = {} for p in range(vp.NUM_PROFILES): btn = tk.Label(self.tabs, text=f"Profil {p}", bg=CARD_BG, fg=TEXT, @@ -248,27 +175,56 @@ class VersaPadViewer(tk.Tk): btn.bind("", lambda e, prof=p: self._rename_tab(prof)) self.tab_buttons[p] = btn + toggles_row = tk.Frame(self, bg=BG) + toggles_row.pack(fill="x", padx=20, pady=(0, 6)) + self.sync_check = tk.Checkbutton( + toggles_row, text="Live-Sync mit Board", variable=self.live_sync, + command=self._on_toggle_sync, bg=BG, fg=TEXT, selectcolor=CARD_BG, + activebackground=BG, activeforeground=TEXT, font=("Segoe UI", 9, "bold")) + self.sync_check.pack(side="left") + self.sync_status = tk.Label(toggles_row, text="aus", bg=BG, fg=TEXT_DIM, font=("Segoe UI", 9)) + self.sync_status.pack(side="left", padx=(8, 20)) + + self.edit_check = tk.Checkbutton( + toggles_row, text="Programmiermodus", variable=self.editing, + command=self._on_toggle_editing, bg=BG, fg=TEXT, selectcolor=CARD_BG, + activebackground=BG, activeforeground=TEXT, font=("Segoe UI", 9, "bold")) + self.edit_check.pack(side="left", padx=(0, 20)) + + self.always_on_top = tk.BooleanVar(value=False) + self.topmost_check = tk.Checkbutton( + toggles_row, text="Immer im Vordergrund", variable=self.always_on_top, + command=self._on_toggle_topmost, bg=BG, fg=TEXT, selectcolor=CARD_BG, + activebackground=BG, activeforeground=TEXT, font=("Segoe UI", 9, "bold")) + self.topmost_check.pack(side="left") + + self.prog_row = tk.Frame(self, bg=BG) + for text, cmd in ( + ("Vom Board laden", self._load_from_board), + ("Zum Board übertragen", self._write_to_board), + ("Datei laden...", self._load_file_dialog), + ("Datei speichern...", self._save_file_dialog), + ): + tk.Button(self.prog_row, text=text, command=cmd, bg=CARD_BG, fg=TEXT, + activebackground=ACCENT, activeforeground="#fff", relief="flat", + padx=10, pady=4, font=("Segoe UI", 9)).pack(side="left", padx=(0, 8)) + self.prog_status = tk.Label(self.prog_row, text="", bg=BG, fg=TEXT_DIM, font=("Segoe UI", 9)) + self.prog_status.pack(side="left", padx=(8, 0)) + # prog_row wird erst bei aktivem Programmiermodus gepackt (siehe _on_toggle_editing) + self.grid_frame = tk.Frame(self, bg=BG) - self.grid_frame.pack(padx=20, pady=(0, 0), anchor="w") + self.grid_frame.pack(padx=20, pady=(8, 0)) tk.Label(self, text="Encoder", bg=BG, fg=TEXT_DIM, font=("Segoe UI", 10, "bold")).pack(anchor="w", padx=20, pady=(20, 8)) self.enc_frame = tk.Frame(self, bg=BG) self.enc_frame.pack(fill="x", padx=20) - footer_row = tk.Frame(self, bg=BG) - footer_row.pack(fill="x", padx=20, pady=(14, 8)) - self.footer = tk.Label(footer_row, text="", bg=BG, fg=TEXT_DIM, font=("Segoe UI", 8)) - self.footer.pack(side="left", anchor="w") + self.footer = tk.Label(self, text="", bg=BG, fg=TEXT_DIM, font=("Segoe UI", 8)) + self.footer.pack(anchor="w", padx=20, pady=(20, 10)) - # Schliessen legt weiterhin ins Tray statt zu beenden (wie die - # offizielle VersaGUI -- das Programm soll im Hintergrund - # weiterlaufen). Minimieren geht jetzt aber ganz normal in die - # Taskleiste; frueher hat ein -Handler auch das ins Tray - # umgeleitet, was ohne Taskleisten-Eintrag sinnvoll war und mit - # einem das Fenster nur unauffindbar machen wuerde. self.protocol("WM_DELETE_WINDOW", self._hide_to_tray) - self._bind_shortcuts() + self.bind("", self._on_unmap) self._serial_thread.start() self.set_profile(0) @@ -414,7 +370,13 @@ class VersaPadViewer(tk.Tk): text = SERIAL_STATUS_TEXT.get(error, error or "Fehler") self.sync_status.configure(text=text, fg=WARN_RED) - # ── Tray-Icon (Schliessen beendet nicht, wie bei der VersaGUI) ── + # ── Tray-Icon (kein Taskleisten-Eintrag beim Minimieren, wie VersaGUI) ── + + def _on_unmap(self, event): + """Minimieren faengt Windows normalerweise als Taskleisten-Icon ab -- + wir wollen stattdessen: Fenster komplett weg, nur noch Tray-Icon.""" + if event.widget is self and self.state() == "iconic": + self._hide_to_tray() def _hide_to_tray(self): self.withdraw() @@ -464,10 +426,12 @@ class VersaPadViewer(tk.Tk): self.combined = vcomb.default_combined() else: self.combined = vcomb.default_combined() - self.prog_row.pack(fill="x", padx=20, pady=(0, 6), after=self.toggles_row) + self.prog_row.pack(fill="x", padx=20, pady=(0, 14), after=self.tabs) + self.header_sub.configure(text="Programmiermodus · Zelle anklicken zum Bearbeiten") else: self.sync_check.configure(state="normal") self.prog_row.pack_forget() + self.header_sub.configure(text="pollt Config-JSONs alle 1.5s") self._update_tab_labels() self._render() @@ -496,9 +460,7 @@ class VersaPadViewer(tk.Tk): macro_slots = vproto.unpack_macros(raw_macros) names = self.combined["profile_names"] if self.combined else None - previous = self.combined - self.combined = vcomb.merge_notes( - vcomb.from_binary(cfg_dict, macro_slots, profile_names=names), previous) + self.combined = vcomb.from_binary(cfg_dict, macro_slots, profile_names=names) self.profile = self.combined["active_profile"] self._status("vom Board geladen", True) self._update_tab_labels() @@ -585,183 +547,6 @@ class VersaPadViewer(tk.Tk): except OSError as e: self._status(f"Auto-Speichern fehlgeschlagen: {e}", False) - # ── Kopieren/Einfuegen + Tastenkuerzel ───────────────────────── - - def _bind_shortcuts(self): - """Fensterweite Kuerzel. Waehrend ein Bearbeiten-Dialog offen ist, - haelt dessen grab_set() die Tastatur -- die Kuerzel hier koennen ihm - also nicht dazwischenfunken.""" - for seq, handler in ( - # Minimieren statt ins Tray: seit das Fenster eine Titelleiste - # und damit einen Taskleisten-Eintrag hat, waere "verschwindet - # spurlos" die unangenehmere Ueberraschung. Ins Tray legt - # weiterhin das ✕ der Titelleiste. - ("", lambda e: self.iconify()), - ("", lambda e: self._rename_tab(self.profile)), - ("", lambda e: self._toggle_editing_shortcut()), - ("", lambda e: self._toggle_editing_shortcut()), - ("", lambda e: self._copy_hovered()), - ("", lambda e: self._copy_hovered()), - ("", lambda e: self._paste_hovered()), - ("", lambda e: self._paste_hovered()), - ): - self.bind(seq, handler) - for p in range(vp.NUM_PROFILES): - self.bind(f"", - lambda e, prof=p: self.set_profile(prof, manual=True)) - - def _toggle_editing_shortcut(self): - self.editing.set(not self.editing.get()) - self._on_toggle_editing() - - def _hint(self, text): - """Kurze Rueckmeldung in der Fusszeile -- die Programmiermodus- - Statuszeile haengt an der Toolbar und ist sonst leicht zu uebersehen. - Der naechste _render() setzt die Quellenangabe wieder ein.""" - self.footer.configure(text=text) - - def _target_entry(self, target): - """Liefert den Eintrag im combined-State, auf den ein Ziel zeigt -- - Button-Karte oder Encoder (dort waehlt target["field"] danach noch - sw/cw/ccw aus).""" - profile = self.combined["profiles"][self.profile] - items = profile["buttons"] if target["kind"] == "button" else profile["encoders"] - return next(item for item in items if item["index"] == target["index"]) - - def _set_hover(self, target): - self._hover = target - - def _clear_hover(self, target): - """ feuert auch beim Wechsel auf ein Kindwidget derselben - Karte -- deshalb erst pruefen, ob der Zeiger die Karte wirklich - verlassen hat.""" - card = target["card"] - if not card.winfo_exists(): - self._hover = None - return - x, y = card.winfo_pointerxy() - inside = (card.winfo_rootx() <= x < card.winfo_rootx() + card.winfo_width() - and card.winfo_rooty() <= y < card.winfo_rooty() + card.winfo_height()) - if not inside and self._hover is target: - self._hover = None - - def _copy_hovered(self): - if self._hover is not None: - self._copy_target(self._hover, "all") - - def _paste_hovered(self): - if self._hover is not None: - self._paste_target(self._hover, "all") - - def _copy_target(self, target, what): - if not self.editing.get() or self.combined is None: - return - entry = self._target_entry(target) - if target["kind"] == "button": - action, led = entry["action"], entry["led"] - else: - action, led = entry[target["field"]], None - self._clip = { - "action": copy.deepcopy(action) if what in ("all", "action") else None, - "led": copy.deepcopy(led) if what in ("all", "led") else None, - } - parts = [name for name, key in (("Belegung", "action"), ("Farbe", "led")) - if self._clip[key] is not None] - self._hint("kopiert: {} von {}".format(" + ".join(parts) or "nichts", target["label"])) - - def _paste_target(self, target, what): - if not self.editing.get() or self.combined is None or not self._clip: - return - entry = self._target_entry(target) - applied = [] - if what in ("all", "action") and self._clip["action"] is not None: - action = copy.deepcopy(self._clip["action"]) - if target["kind"] == "button": - entry["action"] = action - else: - entry[target["field"]] = action - applied.append("Belegung") - if what in ("all", "led") and self._clip["led"] is not None and target["kind"] == "button": - entry["led"] = copy.deepcopy(self._clip["led"]) - applied.append("Farbe") - if not applied: - self._hint("nichts eingefügt -- Zwischenablage passt nicht zu diesem Ziel") - return - self._autosave_combined() - self._render() - self._hint("eingefügt: {} → {}".format(" + ".join(applied), target["label"])) - - def _clear_target(self, target): - if not self.editing.get() or self.combined is None: - return - entry = self._target_entry(target) - # Notiz bleibt bewusst erhalten (gleiche Regel wie beim Typwechsel im - # Dialog): sie beschreibt die Taste, nicht die konkrete Aktion. - if target["kind"] == "button": - entry["action"] = {"type": "None", "data": 0, - "note": entry["action"].get("note", "")} - else: - old = entry[target["field"]] - entry[target["field"]] = {"type": "None", "data": 0, "note": old.get("note", "")} - self._autosave_combined() - self._render() - self._hint("geleert: {}".format(target["label"])) - - def _show_context_menu(self, event, target): - menu = tk.Menu(self, tearoff=0, bg=CARD_BG, fg=TEXT, activebackground=ACCENT, - activeforeground="#ffffff", bd=0, font=("Segoe UI", 9)) - is_button = target["kind"] == "button" - has_action = bool(self._clip and self._clip.get("action")) - has_led = bool(self._clip and self._clip.get("led")) - - menu.add_command(label="Bearbeiten…", command=lambda: self._edit_target(target)) - menu.add_separator() - if is_button: - menu.add_command(label="Kopieren: Belegung + Farbe (Strg+C)", - command=lambda: self._copy_target(target, "all")) - menu.add_command(label="Kopieren: nur Belegung", - command=lambda: self._copy_target(target, "action")) - menu.add_command(label="Kopieren: nur Farbe", - command=lambda: self._copy_target(target, "led")) - else: - menu.add_command(label="Kopieren: Belegung (Strg+C)", - command=lambda: self._copy_target(target, "action")) - menu.add_separator() - paste_all = has_action or (has_led and is_button) - menu.add_command(label="Einfügen: alles (Strg+V)", - state="normal" if paste_all else "disabled", - command=lambda: self._paste_target(target, "all")) - menu.add_command(label="Einfügen: nur Belegung", - state="normal" if has_action else "disabled", - command=lambda: self._paste_target(target, "action")) - if is_button: - menu.add_command(label="Einfügen: nur Farbe", - state="normal" if has_led else "disabled", - command=lambda: self._paste_target(target, "led")) - menu.add_separator() - menu.add_command(label="Leeren (Belegung entfernen)", - command=lambda: self._clear_target(target)) - try: - menu.tk_popup(event.x_root, event.y_root) - finally: - menu.grab_release() - - def _edit_target(self, target): - if target["kind"] == "button": - self._edit_button(target["index"]) - else: - self._edit_encoder_action(target["index"], target["field"], target["field_label"]) - - def _bind_cell_interaction(self, card, widgets, target): - """Linksklick = bearbeiten, Rechtsklick = Kontextmenue; der - Mauszeiger merkt sich das Ziel fuer Strg+C/Strg+V.""" - card.configure(cursor="hand2") - for widget in widgets: - widget.bind("", lambda e: self._edit_target(target)) - widget.bind("", lambda e: self._show_context_menu(e, target)) - widget.bind("", lambda e: self._set_hover(target)) - card.bind("", lambda e: self._clear_hover(target)) - # ── Rendering ──────────────────────────────────────────────────── def _current_profile_view(self): @@ -786,7 +571,7 @@ class VersaPadViewer(tk.Tk): cfg = vp.annotate_profile({ "buttons": [dict(b) for b in raw["buttons"]], "encoders": [dict(e) for e in raw["encoders"]], - }, data.get("macros")) + }) return cfg, f"Quelle: {vcomb.DEFAULT_PATH}" except (KeyError, IndexError, ValueError): pass # kaputte/unvollstaendige Datei -- weiter unten ausweichen @@ -812,7 +597,7 @@ class VersaPadViewer(tk.Tk): cfg = vp.annotate_profile({ "buttons": [dict(b) for b in raw["buttons"]], "encoders": [dict(e) for e in raw["encoders"]], - }, self.combined.get("macros")) + }) source_text = "Programmiermodus -- nicht gespeichert, bis übertragen/exportiert" else: try: @@ -854,23 +639,19 @@ class VersaPadViewer(tk.Tk): tk.Label(card, text=label, bg=CARD_BG, fg=TEXT_EMPTY if empty else TEXT, font=("Segoe UI", 10, "normal" if empty else "bold"), wraplength=CARD_W - 20, justify="left", anchor="nw").place( - x=10, y=24, width=CARD_W - 20, height=28) - - # Notiz bekommt den ganzen Rest der Karte (bis zur Animationszeile) -- - # bei 34px wurden laengere Notizen mitten im Wort abgeschnitten. - tk.Label(card, text=btn["note"], bg=CARD_BG, fg=TEXT_DIM, - font=("Segoe UI", 8), wraplength=CARD_W - 20, justify="left", anchor="nw").place( - x=10, y=53, width=CARD_W - 20, height=CARD_H - 69) + x=10, y=26, width=CARD_W - 20, height=36) anim = "" if empty else vp.ANIM_LABELS.get(btn["led"]["anim"], btn["led"]["anim"]) tk.Label(card, text=anim, bg=CARD_BG, fg=TEXT_DIM, font=("Segoe UI", 7), anchor="sw").place( - x=10, y=CARD_H - 16, width=CARD_W - 20, height=13) + x=10, y=CARD_H - 20, width=CARD_W - 20, height=14) if editable: - target = {"kind": "button", "index": btn["index"], "card": card, - "label": "Button #{}".format(btn["index"])} - self._bind_cell_interaction(card, [card] + list(card.winfo_children()), target) + card.configure(cursor="hand2") + handler = lambda e, idx=btn["index"]: self._edit_button(idx) + card.bind("", handler) + for child in card.winfo_children(): + child.bind("", handler) def _render_encoder(self, enc, editable=False): card = tk.Frame(self.enc_frame, bg=CARD_BG, highlightbackground=CARD_BORDER, @@ -880,31 +661,18 @@ class VersaPadViewer(tk.Tk): inner.pack(fill="both", expand=True, padx=10, pady=8) tk.Label(inner, text=f"Encoder {enc['index']}", bg=CARD_BG, fg=TEXT_DIM, font=("Segoe UI", 8, "bold")).pack(anchor="w", pady=(0, 4)) - for key, field, val, note in (("Druck", "sw", enc["sw_label"], enc["sw_note"]), - ("CW", "cw", enc["cw_label"], enc["cw_note"]), - ("CCW", "ccw", enc["ccw_label"], enc["ccw_note"])): + for key, field, val in (("Druck", "sw", enc["sw_label"]), ("CW", "cw", enc["cw_label"]), + ("CCW", "ccw", enc["ccw_label"])): row = tk.Frame(inner, bg=CARD_BG) - row.pack(fill="x", pady=(1, 4)) - top = tk.Frame(row, bg=CARD_BG) - top.pack(fill="x") - tk.Label(top, text=key, bg=CARD_BG, fg=TEXT_DIM, font=("Segoe UI", 8)).pack(side="left") - tk.Label(top, text=val or "—", bg=CARD_BG, fg=TEXT, font=("Segoe UI", 8, "bold")).pack(side="right") - # Notizzeile nur anlegen, wenn es wirklich eine Notiz gibt -- - # ein leeres Label belegt sonst pro Encoder-Aktion eine Zeile - # Hoehe (12x im Fenster) und schiebt die Fusszeile aus dem Bild. - note_label = None - if note: - note_label = tk.Label(row, text=note, bg=CARD_BG, fg=TEXT_DIM, font=("Segoe UI", 7), - wraplength=150, justify="left", anchor="w") - note_label.pack(fill="x") + row.pack(fill="x", pady=1) + tk.Label(row, text=key, bg=CARD_BG, fg=TEXT_DIM, font=("Segoe UI", 8)).pack(side="left") + tk.Label(row, text=val or "—", bg=CARD_BG, fg=TEXT, font=("Segoe UI", 8, "bold")).pack(side="right") if editable: - target = {"kind": "encoder", "index": enc["index"], "field": field, - "field_label": key, "card": row, - "label": "Encoder {} {}".format(enc["index"], key)} - widgets = [row, top] + list(top.winfo_children()) - if note_label is not None: - widgets.append(note_label) - self._bind_cell_interaction(row, widgets, target) + row.configure(cursor="hand2") + handler = lambda e, ei=enc["index"], f=field, lbl=key: self._edit_encoder_action(ei, f, lbl) + row.bind("", handler) + for child in row.winfo_children(): + child.bind("", handler) if __name__ == "__main__": diff --git a/docs/architecture.md b/docs/architecture.md index 66424cd..142fc34 100644 --- a/docs/architecture.md +++ b/docs/architecture.md @@ -93,17 +93,7 @@ auf. umschaltbaren Modi (siehe „Modi" unten), Tray-Icon, Info-Dialog mit MCP-Doku. - **`action_dialog.py`** — modale Bearbeiten-Dialoge (`ActionEditDialog`, - `MacroStepsDialog`) für den Programmiermodus, auf gemeinsamer Basis - `_ModalDialog`: - - positioniert sich beim Öffnen mittig über dem aufrufenden Fenster - (Aufbau `withdraw()`n, `_center_on_parent()`, dann `deiconify()`); - - bindet ``/`` auf dem Toplevel und verteilt sie: - entweder an eine laufende Tastendruck-Aufnahme, sonst als - Enter = OK / Escape = Abbrechen; - - `_KeyCapture` schaltet ein Label in den Aufnahmemodus und schickt jeden - Tastendruck durch `versapad_data.tk_event_to_hid()` — reine - Tk-Fenster-Events, **kein globaler Tastaturhook** (siehe „Grenzen der - Tastendruck-Erkennung" unten). + `MacroStepsDialog`) für den Programmiermodus. ### MCP-Server @@ -120,18 +110,13 @@ auf. `versapad_data.app_dir()` bestimmt das Basisverzeichnis für die eigene Config-Datei (`versapad_combined.DEFAULT_PATH` = -`app_dir()/versapad_config_all.json`): `%APPDATA%\VersaPadViewer` -(Roaming-AppData), unabhängig davon, ob die gebaute `.exe` oder der -Quellcode gestartet wurde (kein `sys.frozen`-Zweig). +`app_dir()/versapad_config_all.json`): -Bewusst **nicht** das Installationsverzeichnis (`%LOCALAPPDATA%\ -VersaPadViewer`, wo die `.exe` liegt): `build_and_deploy.ps1` räumt das -Zielverzeichnis vor jedem Deploy komplett ab, läge die Config dort, würde -jeder Rebuild sie mitlöschen (genau das ist am 2026-08-15 passiert, siehe -`AGENTS.md`). Ebenso bewusst **nicht** vom Startweg abhängig — sonst sähe -die `.exe` eine andere Datei als ein aus dem Quellcode gestarteter -`versapad_mcp_server.py`, und Änderungen aus dem einen Weg wären im -anderen unsichtbar (ebenfalls am 2026-08-15 beobachtet). +- **Gebaute `.exe`** (PyInstaller `--onedir`): `sys.executable`s Ordner — + die Config liegt also neben `VersaPadViewer.exe`, in welchem + Installationsverzeichnis sie auch liegt. +- **Start aus dem Quellcode** (`py desktop_viewer.py`, `py server.py`, + `py versapad_mcp_server.py`): der Projektordner (`__file__`-Verzeichnis). Fehlt die Datei, legt `load_or_fetch()` sie automatisch an — zuerst per Serial-Versuch vom Board (das ist die eigentliche Quelle der Wahrheit, die @@ -155,70 +140,6 @@ Drei Checkboxen, unabhängig voneinander: | Live-Sync | Fragt per Serial das aktuell aktive Profil ab, schaltet die Ansicht mit | Hintergrund-Thread pollt `read_active_profile()`, hält dafür den COM-Port dauerhaft offen, solange die Checkbox an ist. Schließt sich mit VersaGUI/Programmiermodus/MCP-Board-Zugriff gegenseitig aus (exklusiver Port). | | Programmiermodus | Zellen anklicken zum Bearbeiten | Lädt `self.combined` **einmalig pro Prozesslauf** beim ersten Aktivieren (bevorzugt `DEFAULT_PATH`, sonst `default_combined()`). Jede Bearbeitung speichert sofort automatisch (`_autosave_combined()`). **Achtung:** Da der Snapshot nur einmal geladen wird, sieht der Programmiermodus externe Änderungen (z.B. per MCP) erst nach einem Neustart der exe oder einem expliziten „Datei laden…“. | -### Tastendruck-Erkennung: Position statt Zeichen - -Die Erkennung im Bearbeiten-Dialog nutzt ausschließlich Tk-Events des -fokussierten Fensters — kein globaler `SetWindowsHookEx`-Hook (siehe -`AGENTS.md`). Erste Folge: **nur was das Fenster erreicht, wird erkannt.** -Win+L, Strg+Alt+Entf und andere vom Betriebssystem abgefangene -Kombinationen kommen nie an. - -Zweite und wichtigere Folge betrifft die *Zuordnung*. HID-Keycodes -bezeichnen **physische Tastenpositionen**: das Board sendet eine Position, -erst Windows macht daraus über das aktive Layout ein Zeichen. Wer die -Zuordnung über das *Zeichen* aufbaut, dreht diese Kette falsch herum — auf -deutschem Layout landete dadurch jedes Y auf der Z-Taste des Boards und -ÄÖÜ/#/+ waren gar nicht erfassbar. `versapad_data.tk_event_to_hid()` löst -deshalb in dieser Reihenfolge auf: - -1. **Benannte Tasten über den Tk-keysym** (Enter, Escape, Pfeile, F-Tasten, - Numpad, Entf …). Layoutunabhängig eindeutig — und hier zwingend, weil - `MapVirtualKeyW` für die Pfeiltasten denselben Scan-Code liefert wie für - ihre Numpad-Zwillinge (gemessen: VK_LEFT und VK_NUMPAD4 beide 0x4B). -2. **Zeichentasten über die physische Position** — - `versapad_keylayout.hid_for_vk()`: Virtual-Key → Scan-Code - (`MapVirtualKeyW`) → HID über die layoutunabhängige Tabelle - `SCANCODE_TO_HID`. Der Virtual-Key ist unabhängig davon, ob Shift oder - AltGr mitgehalten wird. -3. **Näherung ohne WinAPI** (keysym-Zeichentabelle, dann VK-Tabelle) — nur - relevant, wenn `versapad_keylayout` nicht verfügbar ist (Nicht-Windows, - kein ctypes). Auf dieser Ebene bleibt es bei der US-Layout-Näherung - inklusive vertauschtem Y/Z. - -### Tastenbeschriftungen - -`versapad_data.hid_key_name()` fragt für Zeichentasten `GetKeyNameTextW` -und zeigt damit den Namen des aktiven Layouts (deutsch: HID 0x1C → „Z“, -0x34 → „ä“). Für alles andere bleiben die gepflegten deutschen Namen aus -`_SPECIAL_KEYS` („Enter“, „Bild↑“, „Num5“) — die lesen sich besser als das, -was Windows dafür liefert („EINGABE“, „4 (ZEHNERTASTATUR)“). - -Diese Namen sind zugleich Schlüssel (Dropdown-Einträge, -`hid_key_code_for_name()` für den MCP-Server) und müssen eindeutig bleiben. -Echte Kollisionen kommen vor: auf deutschem Layout heißt HID 0x31 schlicht -„#“, und diesen Namen trug bisher HID 0x32. Der Layoutname gewinnt, der -verdrängte US-Name wird als „# (US-Layout)“ gekennzeichnet statt verworfen. -`hid_key_code_for_name()` akzeptiert beide Schreibweisen; bei Kollision -gewinnt das Layout, damit `set_button_key(key="Z")` die Taste trifft, die -auf dieser Tastatur ein Z tippt. - -Bestehende Belegungen ändern dadurch ihre **Anzeige, nicht ihre Daten**: -eine früher über das Dropdown gesetzte „Z“ steht als 0x1D in der Config und -wird jetzt wahrheitsgemäß als „Y“ angezeigt, weil sie auf dieser Tastatur -ein Y tippt. Ein Layoutwechsel zur Laufzeit wird nicht nachgezogen. - -### Kopieren/Einfügen zwischen Tasten - -Rechtsklick auf eine Karte im Programmiermodus (bzw. `Strg+C`/`Strg+V` auf -der Karte unter dem Mauszeiger) kopiert Belegung und/oder LED-Farbe auf -andere Tasten. Die Ablage ist eine reine In-Memory-Struktur in -`desktop_viewer.VersaPadViewer._clip` (`{"action": …, "led": …}`), **nicht** -die System-Zwischenablage — dort lägen nur Textrepräsentationen, hier -werden ganze Action-Dicts übertragen. Eingefügt wird immer eine -`copy.deepcopy()`, damit zwei Tasten nicht dasselbe Dict teilen. Encoder -haben keine eigene LED; eine kopierte Farbe auf einen Encoder einzufügen ist -deshalb wirkungslos. - Tk-Aufrufe passieren nie direkt aus dem Serial- oder Tray-Hintergrundthread — Ergebnisse landen in einer `queue.Queue`, der Main-Thread holt sie per `after()`-Polling ab (Absturzrisiko bei Cross-Thread-Tk-Zugriff, siehe @@ -256,11 +177,8 @@ statt dem ACK allein zu vertrauen. Details siehe „Bug beobachtet `build_and_deploy.ps1` installiert `requirements.txt` selbst (`pip install -r`), baut mit PyInstaller (`--onedir --windowed`, nur `desktop_viewer.py` wird gebündelt) und kopiert das Ergebnis nach -`%LOCALAPPDATA%\VersaPadViewer\`. `--onedir` statt `--onefile`, um -AV-Fehlalarme zu verringern. Räumt das Zielverzeichnis vor dem Kopieren -komplett ab, rettet dabei aber zuvor gefundene `versapad_config*.json` -(Altinstallationen, bei denen die Config noch im Installationsordner -liegt) über den Deploy hinweg. Läuft komplett in try/catch mit +`dist\VersaPadViewer\` im Projektordner. `--onedir` statt `--onefile`, um +AV-Fehlalarme zu verringern. Läuft komplett in try/catch mit Exit-Code-Prüfung und pausiert am Ende (Erfolg wie Fehler) auf Tastendruck, außer bei `-NoPause`. Muss lokal laufen, nicht auf einem Netzlaufwerk (Pfadlängen-/DLL-Ladeprobleme, siehe `AGENTS.md`). Details: diff --git a/docs/data-model.md b/docs/data-model.md index 3491f11..88d160f 100644 --- a/docs/data-model.md +++ b/docs/data-model.md @@ -50,16 +50,6 @@ siehe unten): | `0x04` | Alt | | `0x08` | Win | -Jede `Action` trägt zusätzlich ein optionales `note`-Feld (freier Text, -z.B. `"Speichern in Fusion 360"`) — **rein lokal**, wie `profile_names` -(siehe unten): kein Platz dafür in `SAction` (3 Byte, komplett verplant), -`to_binary()`/`pack_config()` ignorieren das Feld beim Schreiben ans -Board, `from_binary()` liefert frisch vom Board immer `note=""`. -`versapad_combined.merge_notes(neu, alt)` kopiert bestehende Notizen nach -jedem `load_from_board()`/`fetch_from_board()` zurück, sonst gingen sie -bei jedem Board-Refresh verloren. Editierbar per Programmiermodus-Dialog -oder MCP (`set_button_note()`/`set_encoder_note()`). - ## LED Pro MX-Button (nicht pro Encoder — Encoder haben keine eigene LED): @@ -110,7 +100,7 @@ kompletten 740B-Block, nie nur ein Profil. "buttons": [ { "index": 0, - "action": { "type": "HidKey", "data": 30, "note": "Speichern in Fusion 360" }, + "action": { "type": "HidKey", "data": 30 }, "led": { "r": 80, "g": 40, "b": 0, "brightness": 255, "anim": "Static", "period_ms": 4000 } } @@ -138,8 +128,7 @@ kompletten 740B-Block, nie nur ein Profil. Profilnamen (`profile_names`) sind **rein lokal** — die Firmware-Structs haben keinen Platz für einen String (Header exakt 32B, jedes Profil exakt 236B, alles verplant), sie landen nie aufs Board, egal welcher -Schreibpfad benutzt wird. Dasselbe gilt für `note` in jeder `Action` -(siehe oben). +Schreibpfad benutzt wird. ## JSON: Legacy-Einzeldatei-Format (`versapad_config1/2/3.json`) diff --git a/server.py b/server.py index 7e1b181..9188d64 100644 --- a/server.py +++ b/server.py @@ -38,7 +38,7 @@ h1 { font-size: 20px; font-weight: 600; margin: 0 0 4px; } .grid { display: grid; grid-template-columns: repeat(4, 140px); - grid-template-rows: repeat(5, 104px); + grid-template-rows: repeat(5, 76px); grid-auto-flow: column; gap: 10px; margin-bottom: 36px; @@ -53,17 +53,15 @@ h1 { font-size: 20px; font-weight: 600; margin: 0 0 4px; } } .cell .idx { position: absolute; top: 8px; right: 10px; font-size: 11px; color: #6a6d78; } .cell .label { font-size: 14px; font-weight: 600; line-height: 1.25; word-break: break-word; } -.cell .note { font-size: 11px; color: #8a8d98; margin-top: 4px; word-break: break-word; } .cell .anim { font-size: 11px; color: #8a8d98; margin-top: 2px; } .cell.empty .label { color: #4a4d58; font-weight: 400; } h2 { font-size: 15px; font-weight: 600; color: #c4c6cf; margin: 0 0 12px; } .encoders { display: grid; grid-template-columns: repeat(4, 1fr); gap: 12px; max-width: 720px; } .enc { background: #1e2129; border: 1px solid #2a2d37; border-radius: 10px; padding: 12px 14px; } .enc .idx { font-size: 12px; color: #6a6d78; margin-bottom: 8px; } -.enc .row { display: flex; justify-content: space-between; font-size: 13px; padding: 3px 0 0; } +.enc .row { display: flex; justify-content: space-between; font-size: 13px; padding: 3px 0; } .enc .row .k { color: #8a8d98; } .enc .row .v { font-weight: 500; text-align: right; } -.enc .note { font-size: 11px; color: #6a6d78; padding-bottom: 6px; word-break: break-word; } footer { margin-top: 40px; color: #6a6d78; font-size: 12px; } """ @@ -75,31 +73,20 @@ def render_cell(btn): grid_col, grid_row = btn["col"] + 1, btn["row"] + 1 style = f"grid-column:{grid_col}; grid-row:{grid_row};" label = html.escape(btn["label"]) if btn["label"] else "—" - note = html.escape(btn["note"]) if btn.get("note") else "" return f"""
#{btn['index']}
{label}
-
{note}
{"" if empty else anim}
""" -def _encoder_row(key, label, note): - label = html.escape(label) or "—" - note_html = f'
{html.escape(note)}
' if note else "" - return f"""
{key}{label}
{note_html}""" - - def render_encoder(enc): - rows = ( - _encoder_row("Druck", enc["sw_label"], enc["sw_note"]) - + _encoder_row("CW", enc["cw_label"], enc["cw_note"]) - + _encoder_row("CCW", enc["ccw_label"], enc["ccw_note"]) - ) return f"""
Encoder {enc['index']}
- {rows} +
Druck{html.escape(enc['sw_label']) or '—'}
+
CW{html.escape(enc['cw_label']) or '—'}
+
CCW{html.escape(enc['ccw_label']) or '—'}
""" @@ -109,7 +96,7 @@ def render_page(profile): cfg = vp.annotate_profile({ "buttons": [dict(b) for b in raw["buttons"]], "encoders": [dict(e) for e in raw["encoders"]], - }, combined.get("macros")) + }) tabs = "".join( f'{html.escape(combined["profile_names"][p])}' for p in range(vp.NUM_PROFILES) diff --git a/versapad_combined.py b/versapad_combined.py index da1f145..aad8f48 100644 --- a/versapad_combined.py +++ b/versapad_combined.py @@ -25,10 +25,10 @@ DEFAULT_NAMES = ["Windows", "Fusion 360", "BricsCAD"] def _empty_profile(): - buttons = [{"index": i, "action": {"type": "None", "data": 0, "note": ""}, + buttons = [{"index": i, "action": {"type": "None", "data": 0}, "led": {"r": 80, "g": 40, "b": 0, "brightness": 255, "anim": "Static", "period_ms": 4000}} for i in range(20)] - none = {"type": "None", "data": 0, "note": ""} + none = {"type": "None", "data": 0} encoders = [{"index": i, "sw": dict(none), "cw": dict(none), "ccw": dict(none)} for i in range(4)] return {"buttons": buttons, "encoders": encoders} @@ -61,54 +61,17 @@ def default_combined(): def from_binary(config_dict, macro_slots, profile_names=None): """config_dict: Ergebnis von versapad_protocol.unpack_config(). - macro_slots: Ergebnis von versapad_protocol.unpack_macros(). Actions - kommen frisch vom Board ohne "note" (die Firmware kennt keine Notizen, - siehe merge_notes()) -- hier nur mit leerem Default versehen, damit das - Feld ueberall verlaesslich existiert.""" - profiles = config_dict["profiles"] - for profile in profiles: - for b in profile["buttons"]: - b["action"].setdefault("note", "") - for e in profile["encoders"]: - for field in ("sw", "cw", "ccw"): - e[field].setdefault("note", "") + macro_slots: Ergebnis von versapad_protocol.unpack_macros().""" return { "active_profile": config_dict["active_profile"], "global_brightness": config_dict["global_brightness"], "enc_sensitivity": config_dict["enc_sensitivity"], "profile_names": profile_names or list(DEFAULT_NAMES), - "profiles": profiles, + "profiles": config_dict["profiles"], "macros": macro_slots, } -def merge_notes(combined, previous): - """Kopiert Notizen (Button/Encoder-Aktion) aus einem vorherigen State in - einen frisch vom Board gelesenen State -- wie profile_names sind Notizen - rein lokal und wuerden bei jedem load_from_board()/fetch_from_board() - sonst verloren gehen, weil die Firmware sie nicht kennt. previous=None - (z.B. allererstes Laden) -> nichts zu tun, combined unveraendert - zurueckgegeben. Aendert combined in-place und gibt es zurueck.""" - if not previous: - return combined - for p_idx, profile in enumerate(combined["profiles"]): - if p_idx >= len(previous["profiles"]): - continue - prev_profile = previous["profiles"][p_idx] - prev_buttons = {b["index"]: b for b in prev_profile.get("buttons", [])} - for b in profile["buttons"]: - prev = prev_buttons.get(b["index"]) - if prev: - b["action"]["note"] = prev["action"].get("note", "") - prev_encoders = {e["index"]: e for e in prev_profile.get("encoders", [])} - for e in profile["encoders"]: - prev = prev_encoders.get(e["index"]) - if prev: - for field in ("sw", "cw", "ccw"): - e[field]["note"] = prev[field].get("note", "") - return combined - - def to_binary(combined): """-> (config_bytes[740], macro_bytes[512])""" config_bytes = proto.pack_config({ @@ -122,7 +85,6 @@ def to_binary(combined): def save_file(combined, path=DEFAULT_PATH): - os.makedirs(os.path.dirname(path), exist_ok=True) with open(path, "w", encoding="utf-8") as f: json.dump(combined, f, indent=2, ensure_ascii=False) return path diff --git a/versapad_data.py b/versapad_data.py index 46705ec..0e147c6 100644 --- a/versapad_data.py +++ b/versapad_data.py @@ -7,41 +7,21 @@ Kein Schreibzugriff auf die JSONs -- reines Lesen/Anzeigen. """ import json import os - -import versapad_keylayout as kl +import sys NUM_PROFILES = 3 -# Groesse der globalen Makrotabelle (SMacroTable, siehe versapad_protocol. -# MACRO_SLOTS) -- hier nochmal, damit dieses Modul importfrei bleibt. -MACRO_SLOTS = 32 - -APP_NAME = "VersaPadViewer" - def app_dir(): """Verzeichnis fuer die eigene Config-Datei (versapad_config_all.json, - siehe versapad_combined.DEFAULT_PATH). Programmatisch aus der Umgebung - abgeleitet, kein hartkodierter Pfad -- laeuft so auf jeder Maschine und - unter jedem Benutzer. - - Bewusst NICHT vom Startweg abhaengig (kein `sys.frozen`-Zweig): die - gebaute .exe, der Start aus dem Quellcode und der MCP-Server muessen - dieselbe Datei sehen, sonst laufen zwei Configs auseinander und - Aenderungen aus dem einen Weg sind im anderen unsichtbar (genau das ist - am 2026-08-15 passiert -- .exe zeigte ein anderes Profil 0 als der - MCP-Server). - - Bewusst auch NICHT das Installationsverzeichnis (%LOCALAPPDATA%\\ - VersaPadViewer, wo die .exe liegt): `build_and_deploy.ps1` raeumt das - Zielverzeichnis vor jedem Deploy komplett ab (`Remove-Item -Recurse`) -- - laege die Config dort, wuerde JEDER Rebuild die Nutzerdaten mitloeschen - (am 2026-08-15 genau so passiert, alle Notizen weg). Programm- und - Datenverzeichnis bleiben deshalb getrennt: Roaming-AppData fuer die - Config.""" - base = (os.environ.get("APPDATA") or os.environ.get("LOCALAPPDATA") - or os.path.join(os.path.expanduser("~"), ".local", "share")) - return os.path.join(base, APP_NAME) + siehe versapad_combined.DEFAULT_PATH): bei der gebauten .exe (--onedir) + das Installationsverzeichnis neben der .exe, sonst der Ordner dieses + Moduls (Projektordner beim Start aus dem Quellcode). Kein hartkodierter + Pfad mehr -- so laesst sich das Tool auf jede Maschine kopieren/ + installieren, ohne Pfade von Hand anzupassen.""" + if getattr(sys, "frozen", False): + return os.path.dirname(sys.executable) + return os.path.dirname(os.path.abspath(__file__)) # CONFIG_PATHS zeigt bewusst weiterhin auf den OneDrive-Desktop -- das sind @@ -114,164 +94,6 @@ _CONSUMER_NAMES = { 0x00B0: "Aufnahme", } -# ── Tastendruck-Erkennung (Tk-Events -> HID) ────────────────────────────── -# -# Bewusst OHNE WinAPI-Hook: die Zuordnung arbeitet nur mit dem, was Tk beim -# Fokus auf dem Bearbeiten-Dialog ohnehin liefert (keysym, keycode, state). -# Damit bleibt es ein normales Fenster-Tastaturereignis -- kein globaler -# Low-Level-Hook, der (wie die Fensterverstecktricks in anderen Projekten) -# AV-Fehlalarme provozieren koennte. Preis: erkannt wird nur, was das -# fokussierte Fenster ueberhaupt erreicht (Win+L, Strg+Alt+Entf und -# aehnliche vom System abgefangene Kombinationen also nicht). - -# Modifier-Tasten selbst sind nie das "Ziel" eines Captures, sie setzen nur -# Bits -- Win ist hier mit dabei (Einzeltaste erlaubt Win), Makro-Schritte -# filtern es spaeter selbst wieder raus (Firmware kennt dort kein Win). -TK_MODIFIER_KEYSYMS = { - "Control_L": 0x01, "Control_R": 0x01, - "Shift_L": 0x02, "Shift_R": 0x02, - "Alt_L": 0x04, "Alt_R": 0x04, "ISO_Level3_Shift": 0x04, - "Super_L": 0x08, "Super_R": 0x08, "Win_L": 0x08, "Win_R": 0x08, -} - -# event.state-Bits unter Windows-Tk (Fallback, falls ein KeyRelease der -# Modifier-Taste verloren ging -- z.B. weil der Dialog waehrenddessen den -# Fokus hatte/verlor). -TK_STATE_MODIFIER_BITS = [(0x0001, 0x02), (0x0004, 0x01), (0x20000, 0x04)] - -# Benannte Tasten: keysym ist layoutunabhaengig, deshalb erste Wahl. -_TK_NAMED_KEYSYMS = { - "Return": 0x28, "Escape": 0x29, "BackSpace": 0x2A, "Tab": 0x2B, - "ISO_Left_Tab": 0x2B, "space": 0x2C, "Caps_Lock": 0x39, - "Print": 0x46, "Scroll_Lock": 0x47, "Pause": 0x48, "Cancel": 0x48, - "Insert": 0x49, "Home": 0x4A, "Prior": 0x4B, "Delete": 0x4C, - "End": 0x4D, "Next": 0x4E, - "Right": 0x4F, "Left": 0x50, "Down": 0x51, "Up": 0x52, - "Num_Lock": 0x53, "KP_Divide": 0x54, "KP_Multiply": 0x55, - "KP_Subtract": 0x56, "KP_Add": 0x57, "KP_Enter": 0x58, - "KP_1": 0x59, "KP_End": 0x59, "KP_2": 0x5A, "KP_Down": 0x5A, - "KP_3": 0x5B, "KP_Next": 0x5B, "KP_4": 0x5C, "KP_Left": 0x5C, - "KP_5": 0x5D, "KP_Begin": 0x5D, "KP_6": 0x5E, "KP_Right": 0x5E, - "KP_7": 0x5F, "KP_Home": 0x5F, "KP_8": 0x60, "KP_Up": 0x60, - "KP_9": 0x61, "KP_Prior": 0x61, "KP_0": 0x62, "KP_Insert": 0x62, - "KP_Decimal": 0x63, "KP_Delete": 0x63, - "Menu": 0x65, "App": 0x65, -} -for _i in range(12): - _TK_NAMED_KEYSYMS[f"F{_i + 1}"] = 0x3A + _i - -# Zeichentasten: HID-Keycodes sind US-Positionen, die keysyms kommen aber vom -# aktiven Windows-Layout. Beide Belegungen (US + Deutsch) stehen deshalb -# nebeneinander. Einzige echte Kollision ist "minus" (US-Layout: Taste neben -# der 0 = 0x2D, deutsches Layout: Taste neben dem Punkt = 0x38) -- dort -# gewinnt die US-Position, damit die Erkennung dieselbe Naeherung liefert wie -# die Dropdown-Beschriftung (siehe Layout-Vorbehalt bei tk_event_to_hid). -_TK_CHAR_KEYSYMS = { - # US-Layout - "minus": 0x2D, "equal": 0x2E, "bracketleft": 0x2F, "bracketright": 0x30, - "backslash": 0x31, "semicolon": 0x33, "apostrophe": 0x34, - "quoteright": 0x34, "grave": 0x35, "quoteleft": 0x35, - "comma": 0x36, "period": 0x37, "slash": 0x38, - # Deutsches Layout -- gleiche physische Tasten, andere Zeichen - "ssharp": 0x2D, "acute": 0x2E, "dead_acute": 0x2E, - "udiaeresis": 0x2F, "plus": 0x30, "numbersign": 0x32, - "odiaeresis": 0x33, "adiaeresis": 0x34, - "asciicircum": 0x35, "dead_circumflex": 0x35, - "less": 0x64, "greater": 0x64, "bar": 0x64, -} - -# Windows-Virtual-Key-Codes (event.keycode) als Rueckfallebene, wenn der -# keysym nichts hergibt -- z.B. wenn Shift/AltGr das Zeichen veraendert -# ("exclam" statt "1"). Nur die Bereiche, die layoutstabil sind. -_WIN_VK_TO_HID = { - 0x08: 0x2A, 0x09: 0x2B, 0x0D: 0x28, 0x13: 0x48, 0x14: 0x39, 0x1B: 0x29, - 0x20: 0x2C, 0x21: 0x4B, 0x22: 0x4E, 0x23: 0x4D, 0x24: 0x4A, - 0x25: 0x50, 0x26: 0x52, 0x27: 0x4F, 0x28: 0x51, - 0x2C: 0x46, 0x2D: 0x49, 0x2E: 0x4C, - 0x30: 0x27, 0x6A: 0x55, 0x6B: 0x57, 0x6D: 0x56, 0x6E: 0x63, 0x6F: 0x54, - 0x90: 0x53, 0x91: 0x47, 0x5D: 0x65, -} -for _i in range(9): - _WIN_VK_TO_HID[0x31 + _i] = 0x1E + _i # '1'-'9' -for _i in range(26): - _WIN_VK_TO_HID[0x41 + _i] = 0x04 + _i # 'A'-'Z' -for _i in range(12): - _WIN_VK_TO_HID[0x70 + _i] = 0x3A + _i # F1-F12 -_WIN_VK_TO_HID[0x60] = 0x62 # Numpad 0 -for _i in range(9): - _WIN_VK_TO_HID[0x61 + _i] = 0x59 + _i # Numpad 1-9 - - -def tk_keysym_to_hid(keysym): - """Tk-keysym -> HID-Keycode, oder None wenn nicht zuordenbar. - Modifier-Tasten liefern bewusst None (sie sind kein Capture-Ziel).""" - if keysym in TK_MODIFIER_KEYSYMS: - return None - if len(keysym) == 1: - upper = keysym.upper() - if "A" <= upper <= "Z": - return 0x04 + (ord(upper) - ord("A")) - if "1" <= keysym <= "9": - return 0x1E + (ord(keysym) - ord("1")) - if keysym == "0": - return 0x27 - if keysym in _TK_NAMED_KEYSYMS: - return _TK_NAMED_KEYSYMS[keysym] - return _TK_CHAR_KEYSYMS.get(keysym) - - -def tk_event_to_hid(keysym, keycode, state, held_modifier=0): - """Ein Tk-KeyPress -> (hid_keycode, modifier_bits) oder None, wenn die - Taste sich nicht auf einen HID-Keycode abbilden laesst (dann im Dialog - einfach weiter warten statt Muell zu speichern). - - keysym/keycode/state kommen direkt aus dem Tk-Event, held_modifier ist - das vom Dialog selbst mitgefuehrte Bitfeld der aktuell gedrueckten - Modifier (siehe TK_MODIFIER_KEYSYMS) -- beides wird verodert, damit ein - verlorenes KeyRelease die Erkennung nicht verfaelscht. - - Aufloesungsreihenfolge, und warum genau so: - - 1. **Benannte Tasten ueber den keysym** (Enter, Pfeile, F-Tasten, - Numpad, Entf ...). Die sind layoutunabhaengig eindeutig -- und der - Weg ueber den Scan-Code waere hier sogar gefaehrlich: Windows liefert - fuer die Pfeiltasten denselben Scan-Code wie fuer ihre - Numpad-Zwillinge (gemessen: VK_LEFT und VK_NUMPAD4 beide 0x4B). - 2. **Zeichentasten ueber die physische Position** - (`versapad_keylayout.hid_for_vk()`, Virtual-Key -> Scan-Code -> HID). - HID-Keycodes SIND Positionen; alles, was ueber das erzeugte Zeichen - geht, ist auf nicht-US-Layouts falsch. Genau hier lag der Fehler, der - auf deutschem Layout Y und Z vertauscht hat und AeOeUe/#/+ gar nicht - erfassbar machte. Der Virtual-Key ist ausserdem unabhaengig davon, ob - Shift oder AltGr mitgehalten wird. - 3. **Naeherung ohne WinAPI** (keysym-Zeichentabelle, dann VK-Tabelle) -- - nur relevant, wenn `versapad_keylayout` nicht verfuegbar ist - (Nicht-Windows, kein ctypes). Auf dieser Ebene bleibt es bei der - US-Layout-Naeherung inklusive vertauschtem Y/Z; bei gehaltenem Shift - zaehlt dort zuerst der VK-Code, weil der keysym dann das verschobene - Zeichen ist (deutsch: Shift+7 -> "slash"). - """ - if keysym in TK_MODIFIER_KEYSYMS: - return None # Modifier sind nie das Ziel, sie setzen nur Bits - - code = _TK_NAMED_KEYSYMS.get(keysym) - if code is None: - code = kl.hid_for_vk(keycode) - if code is None: - if state & 0x0001 or held_modifier & 0x02: - code = _WIN_VK_TO_HID.get(keycode) or tk_keysym_to_hid(keysym) - else: - code = tk_keysym_to_hid(keysym) or _WIN_VK_TO_HID.get(keycode) - if code is None: - return None - - modifier = held_modifier - for bit, mod in TK_STATE_MODIFIER_BITS: - if state & bit: - modifier |= mod - return code, modifier - - ANIM_LABELS = { "Static": "● statisch", "Blink": "◎ blinkend", @@ -283,55 +105,9 @@ ANIM_LABELS = { } -_display_name_cache = None - - -def _display_key_names(): - """{hid_code: Anzeigename} -- Zeichentasten mit dem Namen des aktiven - Windows-Layouts, alles andere mit der gepflegten deutschen Bezeichnung - aus _SPECIAL_KEYS ("Enter", "Bild↑", "Num5"); die liest sich besser als - das, was Windows liefert ("EINGABE", "4 (ZEHNERTASTATUR)"). - - Einmal ermittelt und behalten -- ein Layoutwechsel zur Laufzeit wird - bewusst nicht nachgezogen (siehe versapad_keylayout.key_name()). - - Die Namen sind gleichzeitig Schluessel im Dropdown und in - hid_key_code_for_name(), muessen also eindeutig bleiben. Kollisionen - entstehen real: auf deutschem Layout heisst HID 0x31 (US-Backslash- - Position) schlicht "#" -- und diesen Namen trug bisher HID 0x32 - (Non-US-#). Der Layoutname gewinnt, der verdraengte US-Name wird - gekennzeichnet statt verworfen, damit die Taste ansprechbar bleibt.""" - global _display_name_cache - if _display_name_cache is None: - names = dict(_SPECIAL_KEYS) - layout = {} - for code in sorted(_SPECIAL_KEYS): - if code not in kl.CHARACTER_HID_CODES: - continue - name = kl.key_name(code) - if name: - layout[code] = name - names.update(layout) - claimed = set(layout.values()) - for code, name in list(names.items()): - if code not in layout and name in claimed: - names[code] = f"{name} (US-Layout)" - elif code not in layout: - claimed.add(name) - _display_name_cache = names - return _display_name_cache - - -def hid_key_name(keycode): - """Anzeigename einer Taste, layoutrichtig wo es darauf ankommt - (deutsch: 0x1C -> "Z", 0x34 -> "ä"). Ohne verfuegbare Layout-Abfrage - (Nicht-Windows) bleibt es bei der US-Naeherung aus _SPECIAL_KEYS.""" - return _display_key_names().get(keycode, f"0x{keycode:02X}") - - def hid_key_choices(): """Sortierte [(keycode, name), ...] fuer Dropdown-Auswahl beim Editieren.""" - return [(code, hid_key_name(code)) for code in sorted(_SPECIAL_KEYS)] + return sorted(_SPECIAL_KEYS.items()) def consumer_choices(): @@ -339,28 +115,15 @@ def consumer_choices(): return sorted(_CONSUMER_NAMES.items()) +_KEY_CODE_BY_NAME = {name: code for code, name in _SPECIAL_KEYS.items()} _CONSUMER_ID_BY_NAME = {name: cid for cid, name in _CONSUMER_NAMES.items()} -def _key_code_by_name(): - """Namen -> Keycode, Layoutnamen haben Vorrang vor den US-Namen. - - Beide Schreibweisen bleiben gueltig, damit aeltere Aufrufe nicht brechen. - Bei einer echten Kollision (deutsch: "Z" ist US-Position 0x1D *und* - Layoutname von 0x1C) gewinnt bewusst das Layout: wer "Z" sagt, will die - Taste, die auf dieser Tastatur ein Z tippt -- alles andere waere genau - der Fehler, der hier gerade behoben wurde.""" - by_name = {name: code for code, name in _SPECIAL_KEYS.items()} - by_name.update({name: code for code, name in _display_key_names().items()}) - return by_name - - def hid_key_code_for_name(name): """z.B. 'S' -> 0x16. Wirft ValueError mit Vorschlaegen bei unbekanntem Namen.""" - by_name = _key_code_by_name() - if name not in by_name: - raise ValueError(f"Unbekannte Taste {name!r}. Gueltige Namen: {sorted(by_name)}") - return by_name[name] + if name not in _KEY_CODE_BY_NAME: + raise ValueError(f"Unbekannte Taste {name!r}. Gueltige Namen: {sorted(_KEY_CODE_BY_NAME)}") + return _KEY_CODE_BY_NAME[name] def consumer_id_for_name(name): @@ -385,7 +148,7 @@ def macro_step_label(step): """step: {'keycode','modifier'} -> z.B. 'Strg+S'. Reine Keycode/Modifier-Variante von hid_key_label() (dort steckt beides in einem 16-Bit data-Feld, hier getrennt).""" mods = [name for bit, name in MODIFIER_BITS if step["modifier"] & bit] - key = hid_key_name(step["keycode"]) + key = _SPECIAL_KEYS.get(step["keycode"], f"0x{step['keycode']:02X}") return "+".join(mods + [key]) if mods else key @@ -395,25 +158,12 @@ def macro_slot_label(steps): return " → ".join(macro_step_label(s) for s in steps) -def macro_slot_choices(macros): - """["Slot 0 — Strg+C → Strg+V", ...] fuer die Slot-Auswahl im - Bearbeiten-Dialog: alle 32 Slots samt Inhalt auf einen Blick, statt sich - per Spinbox durch die Tabelle zu klicken.""" - return [f"Slot {slot} — {macro_slot_label(macros[slot] if slot < len(macros) else [])}" - for slot in range(MACRO_SLOTS)] - - -def macro_slot_from_choice(choice): - """Umkehrung von macro_slot_choices(): "Slot 7 — ..." -> 7.""" - return int(choice.split("—")[0].strip().split()[-1]) - - def hid_key_label(data): """data = keycode | (modifier << 8) -> z.B. 'Strg+S'""" keycode = data & 0xFF modifier = (data >> 8) & 0xFF mods = [name for bit, name in MODIFIER_BITS if modifier & bit] - key = hid_key_name(keycode) + key = _SPECIAL_KEYS.get(keycode, f"0x{keycode:02X}") return "+".join(mods + [key]) if mods else key @@ -421,15 +171,8 @@ def consumer_label(data): return _CONSUMER_NAMES.get(data, f"Consumer 0x{data:04X}") -def action_label(action, macros=None): - """Menschenlesbarer Text für eine DeviceAction {type, data}. - - macros: optional die globale 32-Slot-Makrotabelle (siehe - versapad_combined). Ist sie da, zeigt ein Makro die tatsaechliche - Tastenfolge statt nur der Slot-Nummer -- ohne die sagt "Makro (Slot 7)" - im Hauptfenster nichts darueber aus, was die Taste eigentlich tut. - Ohne macros (z.B. bei den Einzel-JSONs aus CONFIG_PATHS, die gar keine - Makro-Schritte enthalten) bleibt es beim Slot-Text.""" +def action_label(action): + """Menschenlesbarer Text für eine DeviceAction {type, data}.""" t = action.get("type") d = action.get("data", 0) if t == "None": @@ -439,8 +182,6 @@ def action_label(action, macros=None): if t == "HidConsumer": return consumer_label(d) if t == "Macro": - if macros is not None and 0 <= d < len(macros): - return f"Makro {d}: {macro_slot_label(macros[d])}" return f"Makro (Slot {d})" if t == "ProfileSwitch": if d in (0xFFFF, 0x00FF): @@ -464,24 +205,16 @@ def button_grid_position(index): return col, row -def annotate_profile(cfg, macros=None): - """Fuegt label/note/col/row-Felder hinzu (fuer die Anzeige) -- egal ob - cfg aus einer Einzel-JSON (kennt keine Notizen, faellt auf "" zurueck) - oder aus dem kombinierten Programmiermodus-State kommt. - - macros wird an action_label() durchgereicht, damit Makro-Belegungen - ihre echte Tastenfolge zeigen statt nur der Slot-Nummer.""" +def annotate_profile(cfg): + """Fuegt label/col/row-Felder hinzu (fuer die Anzeige) -- egal ob cfg aus + einer Einzel-JSON oder aus dem kombinierten Programmiermodus-State kommt.""" for b in cfg["buttons"]: - b["label"] = action_label(b["action"], macros) - b["note"] = b["action"].get("note", "") + b["label"] = action_label(b["action"]) b["col"], b["row"] = button_grid_position(b["index"]) for e in cfg["encoders"]: - e["sw_label"] = action_label(e["sw"], macros) - e["sw_note"] = e["sw"].get("note", "") - e["cw_label"] = action_label(e["cw"], macros) - e["cw_note"] = e["cw"].get("note", "") - e["ccw_label"] = action_label(e["ccw"], macros) - e["ccw_note"] = e["ccw"].get("note", "") + e["sw_label"] = action_label(e["sw"]) + e["cw_label"] = action_label(e["cw"]) + e["ccw_label"] = action_label(e["ccw"]) return cfg diff --git a/versapad_keylayout.py b/versapad_keylayout.py deleted file mode 100644 index f3ef70e..0000000 --- a/versapad_keylayout.py +++ /dev/null @@ -1,152 +0,0 @@ -""" -Abfragen ans AKTIVE Windows-Tastaturlayout: physische Tastenposition -(Scan-Code) und der Name, den diese Taste auf dem aktuellen Layout traegt. - -Warum ueberhaupt WinAPI, wo dieses Projekt sonst konsequent darauf -verzichtet: HID-Keycodes bezeichnen **physische Tastenpositionen** (das -Board sendet eine Position, erst Windows macht daraus ein Zeichen). Aus -einem Tk-Event kommen aber nur `keysym` und Virtual-Key-Code -- beides ist -bereits durch das Layout gefiltert. Auf deutschem Layout landete deshalb -jedes Y auf der Z-Taste des Boards und umgekehrt, und AeOeUe/#/+ waren gar -nicht erfassbar. Die Position ist ohne `MapVirtualKeyW` schlicht nicht zu -bekommen. - -Abgrenzung zu den Domaenenregeln in AGENTS.md: verboten sind dort -**Fenstermanipulation** (`SetWindowLongW`/`ShowWindow` aufs eigene Fenster) -und **globale Tastaturhooks** (`SetWindowsHookEx`) -- genau die Aufrufe, die -in einem anderen Projekt AV-Fehlalarme ausgeloest haben. Hier passiert -nichts davon: `MapVirtualKeyW` und `GetKeyNameTextW` sind passive, -lesende Layout-Abfragen ohne Fenster-, Prozess- oder Eingabezugriff. Die -offizielle VersaGUI (C#) benutzt `GetKeyNameText()` fuer denselben Zweck. - -Alles hier ist optional: auf Nicht-Windows oder wenn `user32` nicht laedt, -bleibt AVAILABLE False und alle Funktionen liefern None -- `versapad_data` -faellt dann auf seine US-Layout-Naeherung zurueck. -""" -import sys - -# Scan-Code (PS/2 Set 1, wie MapVirtualKeyW ihn liefert) -> HID Usage Page -# 0x07. Diese Tabelle ist layoutunabhaengig und damit der eigentliche Kern: -# eine physische Taste hat immer denselben Scan-Code und denselben HID-Code, -# egal welches Zeichen das Layout daraufschreibt. -SCANCODE_TO_HID = { - 0x01: 0x29, # Esc - 0x02: 0x1E, 0x03: 0x1F, 0x04: 0x20, 0x05: 0x21, 0x06: 0x22, - 0x07: 0x23, 0x08: 0x24, 0x09: 0x25, 0x0A: 0x26, 0x0B: 0x27, # 1-9, 0 - 0x0C: 0x2D, 0x0D: 0x2E, # US -/=, deutsch ss/Akut - 0x0E: 0x2A, 0x0F: 0x2B, # Backspace, Tab - 0x10: 0x14, 0x11: 0x1A, 0x12: 0x08, 0x13: 0x15, 0x14: 0x17, # Q W E R T - 0x15: 0x1C, 0x16: 0x18, 0x17: 0x0C, 0x18: 0x12, 0x19: 0x13, # US Y U I O P - 0x1A: 0x2F, 0x1B: 0x30, # US [/], deutsch Ue/+ - 0x1C: 0x28, # Enter - 0x1E: 0x04, 0x1F: 0x16, 0x20: 0x07, 0x21: 0x09, 0x22: 0x0A, # A S D F G - 0x23: 0x0B, 0x24: 0x0D, 0x25: 0x0E, 0x26: 0x0F, # H J K L - 0x27: 0x33, 0x28: 0x34, # US ;/', deutsch Oe/Ae - 0x29: 0x35, # US Backtick, deutsch Zirkumflex - 0x2B: 0x31, # US Backslash, deutsch # - 0x2C: 0x1D, 0x2D: 0x1B, 0x2E: 0x06, 0x2F: 0x19, 0x30: 0x05, # US Z X C V B - 0x31: 0x11, 0x32: 0x10, # N M - 0x33: 0x36, 0x34: 0x37, # Komma, Punkt (in beiden Layouts gleich) - 0x35: 0x38, # US /, deutsch - - 0x37: 0x55, # Num * - 0x39: 0x2C, # Leertaste - 0x3A: 0x39, # Caps - 0x45: 0x53, 0x46: 0x47, # NumLock, Rollen - 0x47: 0x5F, 0x48: 0x60, 0x49: 0x61, 0x4A: 0x56, # Num 7 8 9 - - 0x4B: 0x5C, 0x4C: 0x5D, 0x4D: 0x5E, 0x4E: 0x57, # Num 4 5 6 + - 0x4F: 0x59, 0x50: 0x5A, 0x51: 0x5B, # Num 1 2 3 - 0x52: 0x62, 0x53: 0x63, # Num 0 . - 0x56: 0x64, # ISO-Zusatztaste (deutsch <>|) - 0x57: 0x44, 0x58: 0x45, # F11, F12 -} -for _i in range(10): - SCANCODE_TO_HID[0x3B + _i] = 0x3A + _i # F1-F10 - -HID_TO_SCANCODE = {hid: scan for scan, hid in SCANCODE_TO_HID.items()} - -# Nur fuer diese HID-Codes lohnt der Layout-Name: es sind die Tasten, deren -# Beschriftung sich zwischen Layouts unterscheidet (Buchstaben, Ziffern, -# Satzzeichen). Fuer Enter/F5/Entf/Numpad bleiben die gepflegten deutschen -# Namen aus versapad_data._SPECIAL_KEYS besser als das, was Windows liefert -# ("EINGABE", "4 (ZEHNERTASTATUR)"). -CHARACTER_HID_CODES = ( - frozenset(range(0x04, 0x1E)) # A-Z (US-Positionen) - | frozenset(range(0x1E, 0x28)) # 1-9, 0 - | frozenset(range(0x2D, 0x39)) # Satzzeichen - | frozenset({0x64}) # ISO-Zusatztaste -) - -MAPVK_VK_TO_VSC_EX = 4 - -AVAILABLE = False -_user32 = None - -if sys.platform == "win32": - try: - import ctypes - from ctypes import wintypes - - _user32 = ctypes.WinDLL("user32", use_last_error=True) - _user32.MapVirtualKeyW.argtypes = [wintypes.UINT, wintypes.UINT] - _user32.MapVirtualKeyW.restype = wintypes.UINT - _user32.GetKeyNameTextW.argtypes = [wintypes.LONG, wintypes.LPWSTR, ctypes.c_int] - _user32.GetKeyNameTextW.restype = ctypes.c_int - AVAILABLE = True - except (ImportError, OSError, AttributeError): - _user32 = None # z.B. exotische Python-Builds ohne ctypes - - -def scancode_for_vk(vk): - """Virtual-Key-Code -> Scan-Code der physischen Taste, oder None. - - MAPVK_VK_TO_VSC_EX setzt fuer manche Tasten ein 0xE0-Praefix ins High-Byte - (erweiterte Tasten), fuer die Pfeiltasten hier aber gemessen NICHT -- die - liefern denselben Scan-Code wie ihre Numpad-Zwillinge. Genau deshalb - laeuft die Aufloesung in versapad_data zuerst ueber die eindeutigen - Tk-keysyms und benutzt diesen Weg nur fuer Zeichentasten.""" - if not AVAILABLE: - return None - scan = _user32.MapVirtualKeyW(vk, MAPVK_VK_TO_VSC_EX) - return scan or None - - -def hid_for_vk(vk): - """Virtual-Key-Code -> HID-Keycode ueber die physische Tastenposition. - Das ist der eigentliche Fix gegen vertauschtes Y/Z und nicht erfassbares - AeOeUe/#/+: der Umweg ueber das Zeichen entfaellt komplett.""" - scan = scancode_for_vk(vk) - if scan is None: - return None - if (scan >> 8) == 0xE0: - return None # erweiterte Taste -- kommt hier nicht vor, siehe oben - return SCANCODE_TO_HID.get(scan & 0xFF) - - -def key_name(hid_code): - """HID-Keycode -> Beschriftung dieser Taste auf dem aktiven Layout - (deutsch: HID 0x1C -> "Z", 0x34 -> "ä"), oder None wenn nicht ermittelbar. - - Achtung: das Ergebnis gilt fuer das Layout, das beim Aufruf aktiv ist. - Ein Layoutwechsel zur Laufzeit wird nicht bemerkt -- fuer ein Tool, das - eine Tastatur konfiguriert, ist das vertretbar (der Cache in - versapad_data haelt entsprechend auch nur eine Fassung).""" - if not AVAILABLE: - return None - scan = HID_TO_SCANCODE.get(hid_code) - if scan is None: - return None - buffer = ctypes.create_unicode_buffer(64) - # lParam-Layout von GetKeyNameTextW: Bits 16-23 Scan-Code, - # Bit 24 "erweiterte Taste". - written = _user32.GetKeyNameTextW((scan & 0xFF) << 16, buffer, 64) - if not written: - return None - name = buffer.value.strip() - if not name: - return None - # Windows liefert Mehrzeichen-Namen in Grossbuchstaben ("AKUT", - # "ZIRKUMFLEX") -- als Dropdown-Eintrag zwischen "A" und "ä" liest sich - # Titelschreibung deutlich ruhiger. - if len(name) > 1 and name == name.upper(): - return name.capitalize() - return name diff --git a/versapad_mcp_server.py b/versapad_mcp_server.py index c47ecc6..52e644f 100644 --- a/versapad_mcp_server.py +++ b/versapad_mcp_server.py @@ -61,21 +61,7 @@ def _profile_switch_data(target): def _describe_action(action): - """label zeigt bei Makros die echte Tastenfolge statt nur der - Slot-Nummer (gleiche Darstellung wie im Hauptfenster) -- die Makrotabelle - liegt im selben State, also kein Grund, hier weniger zu verraten.""" - return {"type": action["type"], "data": action["data"], - "label": vp.action_label(action, _cfg().get("macros")), - "note": action.get("note", "")} - - -def _replace_action(old_action, new_type, new_data): - """Baut eine neue Action mit neuem Typ/Daten, behaelt aber die Notiz vom - vorherigen Stand bei (rein lokal, unabhaengig davon was die Aktion tut -- - ein Tastenwechsel soll die Beschreibung 'was der Button macht' nicht - loeschen). Zum Loeschen explizit set_button_note()/set_encoder_note() - mit leerem String.""" - return {"type": new_type, "data": new_data, "note": old_action.get("note", "")} + return {"type": action["type"], "data": action["data"], "label": vp.action_label(action)} # ── Lesen ──────────────────────────────────────────────────────────────────── @@ -130,10 +116,10 @@ def get_macro(slot: int) -> dict: def get_board_status() -> dict: """Prueft per Serial, ob das Board erreichbar ist und welches Profil dort gerade aktiv ist. Schlaegt fehl/liefert busy, wenn VersaGUI oder der - Tkinter-Viewer den COM-Port gerade halten. Gibt den COM-Port danach - sofort wieder frei (kein dauerhaft offen gehaltener Serial-Handle -- - sonst blockiert dieser Prozess andere Tools/Viewer mit "busy", bis er - beendet wird).""" + Tkinter-Viewer den COM-Port gerade halten. Schliesst die Verbindung + danach wieder (siehe write_to_board() fuer den Grund) -- der Port + ist exklusiv, ein einzelner Status-Check darf ihn nicht dauerhaft + fuer Live-Sync/VersaGUI blockieren.""" try: profile = _link.read_active_profile() return {"connected": profile is not None, "active_profile": profile, "error": _link.last_error} @@ -152,7 +138,7 @@ def set_button_key(profile: int, index: int, key: str, modifiers: list[str] = [] btn = _find(_profile(profile)["buttons"], index) keycode = vp.hid_key_code_for_name(key) mod_bits = vp.modifier_bits_for_names(modifiers) - btn["action"] = _replace_action(btn["action"], "HidKey", (mod_bits << 8) | keycode) + btn["action"] = {"type": "HidKey", "data": (mod_bits << 8) | keycode} return _describe_action(btn["action"]) @@ -162,7 +148,7 @@ def set_button_consumer(profile: int, index: int, consumer: str) -> dict: 'Lauter', 'Leiser', 'Nächster Titel', 'Vorheriger Titel'.""" btn = _find(_profile(profile)["buttons"], index) cid = vp.consumer_id_for_name(consumer) - btn["action"] = _replace_action(btn["action"], "HidConsumer", cid) + btn["action"] = {"type": "HidConsumer", "data": cid} return _describe_action(btn["action"]) @@ -171,7 +157,7 @@ def set_button_macro(profile: int, index: int, slot: int) -> dict: """Belegt einen MX-Button mit einem Makro-Slot (0-31). Die Schritte selbst mit set_macro() befuellen.""" btn = _find(_profile(profile)["buttons"], index) - btn["action"] = _replace_action(btn["action"], "Macro", slot) + btn["action"] = {"type": "Macro", "data": slot} return _describe_action(btn["action"]) @@ -179,27 +165,15 @@ def set_button_macro(profile: int, index: int, slot: int) -> dict: def set_button_profile_switch(profile: int, index: int, target) -> dict: """Belegt einen MX-Button mit Profilwechsel. target: 'next' (Zyklus) oder 0/1/2.""" btn = _find(_profile(profile)["buttons"], index) - btn["action"] = _replace_action(btn["action"], "ProfileSwitch", _profile_switch_data(target)) + btn["action"] = {"type": "ProfileSwitch", "data": _profile_switch_data(target)} return _describe_action(btn["action"]) @mcp.tool() def set_button_none(profile: int, index: int) -> dict: - """Entfernt die Belegung eines MX-Buttons (Action = None). Notiz bleibt - erhalten -- zum Loeschen set_button_note(profile, index, "").""" + """Entfernt die Belegung eines MX-Buttons (Action = None).""" btn = _find(_profile(profile)["buttons"], index) - btn["action"] = _replace_action(btn["action"], "None", 0) - return _describe_action(btn["action"]) - - -@mcp.tool() -def set_button_note(profile: int, index: int, note: str) -> dict: - """Setzt/aendert die freie Notiz eines MX-Buttons -- was der Button tut, - unabhaengig von der technischen Aktion (z.B. 'Speichern in Fusion 360'). - Rein lokal, landet nie aufs Board (wie Profilnamen). Leerer String - loescht die Notiz.""" - btn = _find(_profile(profile)["buttons"], index) - btn["action"]["note"] = note + btn["action"] = {"type": "None", "data": 0} return _describe_action(btn["action"]) @@ -231,7 +205,7 @@ def set_encoder_key(profile: int, index: int, field: str, key: str, modifiers: l enc, f = _encoder_field(profile, index, field) keycode = vp.hid_key_code_for_name(key) mod_bits = vp.modifier_bits_for_names(modifiers) - enc[f] = _replace_action(enc[f], "HidKey", (mod_bits << 8) | keycode) + enc[f] = {"type": "HidKey", "data": (mod_bits << 8) | keycode} return _describe_action(enc[f]) @@ -239,7 +213,7 @@ def set_encoder_key(profile: int, index: int, field: str, key: str, modifiers: l def set_encoder_consumer(profile: int, index: int, field: str, consumer: str) -> dict: """Belegt eine Encoder-Aktion mit einer Medientaste.""" enc, f = _encoder_field(profile, index, field) - enc[f] = _replace_action(enc[f], "HidConsumer", vp.consumer_id_for_name(consumer)) + enc[f] = {"type": "HidConsumer", "data": vp.consumer_id_for_name(consumer)} return _describe_action(enc[f]) @@ -247,7 +221,7 @@ def set_encoder_consumer(profile: int, index: int, field: str, consumer: str) -> def set_encoder_macro(profile: int, index: int, field: str, slot: int) -> dict: """Belegt eine Encoder-Aktion mit einem Makro-Slot (0-31).""" enc, f = _encoder_field(profile, index, field) - enc[f] = _replace_action(enc[f], "Macro", slot) + enc[f] = {"type": "Macro", "data": slot} return _describe_action(enc[f]) @@ -257,26 +231,15 @@ def set_encoder_profile_switch(profile: int, index: int, field: str, target) -> Achtung: Encoder 0 'sw' ist normalerweise auf allen 3 Profilen der Profilwechsel -- nicht ohne Ruecksprache mit dem User aendern.""" enc, f = _encoder_field(profile, index, field) - enc[f] = _replace_action(enc[f], "ProfileSwitch", _profile_switch_data(target)) + enc[f] = {"type": "ProfileSwitch", "data": _profile_switch_data(target)} return _describe_action(enc[f]) @mcp.tool() def set_encoder_none(profile: int, index: int, field: str) -> dict: - """Entfernt eine Encoder-Belegung (Action = None). Notiz bleibt erhalten - -- zum Loeschen set_encoder_note(profile, index, field, "").""" + """Entfernt eine Encoder-Belegung (Action = None).""" enc, f = _encoder_field(profile, index, field) - enc[f] = _replace_action(enc[f], "None", 0) - return _describe_action(enc[f]) - - -@mcp.tool() -def set_encoder_note(profile: int, index: int, field: str, note: str) -> dict: - """Setzt/aendert die freie Notiz einer Encoder-Aktion (sw/cw/ccw) -- was - sie tut, unabhaengig von der technischen Aktion. Rein lokal, landet nie - aufs Board. Leerer String loescht die Notiz.""" - enc, f = _encoder_field(profile, index, field) - enc[f]["note"] = note + enc[f] = {"type": "None", "data": 0} return _describe_action(enc[f]) @@ -340,10 +303,10 @@ def load_local(path: str = None) -> dict: @mcp.tool() def load_from_board() -> dict: """Liest die komplette Config + Makros vom Board (per Serial, ~1-2s) und - ersetzt damit den In-Memory-State. Profilnamen UND Notizen bleiben - erhalten (kennt nur wir, nicht das Board). Schlaegt fehl, wenn der - COM-Port gerade von VersaGUI/dem Tkinter-Viewer gehalten wird. Gibt den - COM-Port danach sofort wieder frei (siehe get_board_status()).""" + ersetzt damit den In-Memory-State. Profilnamen bleiben erhalten (die + kennt nur wir, nicht das Board). Schlaegt fehl, wenn der COM-Port gerade + von VersaGUI/dem Tkinter-Viewer gehalten wird. Schliesst die Verbindung + danach wieder (siehe write_to_board() fuer den Grund).""" try: raw_cfg = _link.read_full_config() if raw_cfg is None: @@ -358,9 +321,7 @@ def load_from_board() -> dict: macro_slots = vproto.unpack_macros(raw_macros) names = _cfg()["profile_names"] - previous = _state["combined"] - _state["combined"] = vcomb.merge_notes( - vcomb.from_binary(cfg_dict, macro_slots, profile_names=names), previous) + _state["combined"] = vcomb.from_binary(cfg_dict, macro_slots, profile_names=names) return list_profiles() finally: _link.close() @@ -372,7 +333,11 @@ def write_to_board() -> dict: Board -- ueberschreibt, was dort aktuell im NVM steht. Firmware prueft Magic/CRC/Keycode-Bereich vor jedem Schreiben und antwortet sonst nur mit NACK (kein Risiko fuer Datenmuell). Schlaegt fehl bei belegtem COM-Port. - Gibt den COM-Port danach sofort wieder frei (siehe get_board_status()).""" + Schliesst die Verbindung danach wieder -- VersaPadLink haelt den Port + sonst dauerhaft offen (kein automatisches Schliessen nach einem Befehl), + was Live-Sync/VersaGUI/den naechsten MCP-Aufruf sonst dauerhaft mit + "busy" blockieren wuerde, obwohl der eigentliche Vorgang laengst fertig + ist -- der Port ist exklusiv (siehe versapad_serial.py).""" try: cfg_bytes, macro_bytes = vcomb.to_binary(_cfg()) if not _link.write_full_config(cfg_bytes):