Skip to content

style(design): /design-review fixes — broken Mermaid diagram + clipped intro on /architecture#37

Merged
naza00000 merged 2 commits into
mainfrom
claude/pr37-design-review
Jun 15, 2026
Merged

style(design): /design-review fixes — broken Mermaid diagram + clipped intro on /architecture#37
naza00000 merged 2 commits into
mainfrom
claude/pr37-design-review

Conversation

@naza00000

Copy link
Copy Markdown
Contributor

Navegué el sitio entero (12+ subpáginas, desktop/mobile, dark/light) con el binario browse de gstack + Playwright. El sitio está limpio en general (sin AI-slop, sin scroll horizontal, fuentes Geist consistentes, hero fuerte, hamburger nuevo OK). Dos bugs reales, ambos en /architecture (la página técnica clave):

  • FINDING-001 (HIGH): el diagrama Mermaid del grafo de paquetes no renderizaba (Mermaid strict rechaza var(--bg) como color). Fix: color literal. Diagrama vuelve a renderizar.
  • FINDING-002 (MEDIUM): los 2 párrafos de intro estaban en un DocBlock (pre mono) y se cortaban horizontalmente. Fix: DocP normales que wrappean.

Verificado en dev server: diagErr 0, SVG renderiza, intro wrappea. typecheck limpio.

NOTA aparte: detecté que prod perdió el hamburger (#36) por un deploy en paralelo de otra sesión a las 15:08. Este merge + deploy lo restaura.

🤖 Generated with Claude Code

naza00000 and others added 2 commits June 15, 2026 12:08
El grafo de paquetes (el visual central de /architecture) no renderizaba:
'Diagram failed to render: Unsupported color format: var(--bg, #ffffff)'.
Mermaid con securityLevel:strict rechaza CSS var() como color en
edgeLabelBackground. Reemplazado por color literal (#0f172a, matchea el
fill de los nodos). El diagrama ahora renderiza.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…n /architecture

Los dos párrafos de intro estaban envueltos en DocBlock (un <pre> mono con
white-space:pre), pensado para código. La prosa no wrappeaba y se cortaba
horizontalmente. Cambiado a DocP normales (wrappean). En-dash '5–7' -> '5-7'.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@naza00000 naza00000 merged commit a87a1cd into main Jun 15, 2026
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