chore(frontend): DialogDescription en dialogs para a11y #31

dmolinari · 2026-04-18T23:55:51Z

dmolinari commented

2026-04-18 23:55:51 +00:00

Contexto

Radix Dialog emite warning en consola "Missing Description or aria-describedby". Pre-existente; detectado durante CAT-001 batch 4. Es hygiene de accesibilidad — los screen readers usan la description.

Scope

Todos los dialogs del frontend EXCEPTO Rubros (ya fixeados en PR #30 CAT-001).

Cambios

4 dialogs en features/fiscal (iva + iibb) con DialogDescription agregada
Descripciones significativas (no placeholder), con contexto distinto para modo create vs edit donde aplica
DialogDescription importado correctamente en cada archivo

Archivos modificados:

features/fiscal/iva/components/TipoDeIvaFormModal.tsx
features/fiscal/iva/components/NuevaVigenciaModal.tsx
features/fiscal/iibb/components/IngresosBrutosFormModal.tsx
features/fiscal/iibb/components/NuevaVigenciaIibbModal.tsx

Dialogs que ya tenían description (sin cambios):

features/secciones/components/DeactivateSeccionModal.tsx — AlertDialogDescription ya presente
features/puntos-de-venta/components/DeactivatePuntoDeVentaModal.tsx — AlertDialogDescription ya presente
features/medios/components/DeactivateMedioModal.tsx — AlertDialogDescription ya presente
features/users/components/ResetPasswordModal.tsx — usa raw Radix con aria-describedby manual

Verificación

npx vitest run → 349/349 ✅ (55 test files)
No console warnings sobre aria-describedby en los dialogs tocados

## Contexto Radix `Dialog` emite warning en consola "Missing Description or aria-describedby". Pre-existente; detectado durante CAT-001 batch 4. Es hygiene de accesibilidad — los screen readers usan la description. ## Scope Todos los dialogs del frontend EXCEPTO Rubros (ya fixeados en PR #30 CAT-001). ## Cambios - 4 dialogs en `features/fiscal` (iva + iibb) con `DialogDescription` agregada - Descripciones significativas (no placeholder), con contexto distinto para modo create vs edit donde aplica - `DialogDescription` importado correctamente en cada archivo **Archivos modificados:** - `features/fiscal/iva/components/TipoDeIvaFormModal.tsx` - `features/fiscal/iva/components/NuevaVigenciaModal.tsx` - `features/fiscal/iibb/components/IngresosBrutosFormModal.tsx` - `features/fiscal/iibb/components/NuevaVigenciaIibbModal.tsx` **Dialogs que ya tenían description (sin cambios):** - `features/secciones/components/DeactivateSeccionModal.tsx` — AlertDialogDescription ya presente - `features/puntos-de-venta/components/DeactivatePuntoDeVentaModal.tsx` — AlertDialogDescription ya presente - `features/medios/components/DeactivateMedioModal.tsx` — AlertDialogDescription ya presente - `features/users/components/ResetPasswordModal.tsx` — usa raw Radix con `aria-describedby` manual ## Verificación - `npx vitest run` → 349/349 ✅ (55 test files) - No console warnings sobre aria-describedby en los dialogs tocados

dmolinari added 1 commit 2026-04-18 23:55:52 +00:00

chore(frontend): DialogDescription en dialogs para a11y (silencia Radix warning) 1cb69cbaf3

dmolinari merged commit 18ce4f6841 into main

2026-04-19 10:41:10 +00:00

dmolinari deleted branch chore/dialog-aria-describedby

2026-04-19 10:41:10 +00:00

dmolinari referenced this issue from a commit

2026-04-19 10:41:10 +00:00

Merge pull request 'chore(frontend): DialogDescription en dialogs para a11y' (#31) from chore/dialog-aria-describedby into main

Sign in to join this conversation.