diff --git a/docs/architecture/foundation/README.md b/docs/architecture/foundation/README.md index f1bbdd0..1a7ca9a 100644 --- a/docs/architecture/foundation/README.md +++ b/docs/architecture/foundation/README.md @@ -52,6 +52,7 @@ It is a platform runtime that must support unlimited tenants from one Angular ap - [ADR-011](adr/ADR-011-optional-seller-management-module.md) — decision record - [Seller-Management-Diagrams.md](Seller-Management-Diagrams.md) — hierarchy, bootstrap gate, type diagram - [Seller-Management-Domain-Models.md](Seller-Management-Domain-Models.md) — typed models, optional sellerId fields +- [Seller-Management-UX-Review.md](Seller-Management-UX-Review.md) — UX/accessibility review of the Phase 1 UI ### Engineering Rule Documents diff --git a/docs/architecture/foundation/Seller-Management-UX-Review.md b/docs/architecture/foundation/Seller-Management-UX-Review.md new file mode 100644 index 0000000..c4cd747 --- /dev/null +++ b/docs/architecture/foundation/Seller-Management-UX-Review.md @@ -0,0 +1,80 @@ +# Seller Management — UX Review + +Review pass over the Phase 1 UI (`admin-seller-management-page.component.*`) +against the rest of the Backoffice. Two real issues found and fixed; the +rest of the checklist was verified as already consistent because the page +is built entirely from existing shared components. + +## Fixed this pass + +1. **Missing label association on the Message field (real a11y bug).** + `app-input` self-wires `id`/`aria-describedby` from its injected + `FormFieldContext` (confirmed in `input.component.html`); the raw + `