EU-Data-Act-Reste bereinigt, Slider-Bug behoben, zwei Design-Audit-Runden umgesetzt
Entfernt die letzten Verweise auf die abgeloeste TommiG1/HA_VAG-EU-Data-Act- Integration (Versionstile, Testumgebungs-Fixture) und ersetzt sie durch die FMM003-Werte. Behebt eine CSS-Spezifitaetskollision, durch die alle iOS- Toggle-Switches viel zu breit gerendert wurden. Dokumentiert und behebt sieben Layout-Befunde aus zwei Design-Audit-Runden (Zahnrad-Ueberlauf auf breiten Bildschirmen, zu kleine Ringe als mobiler Einstellungen-Zugang, zu breite Popups, Kopfzeilen-Versatz, ueberlappende Kachel-Pfeile, fehlender Editierbarkeits-Hinweis bei Auswahlfeldern, Haekchen-artige Auf/Zu-Pfeile) - siehe DESIGN_AUDIT_2026-08-13.md fuer Root-Cause und Beleg je Befund. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
This commit is contained in:
@@ -1,7 +1,10 @@
|
||||
# AGENTS.md — Project state, review findings, open items, and working rules
|
||||
|
||||
**Last updated: 2026-08-13** (merged the `umsetzung-datametric360` branch — companion app phases
|
||||
1–10 done, see `UMSETZUNGSPLAN.md`). This file is the entry point for every new agent
|
||||
1–10 done, see `UMSETZUNGSPLAN.md`; later the same day: cleaned up remaining EU Data Act residue,
|
||||
fixed oversized toggle switches, and fixed a desktop-layout audit (settings button / rings logo /
|
||||
popups overflowing past the capped content column) — see `DESIGN_AUDIT_2026-08-13.md` and section
|
||||
C). This file is the entry point for every new agent
|
||||
session: what this repo is, what is finished, what is missing, and how to work here. Detail lives in
|
||||
the linked documents — this file points, it does not duplicate.
|
||||
|
||||
@@ -528,6 +531,76 @@ wraps the web app for iPhone; a PWA home-screen install is the accepted intermed
|
||||
Abbrechen/Speichern and the "Einrichten"/"Setup"/"Fertig" buttons in Einstellungen render with
|
||||
the corrected fills at both mobile and desktop widths, switches render green, no console
|
||||
errors.
|
||||
- [x] Clean up remaining "EU Data Act" residue from the FMM003 pivot (2026-08-13) — the
|
||||
Einstellungen "Version" tile in `audi-dashboard-app.js` hardcoded `Fahrzeugdaten: EU Data
|
||||
Act / Abruf alle 15 Minuten` and `Position: iPhone / Companion App`, both stale: vehicle data
|
||||
and position now come exclusively from the FMM003 (`ZUENDUNG_SENSOR`, `BATTERIE_SENSOR`,
|
||||
`STANDORT_TRACKER` in `einstellungen.py`), not the abandoned HACS integration or the iPhone.
|
||||
Both rows now read `FMM003`. Also fixed `testumgebung/konfiguration.yaml` +
|
||||
`testumgebung/README.md`: the `input_text.test_wlan` / `iPhone WiFi Connection` template
|
||||
sensor simulated WLAN-based trip detection, fully removed 2026-08-12 (`fahrzeug.wlan_ssid` no
|
||||
longer exists anywhere) — its "Fahrt auslösen" example was non-functional. Replaced with
|
||||
`input_boolean.test_zuendung` driving a `binary_sensor.testzone_fmm003_engine_ignition_or_acc_status`
|
||||
template sensor matching the real `ZUENDUNG_SENSOR` entity ID, so trip detection stays
|
||||
testable. The remaining KM/Tank/Range/door/window/lock template sensors correctly still
|
||||
describe the abandoned `TommiG1/HA_VAG-EU-Data-Act` integration (per `einstellungen.py`) and
|
||||
were left in place — those fields are blank by default but stay manually mappable via the
|
||||
Setup menu, so the fixtures remain useful for testing that path; comments updated to make
|
||||
this distinction explicit.
|
||||
- [x] Fix oversized toggle switches ("Slider") across the app (2026-08-13) — `.feld label` in
|
||||
`audi-dashboard.css` (`flex: 1 1 auto`, meant to let the *text* label grow so the control
|
||||
lands at the row's right edge) unintentionally also matched `<label class="switch">`, since
|
||||
the toggle switch is itself implemented as a `<label>` — its own `.switch { flex-shrink: 0;
|
||||
width: 46px }` rule lost because `.feld label` has equal-or-higher specificity and comes
|
||||
later in the file. Every switch inside a `.feld` (Einstellungen, Tanken, Reifen, ...)
|
||||
stretched to fill the row (~260-300px measured live, instead of 46-51px). Fixed by excluding
|
||||
switches from the selector: `.feld label:not(.switch)`. Verified live in `audi_ha_test` at
|
||||
mobile (375px) and desktop widths — switches render at the correct ~51px iOS size, no
|
||||
console errors.
|
||||
- [x] Fix desktop (≥860px container) layout audit findings (2026-08-13, see
|
||||
`DESIGN_AUDIT_2026-08-13.md` for full detail) — user-reported: (1) the settings gear button
|
||||
sat far past the right edge of the visible content column on wide screens because `.topbar`
|
||||
(grid-column:2, same track as `main#view`) had no `max-width:860px` while `main#view` did —
|
||||
the track itself is `minmax(0,1fr)` and stretches to the full remaining window width, so
|
||||
`.head{flex:1}` pushed `.profilbtn` to the track's far edge, ~450px past where `main#view`
|
||||
visually ends. Fixed by giving `.topbar`/`.phone:has(.back.on) .topbar`/`.ptr` the same
|
||||
`max-width:860px`. (2) In phone width the Audi rings (`#marke`/`.rings`) are, per existing
|
||||
code comment, the *only* access to Einstellungen (the gear icon is hidden there) but were
|
||||
42×24px — well under the 44px minimum touch target every sibling icon-button uses. Bumped to
|
||||
58×32px. (3) Found while verifying (1): `.setup-popup`, `.sdpopup`, `.sheet`, and
|
||||
`.standortmenu` are all `position:absolute` with `left`/`right` relative to the *full*
|
||||
`.phone` box (sidebar included), not the content column, and had no desktop max-width either
|
||||
— `.setup-popup` measured 1644px wide on a 1920px window (10px margins only). Fixed with
|
||||
`max-width:560px` (`.setup-popup`/`.sdpopup`/`.sheet`) or `640px` (`.standortmenu`) plus
|
||||
`margin-left/right:auto` in the same container query — deliberately not
|
||||
`transform:translateX(-50%)`, which would fight these elements' own transform-based entry
|
||||
animations. Centering is relative to the full `.phone` width (sidebar included), not the
|
||||
860px content column exactly — a ~138px rightward offset from true content-center remains at
|
||||
1920px; fixing that precisely would require moving these overlays into grid-column 2 in the
|
||||
markup, deferred as a larger structural change. All three verified live in `audi_ha_test` at
|
||||
375px and 1920px, no console errors.
|
||||
- [x] Fix 4 more user-reported design findings (2026-08-13, same session, see
|
||||
`DESIGN_AUDIT_2026-08-13.md` "Runde 2" for full detail with before/after measurements):
|
||||
(1) page title sat ~30px right of the content tiles below it on wide screens — `.back`
|
||||
reserves 44px+gap in `.topbar` even when inactive (`opacity:0`, not `display:none`), fixed
|
||||
with `.back:not(.on){width:0;...}` in the desktop container query, root pages only (detail
|
||||
pages with `.back.on` unaffected, verified back-navigation still works). (2) the chevron on
|
||||
the 5 `.go`-tiles (Fahrzeug/Service/Versicherung/Reifen/Schutzbrief on Mein Audi/Versicherung)
|
||||
was vertically centered on the *entire* multi-row tile (`top:50%` in the iOS overlay) instead
|
||||
of next to the title, overlapping wrapped row text — removed the override, falls back to the
|
||||
base `top:var(--sp-5)` next-to-title position. (3) the Ölwechsel-Intervall `<select>` values
|
||||
("10.000 km"/"1 Jahr") looked like inert text — iOS overlay strips border/background from
|
||||
`.feld select` and colors it the same grey as any other secondary text; added
|
||||
`.feld select{color:var(--ios-tint)}` so picker fields read as editable the way iOS
|
||||
conventionally signals it (accent color), free-text `.feld input`s deliberately left alone.
|
||||
(4) the expand/collapse `.mark` chevron on group headers (Fahrten "2026"/"August") looked
|
||||
like a checkmark — it uses the classic "2 borders + rotate(45deg)" CSS chevron technique,
|
||||
which requires a square box; `.mark` was 6×10px (non-square, sized for the unrelated SVG-path
|
||||
`.chev`/`.go` icons), made it 8×8px. Two more reported findings ("grey box around all menu
|
||||
icons", "red partial frame on Fahrten/Tanken rows") were investigated (computed-style
|
||||
diffing across all 5 tabs in both themes, a temporary 2.5x `transform:scale()` zoom on the
|
||||
live swipe-row DOM) but could not be reproduced — documented in the audit file as open,
|
||||
pending a screenshot from the real device.
|
||||
- [ ] Fix remaining documentation drift (statistics claim, README gaps, obsolete TODO comment) —
|
||||
text-only changes; INSTALL.md's WLAN/TommiG1 drift and stale variable names were fixed
|
||||
2026-08-12 (see section B); `DESIGN_REVIEW_2026-08-13.md` and `REVIEW_main_2026-08-13.md`
|
||||
|
||||
Reference in New Issue
Block a user