Repository navigation
fix(browser): nicht erreichbares Laufwerk zeigt leere Seite (1.0.9) - #24
Merged
Merged
Conversation
…lder; re-check known network drives Pending/unreachable/timeout targets hide the table (empty page) with a centered status text; history/back keep working and a click re-checks. Known paths on removable/network drives show the kept state and are re-validated in the background; failure drops them and shows the empty page. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01LYHP4Js5ggpzQ78JXWRe5E
|
Welcome! Thanks for your first pull request in this repository. A maintainer will review it soon. Please make sure:
Thanks for contributing. |
Collaborator
Author
|
Review (Zwei-Modell-Regel): Claude Opus 5.5 als Operator. Leere Seite für unbestätigte Ziele (Tabelle ausgeblendet, Status zentriert, nie über Einträgen), einheitlicher Fehler-/Timeout-/Nachprüfungspfad über _show_unreachable mit _forget, Zurück/Vor über _switch, Nachprüfung bekannter Netz-/Wechsellaufwerke. Tests test_drive_switch grün (auf master 4 failed/1 error), CI 14/14 grün. Echter Test mit nicht erreichbarem Laufwerk E: steht beim Nutzer aus. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Nutzertest
Laufwerk E: nicht erreichbar: beim Wechsel blieb der vorige Ordner stehen, ein Hinweis überlagerte "wird geladen".
Ursache
Fehler/Timeout blendeten nur den Hinweis aus und riefen
_gonicht auf (alte Ansicht blieb);_busywar ein Overlay über der Tabelle.Änderung
drive_kind >= 1): vorgehaltener Stand sofort, Nachprüfung im Hintergrund; bei Fehler/Timeout Pfad (und Unterpfade) aus_known_dirsentfernt + leere Seite.Tests
tests/test_drive_switch.py+5 (unerreichbar via None/Fehler, Zurück+Wiederholung, Timeout, bekannter Netzpfad fällt weg), 2 ältere Tests an die neue Semantik angepasst. Rot auf master 483f3f4 (4 failed + 1 error), grün auf Branch. Suite in venvexplorerpro-store(-n 2): vorher (PR #23) 1 failed/566 passed; nachher 3 failed/569 passed: drive_query_process (host-bedingt wie bisher), test_gui_gc (Subprozess-Timeout unter Last) und test_store_regressions-Navigation (Ladezeit unter Last) – beide einzeln 17/17 grün.