Skip to content

VIS-PF-02: compact editorial Portfolio rows - #75

Merged
devrodri-com merged 2 commits into
mainfrom
feat/devrodri-vispf02-compact-editorial-rows
Aug 4, 2026
Merged

VIS-PF-02: compact editorial Portfolio rows#75
devrodri-com merged 2 commits into
mainfrom
feat/devrodri-vispf02-compact-editorial-rows

Conversation

@devrodri-com

Copy link
Copy Markdown
Owner

Objetivo

Implementa exclusivamente la opción aprobada COMPACT_EDITORIAL_ROWS para /portfolio y /en/portfolio.

Cambio

  • Cards apiladas por debajo de 1024px y filas editoriales horizontales desde 1024px.
  • Media compacta con ratio estable, independiente de la altura del copy.
  • Contenido de altura natural y acciones al pie en desktop.
  • Sin alturas rígidas, min-heights globales, truncado ni excepciones por proyecto.
  • Preserva assets, fits, copy, links, accesibilidad, No-JS, Home y case study.

Validación local

  • Node 22.20.0 y npm 9.9.4.
  • npm ci, typecheck, lint, test, build y CI agregado: PASS.
  • 28 suites / 265 tests.
  • ES/EN verificados en 320, 390, 430, 768, 834, 1023, 1024, 1440 y 1920 px.
  • 8 imágenes / 8 requests únicas; sin duplicados, clipping, overflow ni copy truncado.
  • JS on/off, filtros, toggles, ARIA y teclado: PASS.
  • Home y case study comparados contra producción canónica: sin regresión.

Alcance

Draft para Preview y aprobación visual de Rodrigo. No mergear.

@vercel

vercel Bot commented Aug 4, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
minisitio-rodrigo Ready Ready Preview Aug 4, 2026 4:26pm

Request Review

@devrodri-com
devrodri-com marked this pull request as ready for review August 4, 2026 17:10
@cursor

cursor Bot commented Aug 4, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@devrodri-com
devrodri-com merged commit 589211d into main Aug 4, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant