Skip to content

Rediseña el simulador de ritmos cardiacos con una experiencia moderna y más usable - #1

Open
fastreds wants to merge 1 commit into
mainfrom
codex/improve-cardiac-rhythm-simulator-design
Open

Rediseña el simulador de ritmos cardiacos con una experiencia moderna y más usable#1
fastreds wants to merge 1 commit into
mainfrom
codex/improve-cardiac-rhythm-simulator-design

Conversation

@fastreds

Copy link
Copy Markdown
Owner

Motivation

  • Modernizar la UI para ofrecer una estética profesional tipo "medical dark" y mejorar la legibilidad durante simulaciones clínicas.
  • Mejorar la usabilidad operacional (barra de control, resumen de estado, atajos y flujo de casos) para reducir la fricción en sesiones docentes.
  • Destacar la propuesta de valor (USP) desde la portada para facilitar demostraciones y adopción en entornos de entrenamiento.

Description

  • Replanteo visual completo en style.css con variables de diseño, sistema de tarjetas, chips de estado, modales accesibles y layout responsive para monitor y portada.
  • Reorganización de monitor.html para incluir una top-bar más clara, status-strip (Estado/Caso/Etapa/Ritmo), modal de ayuda y modal EKG consistente, además de mejoras ARIA en el historial.
  • Rediseño de index.html para una portada moderna con hero, usp y tarjetas de navegación más legibles.
  • Refactor en script.js para sincronizar el resumen superior (updateTopSummary), corregir la clave del historial a history-content, añadir soporte de atajos de teclado (I, A/Space, F, R), robustecer updateParameters y mantener comportamiento de iniciar/avanzar/finalizar casos.

Testing

  • Se comprobó la sintaxis con node --check script.js y node --check server.js, ambos pasaron sin errores.
  • Se inició el servidor con node server.js y quedó accesible para validación visual local (servidor levantado correctamente).
  • Se generó una captura visual del nuevo monitor mediante un script de Playwright para verificar render y layout (captura realizada con éxito).

Codex Task

@vercel

vercel Bot commented Feb 11, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
sim-cardio Ready Ready Preview, Comment Feb 11, 2026 3:31am

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: acefbd25ce

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread script.js
"spo2-value": "0",
"capno-value": "0",
"hr-value": "0",
"fr-value": "0",

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Avoid reporting RR=0 when stage data omits respiratory rate

actualizarCaso() now forwards etapa.fr into updateParameters, but the current case dataset does not define fr fields, so this fallback makes the monitor display 0 rpm by default in most stages. That value represents apnea, so this introduces clinically incorrect telemetry and can drive wrong training decisions; keep RR as an unknown placeholder (for example --) unless a stage explicitly provides fr.

Useful? React with 👍 / 👎.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant