Skip to content

Fix: Cloud-Ordner, Vorschau-Absturz, Black-/Whitelist-GUI und Spracheinstellung - #18

Merged
lukisch merged 5 commits into
masterfrom
claude/laughing-brahmagupta-cwl014
Oct 6, 2026
Merged

lukisch merged 5 commits into
masterfrom
claude/laughing-brahmagupta-cwl014

Conversation

@lukisch

@lukisch lukisch commented Oct 6, 2026

Copy link
Copy Markdown
Collaborator

Zusammenfassung

Review mit Fehlerbehebung für die gemeldeten Probleme:

Problem Ursache Fix
Keine Cloud-Ordner (OneDrive) sichtbar Ohne QDir.System blendet Qt unter Windows Cloud-Platzhalter (Reparse-Points) und .lnk aus Filter erweitert; neue Seitenleisten-Gruppe Cloud-Speicher (Env-Variablen, Benutzerordner, macOS CloudStorage, Windows-SyncRootManager nur lesend)
Oft nur „Ordner“ statt Ordnername Typ-/Datumsspalten mit ResizeToContents drücken die Namensspalte zusammen (z. B. „Komprimierter (gezippter) Ordner“) Feste, verstellbare Spaltenbreiten, Name mindestens 160 px, mittiges Kürzen
Absturz/Hänger bei Klick auf Datei Vorschau las Cloud-Platzhalter (Download im GUI-Thread), lud Bilder/PDFs ungebremst, startete PowerShell für .lnk, zählte Ordner synchron Vorschau fängt alle Fehler ab, überspringt Platzhalter, beachtet Größenlimit/Schalter, skaliert Bilder/PDFs, nativer .lnk-Parser, Zählung über das Modell; crash.log per faulthandler
Black-/Whitelist nicht in der GUI änderbar, Ampel ohne Funktion Kein Editor; Ampel-Klick war ein TODO; Monitor ließ sich nach stop() nicht wieder starten Neuer Datenschutz-Dialog (Listen bearbeiten, filtern, Import/Export TXT/CSV/XLSX, Testbereich), Ampel: Klick → Dialog, Rechtsklick → Schnellmenü
Spracheinstellung ignoriert Großteil der GUI-Texte fest deutsch; t() schrieb zur Laufzeit in den (im Store-Paket schreibgeschützten) Katalog Laufzeit-UI-Übersetzer mit Live-Umschaltung, f-Strings → t()-Vorlagen, Katalog um ~670 Texte in allen 6 Sprachen ergänzt, keine Schreibzugriffe zur Laufzeit

Zusätzlich werden bisher wirkungslose Einstellungen angewendet: Startordner, versteckte Dateien, Farbschema, Schriftgröße, Symbolgröße, Vorschau-Optionen, Überwachung/Hinweise, „Fenstergröße merken“. Der Indexer beachtet maximale Dateigröße und Ausschlussmuster und lädt keine Cloud-Platzhalter herunter.

Tests

  • python -m pytest -q: 527 passed, 10 skipped (5 Läufe, stabil)
  • python manage_translations.py --check: alle 1050 Einträge in 6 Sprachen vollständig
  • python -m compileall -q src tests: ok
  • Neue Regressionstests: tests/test_review_fixes_20261006.py (26), .lnk-Parser in tests/test_shortcut_preview.py
  • Headless-Screenshots auf EN/JA: keine deutschen Restetexte in Hauptfenster, Seitenleisten, Einstellungs- und Datenschutzdialog

Hinweise

  • Windows-spezifisches Verhalten (OneDrive-Platzhalter, SyncRootManager, .lnk) ist per Unit-Tests mit simulierten Attributen abgedeckt, aber nicht auf echtem Windows ausgeführt.
  • „Ordner automatisch indizieren“ und „Beim Programmstart indizieren“ sind weiterhin ohne Wirkung (nicht Teil dieses PRs).

🤖 Generated with Claude Code

https://claude.ai/code/session_01C8yXpLN2HtYBwKbfxo12Bo


Generated by Claude Code

claude added 4 commits October 6, 2026 04:13
…Übersetzung (WIP)

- Dateiliste zeigt OneDrive-/Cloud-Platzhalter und .lnk (QDir.System unter Windows)
- Namensspalte wird nicht mehr von Typ/Datum zusammengedrückt
- Cloud-Speicher (OneDrive, Dropbox, iCloud, ...) in der Seitenleiste
- Vorschau gehärtet: keine Downloads von Platzhaltern, Größenlimit,
  skaliertes Bilddekodieren, nativer .lnk-Parser statt PowerShell
- Datenschutz-Dialog mit editierbarer Black-/Whitelist, Import/Export,
  Testbereich; Ampel per Klick/Rechtsklick bedienbar
- Laufzeit-UI-Übersetzer mit Live-Sprachwechsel; Katalog wird zur
  Laufzeit nicht mehr geschrieben
- Einstellungen (Startordner, Theme, Schrift, Symbolgröße, Vorschau,
  Überwachung) werden tatsächlich angewendet
- Crash-Log via faulthandler

Co-Authored-By: Claude Opus 5.5 <[email protected]>
Claude-Session: https://claude.ai/code/session_01C8yXpLN2HtYBwKbfxo12Bo
… Schlüssel auf

- Fehlende GUI-Texte (DE, EN, ES, ZH, JA, RU) ergänzt; manage_translations --check ist grün
- t() übersetzt auch Varianten mit Emoji/Doppelpunkt/Auslassungspunkten
- NO_TRANSLATE schützt nur den Inhalt, Tooltips/Barrierefreiheit werden übersetzt
- UI-Übersetzer wird vor dem Abbau der QApplication deinstalliert

Co-Authored-By: Claude Opus 5.5 <[email protected]>
Claude-Session: https://claude.ai/code/session_01C8yXpLN2HtYBwKbfxo12Bo
@github-actions

github-actions Bot commented Oct 6, 2026

Copy link
Copy Markdown

Welcome! Thanks for your first pull request in this repository.

A maintainer will review it soon. Please make sure:

  • Your changes are tested
  • Documentation is updated if needed
  • The PR description explains what changed and why

Thanks for contributing.

@lukisch
lukisch merged commit 7311397 into master Oct 6, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants