fix: cross-only full-screen diagram close control #14

2026-10-04T04:43:31-07:00

jarvis-at-skic commented

2026-10-04 04:43:31 -07:00

Replaces visible Close text with an aria-hidden cross, retaining accessible name and all existing modal behavior. Square 44px target. Shared component covers Structure, SDLC and Strategy TO-BE.

Validation: RED regression reproduced, 37 tests passed; source contracts, typecheck, production build and built route checks passed. Chromium desktop 1440px/mobile 390px verified all three diagrams: cross-only visible text, accessible name, 44px target, autofocus, click/Escape and focus restoration. Screenshots inspected; diff reviewed, no content or navigation changes.

Risk: low, shared visual control only. Rollback: revert this commit.

Replaces visible Close text with an aria-hidden cross, retaining accessible name and all existing modal behavior. Square 44px target. Shared component covers Structure, SDLC and Strategy TO-BE. Validation: RED regression reproduced, 37 tests passed; source contracts, typecheck, production build and built route checks passed. Chromium desktop 1440px/mobile 390px verified all three diagrams: cross-only visible text, accessible name, 44px target, autofocus, click/Escape and focus restoration. Screenshots inspected; diff reviewed, no content or navigation changes. Risk: low, shared visual control only. Rollback: revert this commit.

jarvis-at-skic added 1 commit 2026-10-04 04:43:31 -07:00

fix: use cross-only fullscreen diagram close control

Build and publish Eldorado documentation / build (pull_request) Successful in 33s

Details

d3b9f7af14

jarvis-at-skic merged commit 7f40de708c into main

2026-10-04 04:44:22 -07:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: eldorado-v1/documentation#14