# Review feat/presentation-viewer @ e5cccca0

- Fecha: 2026-10-22T15:24:23+00:00
- Verdict: **FAIL**
- Build rama: PASS
- Build merge: SKIP
- Acción: review-fail
- Modo: auto-merge

## Detalle


> reviewer · deepseek-v4.1-flash
✱ Glob "*" 54 matches
✱ Glob "deck/**/*" 0 matches
> reviewer · deepseek-v4.1-flash
→ Read . 
✱ Glob "**/*.patch" 0 matches
> reviewer · deepseek-v4.1-flash
→ Read .git/HEAD 
✱ Grep "demo-shot|fit-fail|vinted-final|20-cover" in .git · 0 matches
> reviewer · deepseek-v4.1-flash
✱ Glob ".git/**" 0 matches
> reviewer · deepseek-v4.1-flash
→ Read .git 
> reviewer · deepseek-v4.1-flash
→ Read .git/packed-refs 
✱ Grep "deck/" in .git/index · 0 matches
> reviewer · deepseek-v4.1-flash
→ Read .git/worktrees 
✱ Grep "deck" in .git/info · 0 matches
> reviewer · deepseek-v4.1-flash
VERDICT: FAIL
SUMMARY: Deck theme overhaul works structurally, but references a screenshot asset not added in the diff and introduces several regressions in the new light theme/CSS cascade.

FINDINGS:
- [blocker] deck/slides/20-cover.html:44 — `assets/demo-shot.png` is referenced for the hero phone mockup, but the diff only adds `deck/assets/fit-fail.jpg`; if that screenshot is not already on `main`, the first slide (main cover) renders a broken image. Same asset also used at deck/slides/22-demo.html:76. Verify it exists on `main`; otherwise the demo deck's opening slide is broken.
- [risk] deck/css/vinted-cover.css:41 — `.slide.oc-cover::before` / `.oc-problem::before` backgrounds are dead code: `vinted.css` sets `.slide::before,.slide::after{display:none!important}` before cover CSS loads, and `!important` cannot be overridden. Cover/problem lose the intended animated/washed background.
- [risk] deck/css/vinted.css:78 — global `.slide{background:transparent}` plus `.slide.oc-cover{background:var(--oc-paper)}` makes the new cover opaque, hiding the `.vc-bg` blobs/waves the README advertises ("fondo animado"). Visual/regression risk, not fatal.
- [risk] deck/slides/22-demo.html:50 — `.oc-demo__external` and `.oc-demo__hint` markup is added but immediately hidden via `display:none!important` in `vinted-final.css:38`; dead markup and lost fallback ("Abrir en pestaña") if focus iframe fails.
- [risk] deck/assets/fit-fail.jpg — new binary asset with no size check in diff; repo rule forbids assets >5 MB. Confirm size before merge.
- [risk] deck/index.html:66 — `<noscript>` message still uses light color `#F2EFE8` on the new light `#F7F6F1` background, making the no-JS message unreadable.
- [risk] deck/css/vinted-stats.css:88 — fixed 1920×1080 layout with 188px hero numerals in a ~748px left column; "750.000 €" risks overflow/clipping at stage scale. Verify visually.
- [risk] deck/manifest.js:12 — removing 7 of 12 slides changes `?slide=N` numbering; any external links/bookmarks to old slide numbers (e.g. old demo at `?slide=10`) now silently clamp to a different slide.