Files
rar-autopass/docs/raport-comparatie-mockup-5.16.md
Claude Agent 8f39dfbc1e feat(5.16): aliniere lista/preview la mockup + fix lock seed la boot
Implementeaza planul aprobat din docs/raport-comparatie-mockup-5.16.md (T-1..T-9):

- T-1/T-8: rand lista 4->2 linii (placuta primar + cod RAR · operatie · data + pill),
  fallback placuta, eticheta-problema 10px->--fs-xs (_submissions.html, base.html)
- T-2: pill slim restilat fill-tint + dot 7px + text colorat per stare (base.html)
- T-3: bug 4a coliziune pill/vehicul in preview — col-stare 104->140px (base.html)
- T-4: preview 8->5 coloane (scos #, KM, Note; motivul -> title pe pill)
- T-5: titlu sectiune "Trimiterile tale" -> sr-only (a11y) + badge/export discret
- T-6: linia plan N/60 in corp doar pe avertizare; consum normal in badge+burger
- T-7: guard chenar gol chips extra (_chips_prestatii.html)
- T-9: "Anuleaza"->"Renunta"; nume operatie emfatic bold

Fix boot: init_db reincarca seedul de ~17k operatii (5.18) pe FIECARE pornire, pe
API + worker concurent -> "database is locked" la al doilea proces. Guard "_if_empty"
pe mapping_suggestions (ca seed_nomenclator_if_empty) -> boot rapid, fara cursa.

Teste actualizate (slim 2-linii, fallback placuta, plan in burger). TODOS.md:
defer trackuit (eroare HTMX lista, retokenizare px, diacritice).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
2026-06-29 14:44:10 +00:00

535 lines
34 KiB
Markdown
Raw Permalink Blame History

This file contains ambiguous Unicode characters

This file contains Unicode characters that might be confused with other characters. If you think that this is intentional, you can safely ignore this warning. Use the Escape button to reveal them.

<!-- /autoplan restore point: /home/claude/.gstack/projects/romfast-rar-autopass/feat-5.18-corpus-knn-exemple-etichetate-autoplan-restore-20260629-070833.md -->
# Raport comparatie UI real vs. mockup-uri (PRD 5.16 + 5.17)
**Data**: 2026-06-29
**Metoda**: comparatie in browser (Playwright, 1280px + 390px) intre aplicatia live
(`http://localhost:8010`, cont 2 "Romfast SRL", 34 trimiteri) si mockup-urile de
referinta din `docs/mockups/`. Pentru fiecare pagina/formular am pus fata in fata
implementarea reala si intentia de design, apoi am evaluat in spiritul PRD-urilor.
> Concluzie pe scurt: **antetul, /login, selectorul de tema, contoarele si modalele
> sunt conforme**. Abaterea dominanta este **densitatea informationala**: lista de
> trimiteri si tabelul de preview din import afiseaza mult mai multa informatie pe rand
> decat mockup-ul minimalist — exact observatia userului ("randurile foarte late").
> Plus un **bug de layout** (coliziune coloane in preview-ul de import) si cateva
> abateri minore de copy/stil.
---
## 1. Lista de trimiteri — rand cu 4 linii in loc de 2 (PRIORITATE INALTA)
**Aceasta e problema semnalata de user.**
| | Mockup (`prd-5.16-dashboard.html`) | Real (`_submissions.html:100-139`) |
|---|---|---|
| Linii / rand | **2**: VIN + `operatie · ora` | **4**: VIN; `operatie · data+ora+secunde`; cod RAR; `nr · data · #id` |
| Pastila de stare | DOAR pe exceptii (In coada / De corectat / Trimis); finalizatele **nu au pastila** | **pe fiecare rand**, inclusiv "Finalizat" |
| Marca de timp | ora scurta (`09:42`) | datetime complet cu secunde (`27.06.2026 22:25:52`) |
| Inaltime efectiva | ~2 randuri text | ~2x mai mare; pe mobil un rand se desfasoara pe 5-6 linii |
Cauza in cod (`app/web/templates/_submissions.html`):
- **Linia 3** — codul RAR (`OE-8`) / "nemapat": liniile 113-119.
- **Linia 4** — `vehicul_nr · data_prestatie · #id_prezentare`: liniile 121-127.
- **Marca de timp** foloseste `r.updated_at` complet (data+ora+secunde): linia 111
(mockup-ul foloseste ora scurta).
- **Pastila mereu randata** cu `r.stare_scurt`: liniile 137-139 (mockup-ul ascunde
pastila pe starea implicita/finalizata — minimalism "linistit cand e ok, zgomotos
cand e exceptie", in spiritul D6/zero-silent-failures).
**Recomandari** (in ordinea impactului):
1. **Comprima la 2 linii pe starea normala**: pastreaza linia 1 (VIN) + linia 2
(`operatie · data`). Muta cod RAR, nr. inmatriculare si `#id_prezentare` in modalul
de detaliu (care le are deja — vezi sectiunea 5) sau intr-un al doilea rand afisat
doar la hover/expand. Informatia completa nu trebuie sa coabiteze pe rand cu lista.
2. **Ascunde pastila pe starea finalizata** (afiseaz-o doar pe `queued/sending/
needs_*/error`), exact ca mockup-ul. Finalizat = implicit linistit.
3. **Scurteaza marca de timp**: data fara secunde (`27.06.2026`) sau `data · ora`
fara secunde. Secundele sunt zgomot.
4. Daca cod RAR / nr. inmatriculare sunt considerate esentiale in lista, fa-le optional
(toggle "afiseaza detalii") in loc sa fie mereu prezente — implicit colapsat.
5. Minor: `eticheta-problema` are `font-size:10px` (`_submissions.html:133`) — sub
pragul de 12px din scala 5.16/US-002; recableaza pe `--fs-xs`.
---
## 2. Acasa — titlu de sectiune + toolbar mult mai greu decat mockup-ul (PRIORITATE MEDIE)
PRD 5.16/US-002 cere explicit: *"Se ELIMINA titlul de sectiune ... lista incepe direct
sub tab-uri/filtre"* si *"fara subtitlu de sectiune"*. In real:
- **Titlul "Trimiterile tale" (h2) + link-urile "export CSV: trimise | toate"** sunt inca
prezente ca antet de sectiune deasupra listei. Mockup-ul nu are titlu de sectiune —
lista porneste direct sub tab-uri.
- **Toolbar-ul de filtre e mult mai dens** decat mockup-ul. Mockup: 4 pastile simple de
stare (`Toate / In coada / Trimise / De corectat`). Real: pastile de timp
(`Azi / 7 zile / 30 zile / Custom`) + camp cautare `Vehicul (nr/VIN)` + butoane
`Filtreaza` + `Toate` + un AL DOILEA rand de actiuni bulk (`Cod RAR ... / Aplica cod
/ Sterge selectate`). Sunt functii reale, dar contrazic intentia minimalista.
**Recomandari**:
1. Elimina antetul "Trimiterile tale" (sau redu-l la un label discret); muta link-urile
de export CSV langa tab-uri sau in meniul de cont.
2. Pastreaza filtrele de timp + cautarea (sunt utile), dar **colapseaza randul de actiuni
bulk** (Cod RAR / Aplica cod / Sterge selectate) intr-un buton "Actiuni" care se
deschide doar cand exista selectie — azi ocupa un rand permanent.
3. Aliniaza pastilele de stare cu mockup-ul (stari, nu doar timp), eventual ambele
grupuri pe acelasi rand.
---
## 3. Linia "Plan: Gratuit · 34/60 luna asta" reintroduce un meta-rand sub tab-uri (PRIORITATE MEDIE)
PRD 5.17/US-006 + 5.16 cer planul ca **badge in antet** (exista — "GRATUIT") si **linie
in meniul burger**, NU ca rand in corpul paginii. Real afiseaza consumul si ca **rand
standalone sub tab-uri**, pe FIECARE tab (Acasa, Mapari, Integrare). Asta:
- duplica informatia din antet, si
- recreeaza exact "meta-randul de sectiune" pe care 5.16/US-002 voia sa-l elimine.
**Recomandare**: muta `N/60 luna asta` in meniul burger / pagina Cont (cum cere PRD-ul);
pastreaza in antet doar badge-ul de plan. Daca avertizarea de consum (>=80%) trebuie sa
fie vizibila in corp, afiseaz-o **doar** in starea de avertizare, nu permanent.
---
## 4. Import — preview pas 3: coliziune de coloane + tabel mai greu decat mockup-ul
### 4a. BUG layout — pastila STARE se suprapune peste coloana VEHICUL (PRIORITATE INALTA)
In tabelul de preview (pas Verifica), pastila de stare ("Date incomplete" / "Cod RAR
lipsa") se **suprapune vizual** peste textul din coloana VEHICUL (`CT88NOE` / `B123ABC`
apar lipite/sub pastila). Vizibil clar la 1280px. E un bug de latime de coloana / pastila
fara `white-space:nowrap` sau coloana STARE prea ingusta.
**Recomandare**: largeste coloana STARE / pune pastila pe `nowrap` cu min-width, sau
muta stare si vehicul pe coloane clar separate; testeaza la 1280 si 390.
### 4b. Densitate — tabel cu 8 coloane vs. 4 in mockup (PRIORITATE MEDIE)
Mockup pas 3 = 4 coloane (`VIN / OPERATIE / DATA / STARE` + link editeaza). Real = 8
coloane (`# / STARE / VEHICUL / OPERATIE / DATA / KM FINAL / NOTE / ACTIUNI`), cu coloana
NOTE care afiseaza inline mesaje de validare lungi ("VIN trebuie sa aiba exact 17
caractere..."). Aceeasi tendinta ca lista de trimiteri: prea multa informatie pe rand.
**Recomandare**: redu la coloanele esentiale (Stare / Vehicul / Operatie / Data +
Editeaza); muta KM si mesajul de validare in randul de editare (care le are deja) sau
intr-un tooltip pe pastila de stare.
### 4c. Pastilele de filtru sunt toate albastru-plin (par toate active) (PRIORITATE MICA)
`Toate (2) / Cod RAR lipsa (1) / Date incomplete (1)` sunt randate ca butoane albastru
plin — toate par selectate simultan. Mockup-ul foloseste pastile subtiri cu dot colorat,
doar cea activa accentuata.
**Recomandare**: stil outline + dot pentru filtrele inactive; plin doar pentru cel activ.
---
## 5. Import — pas 1: dropzone compact vs. zona mare din mockup (PRIORITATE MICA)
Mockup pas 1 = zona mare cu chenar punctat, iconita upload centrata, "Trage fisierul
aici", buton "Alege fisier" + chips de format (`.xlsx .csv .xls`). Real = o bara
orizontala slim ("Importa: [Alege fisier] sau trage aici"). Bara compacta se potriveste
cu "import colapsat", deci e o abatere **acceptabila**; daca se doreste fidelitate cu
mockup-ul, zona se poate inalta cand `<details>` e deschis (chenar punctat + iconita).
Pozitiv: stepper-ul (4 pasi, cifre in cerc, pas finalizat = bifa verde) si saltul automat
peste pas 2 la format recunoscut sunt conforme si bune.
---
## 6. Formularul de editare (modal corectie / editare rand)
Comparatie cu modalul din mockup ("Corecteaza trimiterea / randul"):
- **Conform**: structura (VIN; Data + Nr. inmatriculare pe 2 coloane; Observatii;
"Prestatii — cod RAR pe fiecare operatie"; picker cu denumiri; "+ Adauga alta
operatie / cod RAR"). Bug-urile US-004..007 sunt rezolvate functional.
- **Anomalie (PRIORITATE MEDIE)**: intre randul de operatie si controlul "+ Adauga alta
operatie" apare un **chenar gol** (container de chips fara continut) — pare nefinisat /
neintentionat. De ascuns cand nu are chips.
- **Stil nume operatie (PRIORITATE MICA)**: mockup-ul afiseaza numele operatiei
**bold/uppercase, proeminent** ("SCHIMB PLACUTE FRANA — lipsa cod"); real il arata
in greutate normala, mic ("Schimb placute frana · lipsa cod"). Mai putin emfatic.
- **Copy butoane (PRIORITATE MICA)**: real "Salveaza / Anuleaza"; mockup + PRD/US-007
spun "Renunta" (si "Salveaza si retrimite" in modalul de detaliu). Aliniaza eticheta
"Anuleaza" -> "Renunta".
---
## 7. Tema transversala — diacritice in textul vizibil (PRIORITATE MICA)
Mockup-urile (intentia de design) folosesc diacritice romanesti complete in textul catre
user ("Observatii" -> "Observații", "Salveaza" -> "Salvează", "Numar inmatriculare" ->
"Număr înmatriculare", "Adauga" -> "Adaugă", "In coada" -> "În coadă"). Aplicatia reala
omite diacriticele in majoritatea label-urilor. US-001 a confirmat ca fontul de sistem
randeaza corect diacriticele, iar landing-ul le foloseste deja — deci e o diferenta de
finisaj fata de mockup, nu o limitare tehnica.
**Recomandare**: aplica diacritice la **textul vizibil pentru user** (label-uri, butoane,
titluri), pastrand codul/comentariile fara diacritice ca azi. Optional (non-blocant);
de decis daca se urmareste fidelitate completa cu mockup-urile.
---
## 8. Pagini fara mockup dedicat (judecate dupa design system) — CONFORME
- **Mapari** (`?tab=mapari`): carduri, tabele, fonturi uniforme — coerent cu sistemul.
Singura observatie: cardul gol "De rezolvat" cand nu exista needs_mapping (se poate
ascunde cand e gol).
- **Integrare** (`?tab=integrare`): tab-uri de limbaj (curl/Python/PHP/C#/Node/VFP),
blocuri de cod, carduri export + test cheie — curat si profesional.
---
## 9. Ce este DEJA conform mockup-urilor (pentru context — fara actiune)
- **/login**: layout brandeit pe 2 coloane (panou ROMFAST + formular), badge mediu,
link signup — conform `prd-5.16-header-login-tema.html`.
- **Antet**: titlu "ROMFAST AUTOPASS" + badge mediu (TEST) + badge plan (GRATUIT) +
"Service auto: Romfast SRL" + pastila "RAR online" (dot verde) + meniu burger.
Conform US-010/003.
- **Selector tema**: pill cu iconita + eticheta ("Auto"), iconita-only pe mobil.
Conform US-011.
- **Contoare**: 5 carduri separate desktop (Total / Luna asta / Azi / In coada /
De corectat); bara compacta de cifre pe mobil. Conform US-002. (Minor: eticheta
"Total" vs mockup "Total trimise"; pe mobil "Erori" vs mockup "Corectat".)
- **Import colapsat pe Acasa** (`<details>` slim "+ Importa fisier"). Conform US-013.
- **Modal detaliu trimitere finalizata**: read-only, label-uri clare, "Detalii tehnice"
colapsabil — curat si conform.
---
## Rezumat prioritati
| # | Constatare | Prioritate | Fisier principal |
|---|---|---|---|
| 1 | Rand lista cu 4 linii + pastila mereu (rânduri late) | **INALTA** | `_submissions.html:110-139` |
| 4a | Coliziune pastila STARE / coloana VEHICUL in preview import | **INALTA** | `_preview_import.html` |
| 2 | Titlu sectiune "Trimiterile tale" + toolbar bulk permanent | MEDIE | `_acasa.html` / `_submissions.html` |
| 3 | "Plan: N/60" ca rand in corp (duplica antetul) | MEDIE | `_acasa.html` / context layout |
| 4b | Tabel preview cu 8 coloane vs 4 | MEDIE | `_preview_import.html` |
| 6 | Chenar gol de chips in formularul de editare | MEDIE | `_chips_prestatii.html` |
| 4c | Pastile de filtru toate albastru-plin | MICA | `_preview_import.html` |
| 5 | Dropzone import compact vs zona mare | MICA | `_upload.html` |
| 6 | Nume operatie ne-emfatic + copy "Anuleaza" vs "Renunta" | MICA | `_form_editare.html` / `_chips_prestatii.html` |
| 7 | Diacritice lipsa in textul vizibil | MICA | transversal |
**Cele doua corectii cu impact maxim**: (1) comprimarea randului de lista la 2 linii +
ascunderea pastilei pe finalizat, si (4a) bug-ul de coliziune din preview-ul de import.
Restul sunt finisaje de aliniere la spiritul minimalist al mockup-urilor.
---
---
<!-- ================= /autoplan REVIEW APPENDIX ================= -->
# /autoplan — Revizuire automata (CEO → Design → Eng)
> Tratam acest raport ca **plan**: cele 10 recomandari (sectiunile 1-7) sunt
> elementele de implementat. Scope UI: DA (Design conduce). Scope DX: NU
> (sectiunea 8 "Integrare" e marcata CONFORM, fara actiune pe suprafata API/CLI).
> Voci duale: Claude subagent + Codex per faza. Decizii intermediare auto-decise
> pe cele 6 principii; deciziile de gust merg la poarta finala.
## Faza 1 — CEO (Strategie & Scope)
### 0A. Provocarea premiselor
Planul (raportul) se sprijina pe 4 premise implicite:
- **P1 — Fidelitatea fata de mockup este tinta.** Mockup-urile reprezinta intentia
corecta de design; orice abatere a UI-ului real e un defect. *Status: in mare
valida, dar nu absoluta* — raportul insusi recunoaste ca UI-ul real a adaugat
FUNCTII pe care mockup-ul minimalist nu le are (cautare, filtre de timp, bulk-fix
cod RAR, cod RAR + #id_prezentare pe rand). Acele functii pot sa-si merite densitatea.
- **P2 — "Densitatea informationala" e problema centrala**, iar minimalismul ("linistit
cand e ok, zgomotos pe exceptie", D6/zero-silent-failures) e principiul corect.
*Status: validata de durere reala* — userul s-a plans explicit de "randurile foarte
late". Aici premisa e bine sustinuta.
- **P3 — Criteriile de acceptare PRD 5.16/5.17 sunt obligatorii** si UI-ul real a
derivat de la ele (titlu sectiune de eliminat `_coada.html:10`; plan ca badge nu rand
in corp `_status.html:140`; prag tipografic 12px incalcat de `font-size:10px`
`_submissions.html:133`). *Status: validata — sunt AC contractuale, nu preferinte.*
Acestea NU sunt decizii de gust; sunt conformare la PRD.
- **P4 — Mutarea informatiei de pe rand nu pierde nimic** fiindca e deja in modalul
de detaliu / randul de editare. *Status: tehnic adevarata* (verificat: modalul are
cod RAR/nr/#id; randul de editare are KM + mesaj validare), dar muta un cost de la
"vizibil la scanare" la "vizibil dupa click" — un compromis de UX, nu zero-cost.
**Premisa care merita judecata umana** (poarta de mai jos): pentru informatia scoasa
de pe rand (cod RAR, #id_prezentare, marca de timp completa) — o **ascundem in modal**
(minimalism strict, fidel mockup-ului) sau o **pastram in spatele unui toggle
compact/detaliat** (operatorul de service poate vrea sa scaneze cod RAR/#id fara click)?
Userul s-a plans de latime, NU neaparat ca informatia in sine e inutila.
### 0B. Harta de leverage (ce exista deja)
| Sub-problema | Cod existent reutilizat | Tip schimbare |
|---|---|---|
| Compresie rand lista | modal detaliu (`_fragments/trimitere/{id}`) are deja cod RAR/nr/#id | SCADERE (sterge L3/L4 din `_submissions.html`) |
| Pastila pe finalizat | `r.stare_css/stare_scurt` exista; conditie lipsa | conditie `{% if %}` in jurul liniei 138 |
| Prag tipografic 12px | sistemul de token-uri `--fs-xs:12px` exista deja in mockup/base | re-cablare literal `10px` → `--fs-xs` |
| KM + validare in preview | randul de editare le are deja | SCADERE coloane din `_preview_import.html` |
| Chenar gol chips | `_has_ops`/`_chips` deja calculate | conditie `{% if _chips %}` pe container |
Concluzie: planul e **dominant SCADERE + re-tokenizare**, putin cod nou, leverage mare.
### 0B-bis. Pattern de fond depistat (in afara raportului, in blast radius)
`_submissions.html` foloseste **literali px inline** peste tot (`font-size:13px`,
`12px`, `11px`, `10px` — liniile 18, 45, 54, 63, 133, 153, 182...) in loc de token-uri
`--fs-*`. Raportul a prins DOAR instanta de 10px (US-002). Cauza-radacina e ca scala
tipografica 5.16 nu e aplicata sistematic in template-urile de lista/preview. *Flag
pentru poarta finala: extindem fix-ul la re-tokenizarea template-urilor atinse, sau
doar instanta 10px?* (In blast radius, < 1 zi CC — candidat de auto-aprobat pe P2.)
### 0C. Dream-state delta
```
CURENT → ACEST PLAN → IDEAL 12 LUNI
UI real, dens, derivat de Aliniat la minimalismul Sistem de token-uri aplicat
la AC-urile PRD 5.16/5.17; mockup-ului; bug 4a rezolvat; uniform (zero literali px);
bug coliziune coloane; randuri 2 linii; tipo 12px+ teste de regresie design vs
literali px imprastiati. pe instantele semnalate. mockup (Playwright snapshot).
```
Delta ramasa dupa plan: re-tokenizarea completa + testele de regresie vizuala (defer).
### 0C-bis. Alternative de implementare
| # | Abordare | Efort (CC) | Risc | Pro / Contra |
|---|---|---|---|---|
| A | Fix exact ca raportul (scade L3/L4 in modal, ascunde pastila, fix bug, polish) | ~30 min | mic | + fidel mockup, simplu / operatorul pierde cod RAR/#id la scanare |
| B | Ca A, dar info de rand in spatele unui toggle compact/detaliat | ~60 min | mediu | + nu pierde info / complexitate noua, contrazice "explicit over clever" (P5) |
| C | Ca A + re-tokenizare px→token in template-urile atinse | ~50 min | mic | + rezolva cauza-radacina P2 / atinge mai multe linii |
Recomandare CEO: **A pentru structura** (P5 explicit, P1 completeness fata de mockup),
cu **C ca extindere in blast radius** (P2 boil-the-lake pe tipografie). B intra la poarta
finala ca decizie de gust (toggle vs. mutare-in-modal).
### 0D. Mod: **SELECTIVE EXPANSION**
Nucleu = sectiunile 1 + 4a (impact maxim, una e bug). Extindere selectiva in blast
radius = re-tokenizarea (0B-bis) + AC-urile PRD (2, 3). Restul (polish MICA) = inclus,
cost trivial.
### 0E. Interogare temporala
- **Ora 1**: bug 4a (coliziune `_preview_import.html`) + compresie rand `_submissions.html`
+ ascundere pastila finalizat. Astea ating durerea userului + singurul bug real.
- **Ora 6+**: sectiunile 2, 3 (conformare AC), chenarul gol chips (6), polish copy/stil,
diacritice (decizie separata).
### 0F. Confirmare mod
SELECTIVE EXPANSION confirmat: planul livreaza nucleul de impact + extinderile in blast
radius care isi platesc costul, defera testele de regresie vizuala.
### POARTA DE PREMISE — REZOLVATA (directiva user, 2026-06-29)
Userul a dat o directiva mai precisa decat oricare optiune A/B/C. **Spec guvernanta
pentru randul de lista:**
> **2 linii MAXIM** (inaltime minimalista, ca in mockup), dar randul CONTINE:
> **nr. inmatriculare · operatia RAR (cod) · operatia din service (denumire) · data**,
> plus **pill de stare (inclusiv "Finalizat")**.
Consecinte (override-uri fata de recomandarile raportului):
- **OVERRIDE rec. 1.1** (partial): cod RAR si operatia din service RAMAN pe rand, NU se
muta in modal. Doar VIN (ca identificator primar), #id_prezentare si secundele din
timestamp se scot. Identificatorul primar devine **nr. inmatriculare**, nu VIN.
- **OVERRIDE rec. 1.2**: pastila RAMANE pe finalizat (userul cere explicit "+ pill
finalizat"). NU se ascunde pe starea normala. (Raportul recomanda ascunderea — anulat.)
- **CONFIRMA rec. 1.3**: marca de timp scurta (data, fara secunde).
- **CONFIRMA rec. 1.4**: implicit 2 linii (fara toggle detaliat — userul nu vrea toggle).
Aceasta devine cerinta de design pentru Faza 2 (aranjarea celor 5 campuri in 2 linii).
Campuri necesare pe rand: `vehicul_nr`, `cod_rar`, `operatie` (denumire service), `data`,
`pill`. Campuri eliminate: `vin_scurt` (sau retrogradat), `#id_prezentare`, secunde.
> Nota proces: aceasta a fost singura poarta de judecata umana din Faza 1. Suprafata
> strategica (minimalism vs. densitate) a fost decisa de user; nu mai exista premisa
> deschisa de provocat. Vocile duale CEO sunt redundante pe aceasta suprafata si se
> consolideaza in Faza 3 (vezi nota de proportionalitate).
### Voci (proportionalitate)
- Codex: **INDISPONIBIL** (limita de utilizare atinsa, reset 18 iul) → tag `[subagent-only]`.
- Claude subagent Design + Claude subagent Eng: rulate la adancime completa, pe cod real
(template-uri + rute + teste), nu pe proza. Acestea sunt vocile substantiale.
## Faza 2 — Design (UI/UX)
### Aranjarea randului de 2 linii (livrabilul central)
Placuta-primul e corect: un operator identifica masina dupa nr. inmatriculare de pe
comanda, nu dupa VIN de 17 caractere. Layout propus (peste `.trimitere-slim` existent):
```
L1: B-123-ABC (placuta, --fs-md, weight 600, ink) ............ [ PILL dreapta ]
L2: OE-8 (cod RAR, mono/accent) · Schimb placute frana (operatie, ink, ellipsis) · 27.06.2026 (muted)
```
- L1 = `vehicul_nr` (stanga, `flex:1 1 auto; min-width:0`) + pill (dreapta, `flex:0 0 auto`).
- L2 = flex 3 celule: cod RAR (auto, primul — e identificatorul scanabil) · operatie
(`flex:1 1 auto; min-width:0; white-space:nowrap; text-overflow:ellipsis` — ellipsis-ul
pe operatie garanteaza ca randul NU trece pe a 3-a linie nici la 390px) · data (muted, ultima).
- Operatia ramane **ink, nu muted** (e al doilea cel mai citit camp dupa placuta).
- Ierarhie vizuala: placuta → pill → cod+operatie → data.
### CONSTATARI DESIGN dincolo de raport (corectii)
| # | Constatare | Sev | Fix |
|---|---|---|---|
| D-1 | Linia `eticheta_problema` (L:129-134) e a **5-a linie** → strica "2 linii MAX" pe randurile de eroare | inalta | DECIZIE DE GUST (vezi poarta) — drop vs micro-linie doar pe eroare |
| D-2 | Pastilele **NU sunt conforme** (raportul sec.9 gresit): chip outline gri, fara dot/fill, doar culoare text. Cu pill permanent pe orice rand → zgomot gri permanent | medie-inalta | restileaza pill ca mockup: fill tint + dot 7px + text colorat (DECIZIE DE GUST) |
| D-3 | Bug 4a cauza-radacina: `table-layout:fixed` + `.col-stare width:104px` (base.html:401) + pill `nowrap` → overflow peste col-vehicul | inalta | widen `.col-stare`→~140px; reducerea 8→4 col NU rezolva bug-ul (curge in coloanele fluide, nu in col-stare fixa) |
| D-4 | Lipsa stare de eroare la incarcarea listei (HTMX `/_fragments/submissions` 500 → spinner blocat) | medie | adauga partial de eroare / `hx-on::response-error` (DEFER TODOS — pre-existent) |
| D-5 | Filtre 4c "toate albastru": raportul e **STALE** — codul are deja `background:transparent` + doar activ plin (`_preview_import.html:56-58,277`). Ramane doar diferenta stilistica (fara dot) | mica | NO-ACTION pe bug; eventual dot pe mockup (gust, optional) |
### Litmus design (consens)
```
DESIGN — voci: Claude-sub Codex Consens
1. Layout 2 linii fezabil/curat? DA N/A Confirmat (single voice)
2. Placuta-primul corect? DA N/A Confirmat
3. Bug 4a cauza reala identificata? DA N/A Confirmat
4. Pill conform mockup? NU N/A Flag (D-2)
5. Stari complete (loading/error/mobil)? partial N/A Gap (D-4 error state)
6. Polish: defect vs gust separat? DA N/A Confirmat (4c stale, 6 real)
```
## Faza 3 — Eng (arhitectura, regresie)
### Arhitectura (grafic dependente)
```
_acasa.html ─include─ _coada.html ─include─ _submissions.html (LISTA: .lista-trimiteri-slim)
└─ titlu "Trimiterile tale" (h2, L:10) + export CSV ← scoate (PRD)
_preview_import.html (.tabel-trimiteri) ─include─ _preview_rand.html (pill inline-flex) ← bug 4a
_chips_prestatii.html (.chips operatii-mode, L:122) ← chenar gol
_status.html:140 rand plan N/60 in corp ← muta in burger/cont (PRD)
DATE: r.prez = prezentare_din_payload (payload_view.py:86) → vehicul_nr, cod_rar,
operatie, data_prestatie TOATE prezente. Schimbare = TEMPLATE-ONLY (fara rute).
```
### Decizie semantica: marca de timp
`r.updated_at` (L:111) = `format_data_rar` care adauga MEREU `%H:%M:%S` (labels.py:158) →
sursa secundelor zgomotoase. **Auto-decis: foloseste `r.prez.data_prestatie`** (data
prestatiei declarate, deja date-only `2026-06-18`) — semantic e "data" pe care o cere
userul, langa celelalte campuri de prezentare. (Alternativa: helper `format_data_scurta`
%d.%m.%Y daca trebuie pastrat updated_at — respins ca redundant.)
### Eng consensus table
```
ENG — voci: Claude-sub Codex Consens
1. Arhitectura sunet (template-only)? DA N/A Confirmat
2. Acoperire teste suficienta? NU (3 rup) N/A Gap mapat (vezi test plan)
3. Riscuri performanta? nule N/A Confirmat (subtractiv)
4. Securitate? N/A N/A Fara suprafata noua
5. Cai de eroare tratate? partial N/A Gap: vehicul_nr=='—' + D-4
6. Risc deploy gestionabil? DA N/A Confirmat (4 teste de update)
```
### Regresie (artefact pe disc)
Test plan scris: `~/.gstack/projects/romfast-rar-autopass/feat-5.18-corpus-knn-test-plan-20260629-071500.md`
- **3 teste se strica HARD**: `test_vin_pe_rand_separat_sub_nr`, `test_rand_slim_vin_operatie_pill`,
`test_submissions_coloane_umane` (toate hard-codeaza VIN-primar / #id-pe-rand).
- **2 la risc**: depind de numele claselor → **pastreaza `slim-vin`/`slim-meta`** (reumple, nu redenumi).
- Invariant cod_rar ("OE-2 vizibil, fara prefix, nemapat") **pastrat** de spec.
### Registru moduri de esec
| Mod | Trigger | Tratare in plan | Gap? |
|---|---|---|---|
| Placuta lipsa | payload fara `vehicul_nr` → `'—'` | azi mascat de VIN-primar | **GAP — auto-include fallback** (nu randa em-dash singur) |
| cod_rar lipsa | nemapat | guard `!= '—'` → "nemapat" | OK (pastrat) |
| operatie lunga la 390px | denumire lunga | ellipsis + min-width:0 (vezi L2) | OK daca se aplica layout-ul |
| Lista 500 / network drop | HTMX swap esueaza | — | GAP D-4 (defer TODOS) |
| Pill finalizat a11y | text-in-pill | stare prin TEXT + title | OK (invariant respectat) |
### Retokenizare px (auto-decis: BOUNDED)
Eng: retokenizarea completa px→token e scope creep (`13px→--fs-sm`=13.5px schimba layout,
risc regresie vizuala fara baza AC). **Auto-decis: doar instanta sub-12px** (`eticheta-problema`
10px→`--fs-xs`) — singura cu acoperire AC. (Suprascrie sugestia CEO 0B-bis de auto-aprobare larga.)
## Decision Audit Trail
| # | Faza | Decizie | Clasificare | Principiu | Rationament | Respins |
|---|---|---|---|---|---|---|
| 1 | CEO | Rand=2 linii cu placuta+codRAR+op+data+pill | Premisa (user) | — | directiva user la poarta | mutare cod RAR in modal |
| 2 | CEO | Identificator primar=placuta, nu VIN | Mechanical | P1 | operator scaneaza placuta | VIN primar |
| 3 | Eng | "data" = `data_prestatie`, nu `updated_at` | Mechanical | P5 | semantic corect, fara secunde, fara helper nou | slice updated_at |
| 4 | Eng | Pastreaza clase `slim-vin`/`slim-meta` | Mechanical | P3 | minimizeaza churn de teste | redenumire clase |
| 5 | Eng | Fallback `vehicul_nr=='—'` | Mechanical | P1 | evita em-dash singur ca id primar | lasa em-dash |
| 6 | Design | Bug 4a: widen `.col-stare`~140px | Mechanical | P5 | cauza reala (fixed 104px+nowrap) | doar nowrap/min-width |
| 7 | Design | 8→4 coloane preview (densitate) | Mechanical | P1 | match mockup; NU rezolva 4a singur | pastreaza 8 col |
| 8 | Eng | Guard `{% if _extra %}` pe `.chips` | Mechanical | P5 | elimina chenar gol | container mereu |
| 9 | Eng | Retokenizare px BOUNDED (doar 10px) | Taste→auto | P5 | evita shift vizual nebazat AC | retokenizare larga |
| 10 | Design | Filtre 4c: NO-ACTION (raport stale) | Mechanical | P4 | codul deja corect | re-implementare |
| 11 | CEO | Sec.2 titlu + sec.3 plan N/60: scoate | Mechanical | P1 | AC PRD 5.16/5.17 obligatorii | pastreaza |
| 12 | Design | Stare eroare lista (D-4): DEFER TODOS | Mechanical | P3 | pre-existent, in afara cererii | include acum |
| T1 | Design/Eng | eticheta_problema: **PASTREAZA micro-linie doar pe rand de eroare** (user) | Gust→rezolvat | — | normal/finalizat=2 linii strict; eroare=2+motiv (D6 loud-on-exception) | drop complet |
| T2 | Design | **DA — restileaza pill fill+dot ca mockup** (user) | Gust→rezolvat | — | pill permanent isi merita greutatea vizuala | lasa contur gri |
| T3 | trans | **NU aplica diacritice** (user) | Gust→rezolvat | — | non-blocant; ramane divergenta de finisaj acceptata | include/pas separat |
## PLAN APROBAT (user, 2026-06-29) — implementarea NU se executa in aceasta sesiune
> Status: **APROBAT ca plan**. User a ales "doar planul, nu implementa inca". Task-urile
> de mai jos sunt gata de executat intr-o sesiune viitoare (sau /ship cand exista cod).
### Spec final randul de lista (de implementat in `_submissions.html`)
- **L1**: `vehicul_nr` (placuta, primar, `--fs-md`/weight 600, `.slim-vin` reumplut) + **pill** dreapta.
- **L2** (`.slim-meta`): `cod_rar` (sau "nemapat", mono/accent, prima) · `operatie` (ink, ellipsis,
`min-width:0`) · `data_prestatie` (muted). Scoate: VIN primar, `#id_prezentare`, secundele.
- **Pill**: ramane pe FIECARE rand inclusiv Finalizat; restilat fill-tint + dot 7px + text colorat per stare.
- **eticheta_problema**: ramane micro-linie conditionala DOAR pe stari de problema; `10px`→`--fs-xs`.
- **Fallback**: `vehicul_nr == '—'` → nu randa em-dash singur (mesaj fallback).
- Pastreaza numele claselor `slim-vin`/`slim-meta` (reumple, nu redenumi) — minimizeaza churn teste.
### Implementation Tasks (agregat) — LIVRAT 2026-06-29 (toate verzi, 1392 teste)
- [x] **T-1 (INALTA) — `_submissions.html`** — refactor rand 4→2 linii cu placuta+codRAR+operatie+data_prestatie+pill; fallback placuta; clase pastrate. Teste rescrise: test_rand_slim_vin_operatie_pill, test_submissions_coloane_umane, test_placuta_pe_rand_identificator_primar (fost test_vin_pe_rand_separat_sub_nr), test_placuta_lipsa_nu_genereaza_rand_gol (fallback "fara numar").
- [x] **T-2 (INALTA) — `base.html` (CSS pill) + `_submissions.html`** — pill slim restilat (fill tint + dot 7px + text colorat per `stare_css` via currentColor), scopat `.lista-trimiteri-slim .pill`; ramane pe finalizat.
- [x] **T-3 (INALTA) — `base.html`** — bug 4a: `.tabel-trimiteri .col-stare` 104px→140px. nowrap pe col-vehicul neatins.
- [x] **T-4 (MEDIE) — `_preview_import.html` + `_preview_rand.html`** — reducere la 5 coloane (Stare/Vehicul/Operatie/Data/Actiuni); scoase col-id, col-km, col-note; motivul mutat in `title` pe pill, KM in modal.
- [x] **T-5 (MEDIE) — `_coada.html`** — titlu vizibil "Trimiterile tale" → `<h2 class="sr-only">` (a11y pastrat); badge "de rezolvat" + export CSV intr-un rand discret. `.sr-only` adaugat in base.html.
- [x] **T-6 (MEDIE) — `_status.html`** — linia plan in corp DOAR pe avertizare (`plan_warn`/`plan_limita_atinsa`); consum normal in badge antet + burger. Teste status mutate pe pagina completa.
- [x] **T-7 (MEDIE) — `_chips_prestatii.html`** — guard `{% if _extra_chips %}` pe containerul `.chips`, chenarul gol eliminat.
- [x] **T-8 (MICA) — `_submissions.html` / base.html** — `font-size:10px`→`var(--fs-xs)` (eticheta-problema, prin clasa scopata `.lista-trimiteri-slim .eticheta-problema`).
- [x] **T-9 (MICA) — `_form_editare.html` + base.html** — "Anuleaza"→"Renunta" (default); `.op-row-name` emfatic (bold, `--fs-sm`).
- [ ] **Defer — tracked in `TODOS.md`** (la cererea userului 2026-06-29): stare eroare HTMX lista (D-4); retokenizare px completa; diacritice in textul vizibil.
- [x] **Defer — inchis ca acceptabil** (netrackuit): teste regresie vizuala (tooling viitor); dropzone zona-mare (sec.5, raportul il marcheaza acceptabil).
### Verificare la implementare
`python3 -m pytest tests/test_web_submissions.py tests/test_web_submissions_layout.py tests/test_web_responsive.py tests/test_web_preview_compact.py -q`
Test plan complet: `~/.gstack/projects/romfast-rar-autopass/feat-5.18-corpus-knn-test-plan-20260629-071500.md`
## ADDENDUM 2026-06-29 — bug live mobil Mapari (CORECTIE la sectiunea 8)
Sectiunea 8 a raportului a marcat **Mapari ca "CONFORME"**, dar nu a testat corect
randarea mobila. User a raportat (cu screenshot, 390px) doua probleme reale, **REZOLVATE**:
1. **Butoanele Salveaza/Sterge taiate pe mobil.** Cauza: `.tabel-card td button {width:100%}`
(base.html, specificitate 0,1,2) batea `.act {width:44px}` (0,1,0) → cele doua butoane
`.act` deveneau full-width, iar al doilea (Sterge) iesea din card (celula are `nowrap`).
Fix: bloc `@media (max-width:767px)` nou (ultimul in `<style>`) — celula Actiuni devine
flex-row, butoanele `.act` `width:auto; flex:1 1 0` cu text vizibil. Acum ambele butoane
sunt complet vizibile, una langa alta, cu eticheta.
2. **Carduri prea inalte + label-uri inutile.** Cauza: `.tabel-card` randa etichetele
`data-eticheta` ca pseudo-titluri ("Operatie"/"Cod RAR"/"Actiuni") + linia redundanta
"acum: COD — nume" (duplica select-ul). Fix: pe mobil se ascund pseudo-etichetele
(`.tabel-card td::before{display:none}`) si linia "acum:" (`.map-acum{display:none}`),
padding strans. Cardul trece de la ~7 elemente la ~3 (nume + select + butoane).
Fisiere: `app/web/templates/base.html` (CSS), `app/web/templates/_mapari.html` (clasa `map-acum`).
Verificare: 80 teste web verzi (test_web_responsive + mapari + submissions + tabs + modal);
confirmare vizuala la 390px (render TestClient → screenshot Playwright). Atributele
`data-eticheta` raman in DOM (a11y + teste). NEPLASAT inca: commit (la cererea userului).
> Lectie pentru viitor: "conform" in raportul vizual trebuie sa includa explicit verificarea
> la 390px a PAGINILOR ACTIONABILE (butoane, formulare), nu doar a layout-ului general.
## GSTACK REVIEW REPORT
- Faze: CEO (premisa rezolvata de user) → Design (subagent, full) → Eng (subagent, full). DX: skip (fara suprafata developer).
- Voci: `[subagent-only]` — Codex indisponibil (limita utilizare, reset 18 iul). 2 subagenti Claude pe cod real.
- Decizii: 15 (12 auto, 3 gust rezolvate de user). Audit trail complet mai sus.
- Status: **APROBAT ca plan**; implementare amanata la cererea userului.
- Artefacte: test plan pe disc; restore point pe disc; task list agregat mai sus.
## NOT in scope (defer TODOS.md)
- Stare de eroare HTMX la incarcarea listei (D-4) — robustete pre-existenta, separata de cerere.
- Teste de regresie vizuala (Playwright snapshot vs mockup) — ideal 12 luni.
- Retokenizare px completa in template-uri — risc shift vizual fara baza AC.
- Dropzone import zona-mare (sec.5) — raport il marcheaza acceptabil.
## Ce exista deja (leverage)
- Toate cele 5 campuri pe `r.prez` (payload_view.py:86) → schimbare template-only.
- Modal detaliu are deja VIN integral + #id (test_detaliu_trimitere) → P4 confirmata, zero pierdere date.
- Sistem token `--fs-*` exista (base.html:52); lista si preview sunt suprafete CSS separate
(`.lista-trimiteri-slim` vs `.tabel-trimiteri`) → widen col-stare e SIGUR pt lista.