fix(web): Toast, Karten-Überlauf, Versionsnummer, Footer und Verbindungstest - #70
Merged
Conversation
Field report from the running appliance: cards overlapped, the test button gave no feedback, the version showed 0.0.0 and the footer was missing. - toast: the redesigned stylesheet had no .toast rule, so every notification rendered as unstyled text below the fold and no action ever appeared to do anything. Restore it as a fixed, elevated, tone-coloured toast above the bottom nav. - resource cards: a long scanner URI blew up the implicit grid track and pushed the card ~140px over its container into the neighbouring column. Pin the track to minmax(0,1fr) and add the missing min-width:0 guards on the flex/grid children that carry truncated text. - connection test: the card action is now "Verbindung prüfen" (so "Testen" is not on screen twice), it locks while running and leaves the result in the card instead of only in a toast. Same for scanners. Target form errors are shown inline as well. - version: in the image app/ is copied to /app, so the lookup landed on /VERSION and always failed. Copy VERSION into the image and resolve it next to the app, at the repo root, or from SCAN2TARGET_VERSION. - settings: add the auto-refresh switch, explain the sign-in card when no account form is shown, and name the Home Assistant endpoint and its method (a browser GET answers 405). - footer: version, source and documentation links plus live/polling state under every view. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01PnunQ6KenoLUPCZRxjvhDd
fgrfn
marked this pull request as ready for review
September 8, 2026 01:52
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
2 participants
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.
Behebt die Punkte aus dem Praxistest auf der laufenden Appliance.
Anzeige überlappt (Verwalten → Scanner)
Die lange Scanner-URI (
airscan:escl:HP_ENVY_6400_series:http://…/eSCL/) hat die implizite Grid-Spur von.resource-gridauf ihre Min-Content-Breite aufgeblasen. Im Browser nachgemessen: die Karte ragte 140 px über ihren Container in die rechte Spalte..resource-gridaufminmax(0,1fr)festgenageltmin-width:0-Guards auf den Flex-/Grid-Kindern ergänzt, die abgeschnittenen Text tragen (.resource-title>div,.list-row>div,.history-copy)Nachmessung nach dem Fix: −21 px, die Karte liegt wieder innerhalb des Containers.
Test-Button zeigt kein Ergebnis
Ursache: das neue Stylesheet enthielt überhaupt keine
.toast-Regel. Die Benachrichtigung wurde als unformatierter Text unterhalb des sichtbaren Bereichs gerendert (position: static, kein z-index) — jede Aktion sah folgenlos aus. Das erklärt auch die mehrfachenPOST /targets/Unraid/testim Log.Zwei „Testen"-Buttons auf Ziele
Die Karten-Aktion heißt jetzt „Verbindung prüfen" (
checkConnection), im Formular bleibt „Testen & speichern" / „Ohne Test speichern". Damit steht „Testen" nur noch an einer Stelle. Gleiche Benennung bei den Scannern.Versionsnummer 0.0.0
Im Image wird
app/nach/appkopiert,get_version()suchte deshalb/VERSION— die Datei existiert dort nie (Could not read VERSION file (/VERSION)im Log).COPY VERSION ./VERSIONim Backend-Stageget_version()prüftSCAN2TARGET_VERSION, dann/app/VERSION, dann die Repo-WurzelEinstellungen unvollständig / Footer fehlt
POST /api/v1/homeassistant/scan— ein Aufruf im Browser (GET) antwortet mit 405Verifikation
npm run build✓,npm run check0/0,npm run test:e2e1/1pytest tests/25/25Offen
GET /api/v1/homeassistant/scanbleibt bewusst 405. Eine GET-Variante wäre bequem für Taster, ließe sich aber von jeder Webseite per'<img src=…>'auslösen — der Endpunkt ist standardmäßig ungeschützt. Sag Bescheid, wenn du sie trotzdem willst (dann am besten nur mit gesetztemSCAN2TARGET_HA_API_KEY).🤖 Generated with Claude Code
https://claude.ai/code/session_01PnunQ6KenoLUPCZRxjvhDd
Generated by Claude Code