Trap focus and handle Escape in all 5 modals Closes #26

[pasanghilp-art]

shared createFocusTrapp helper,
wired into open/close for all 5 modals,
extended the Excape keydown handler,
note npm test passes with no regressions,
couldnt fully test live due to supabase access.

[cursor]

Maintainer follow-up pushed to this branch (thanks @pasanghilp-art — your trap + wiring stays as the base commit):

• Shared trapModal / closeModal so save/import closes also release the trap and restore focus
• One Escape handler (extends the existing builder/drawer listener) instead of five duplicates
• role="dialog" aria-modal="true" aria-labelledby=… on all five modals
• Hardened createFocusTrap (.box scope, visible focusables, optional initial focus)
• Smoke checks for the helper + dialog names

This should fully satisfy #26. Ready to merge from here; duplicate #41 closed.

[cursor]

Follow-up on this branch (still credited to @pasanghilp-art as author of the base work):

1. inert on page behind — while a modal is open, every other body child is marked inert (SR + Tab can’t reach the board). Cleared before focus restore (browsers ignore focus() into inert trees).
2. Backdrop click closes — click the dimmed .modal chrome (not .box) → closeModal.
3. Live keyboard pass — npm run test:modal-a11y (Playwright): initial focus, Tab / Shift+Tab wrap, Escape, backdrop, opener restore, inert on/off. All green locally.

Should fully cover #26 plus the three gaps from review. Ready to merge once CI finishes.