Skip to content

fix(ui): render modal as centered overlay instead of inline block (#54) - #59

Merged
licette32 merged 1 commit into
mainfrom
feat/bug-54-register-asset-modal
Aug 22, 2026
Merged

licette32 merged 1 commit into
mainfrom
feat/bug-54-register-asset-modal

Conversation

@licette32

Copy link
Copy Markdown
Owner

Summary

Cierra el issue #54. El formulario "Register new asset" (y los otros 3 modales que comparten el mismo bloque: report, complete, decommission) renderizaban inline debajo de la lista de equipos, lo que obligaba a scrollear más allá de toda la lista para encontrarlos. Ahora renderizan como overlay centrado, reutilizando el patrón ya existente en ActionsPanel.tsx.

Changes

  • EquipmentTable.tsx: el bloque {activeModal && (...)} ahora es un overlay fijo (fixed inset-0, backdrop bg-black/70, role="dialog", aria-modal="true"), con cierre al clickear afuera y stopPropagation en el contenido para evitar cierres accidentales.
  • El botón de trigger ("+ Register asset") ya estaba en el header de la sección Equipment; no requirió reubicación.
  • max-h-[90vh] overflow-y-auto para contenido largo.

Testing

  • tsc --noEmit: sin errores.
  • npm run dev en WSL: sin errores de compilación/parseo.
  • Verificado manualmente en navegador: los 4 modales (register, report, complete, decommission) abren centrados, cierran al clickear afuera, y el submit de cada uno dispara la transacción on-chain correspondiente sin cambios de comportamiento.

Notes

  • Sin cambios en validación, lógica de submit, ni llamadas on-chain, solo colocación/presentación.

@vercel

vercel Bot commented Aug 22, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
medovant-protocol Ready Ready Preview Aug 22, 2026 11:05pm

@licette32 licette32 added this to the v1.1 — Demo Day milestone Aug 22, 2026
@licette32
licette32 merged commit 69ccea3 into main Aug 22, 2026
4 checks passed

This branch was successfully deployed

1 active deployment
Preview — 63267cff Deployed Aug 22, 2026 by vercel[bot]
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