lekss361/gendesign
1
0
Fork 0
Code Issues 57 Pull requests Projects Releases Packages Wiki Activity Actions

fix(tradein/v2/a11y): немодальный SectionOverlay + контраст токенов + headings — a11y round 3 (#2264) #2279

Merged
lekss361 merged 1 commit from fix/tradein-v2-a11y-round3 into main 2026-07-03 07:47:09 +00:00
Conversation 0 Commits 1 Files changed 8 +224 -56
lekss361 commented 2026-07-03 07:40:59 +00:00
Owner
Copy link

Summary

A11y round 3 по объединённому аудиту #2264 (code-review + живой клавиатурный проход, эпик #2262):

  • SectionOverlay → честный немодальный fullscreen-view (закрывает aria-modal-tradeoff #2193): снят aria-modal, focus-trap переписан — в Tab-цикл входят живые табы верхнего nav + контролы оверлея (inert-поддеревья исключены явно); Esc/возврат фокуса/начальный фокус сохранены. Паритет mouse/keyboard/SR восстановлен.
  • Контраст: 6 muted/disabled-токенов затемнены до ≥4.6:1 vs worst-case grid-фона #dce6f1 (остаток round-2 #2081); hue сохранён, таблица old→new в отчёте.
  • Headings: sr-only h1 + h2 на оверлее/панелях (визуал не меняется, UA-margin сброшен).
  • accent→accentDeep для мелкого текста (номер секции, счётчик «Мои отчёты»); «КОЭФ. ЛОКАЦИИ» теперь кнопка с клавиатурным доступом; фокус после submit переносится на результат-регион; focus-ring инпутов усилен до solid accentDeep; aria-live «N подсказок» в комбобоксе; aria-required на адрес/площадь; aria-hidden на 14 декоративных SVG.
  • Пункты «drawer без aria-hidden на первом маунте» и «preview без main/h1» оказались уже исправленными на HEAD — изменений не вносилось (проверено git diff).

Test plan

  • tsc --noEmit: exit 0
  • next build: exit 0 (13 страниц, /v2 40.3 kB)
  • pre-commit прошёл
  • post-merge: прод-QA под pilottest — keyboard-проход оверлеев (nav достижим Tab'ом), axe re-scan

Известный follow-up (вне scope): accent-текст в «02»/«03»/CV/headline-цене 40px (≈2.87:1) — визуально-тяжёлая правка, отдельным решением.

Refs #2264, #2262, #2193, #2081

## Summary A11y round 3 по объединённому аудиту #2264 (code-review + живой клавиатурный проход, эпик #2262): - **SectionOverlay → честный немодальный fullscreen-view** (закрывает aria-modal-tradeoff #2193): снят `aria-modal`, focus-trap переписан — в Tab-цикл входят живые табы верхнего nav + контролы оверлея (inert-поддеревья исключены явно); Esc/возврат фокуса/начальный фокус сохранены. Паритет mouse/keyboard/SR восстановлен. - **Контраст:** 6 muted/disabled-токенов затемнены до ≥4.6:1 vs worst-case grid-фона #dce6f1 (остаток round-2 #2081); hue сохранён, таблица old→new в отчёте. - **Headings:** sr-only h1 + h2 на оверлее/панелях (визуал не меняется, UA-margin сброшен). - accent→accentDeep для мелкого текста (номер секции, счётчик «Мои отчёты»); «КОЭФ. ЛОКАЦИИ» теперь кнопка с клавиатурным доступом; фокус после submit переносится на результат-регион; focus-ring инпутов усилен до solid accentDeep; aria-live «N подсказок» в комбобоксе; aria-required на адрес/площадь; aria-hidden на 14 декоративных SVG. - Пункты «drawer без aria-hidden на первом маунте» и «preview без main/h1» оказались уже исправленными на HEAD — изменений не вносилось (проверено git diff). ## Test plan - [x] tsc --noEmit: exit 0 - [x] next build: exit 0 (13 страниц, /v2 40.3 kB) - [x] pre-commit прошёл - [ ] post-merge: прод-QA под pilottest — keyboard-проход оверлеев (nav достижим Tab'ом), axe re-scan Известный follow-up (вне scope): accent-текст в «02»/«03»/CV/headline-цене 40px (≈2.87:1) — визуально-тяжёлая правка, отдельным решением. Refs #2264, #2262, #2193, #2081
lekss361 added 1 commit 2026-07-03 07:41:00 +00:00
fix(tradein/v2/a11y): немодальный SectionOverlay + контраст токенов + headings — a11y round 3 (#2264)
All checks were successful
CI Trade-In / changes (pull_request) Successful in 16s
Details
CI / changes (pull_request) Successful in 16s
Details
CI Trade-In / backend-tests (pull_request) Has been skipped
Details
CI / backend-tests (pull_request) Has been skipped
Details
CI / frontend-tests (pull_request) Has been skipped
Details
CI / openapi-codegen-check (pull_request) Has been skipped
Details
CI Trade-In / frontend-checks (pull_request) Successful in 1m13s
Details
a3cab9023f 
- SectionOverlay: убран aria-modal (честный non-modal fullscreen-view); focus-trap
  теперь охватывает живой верхний nav + оверлей (query по артборду, фильтр [inert]),
  Tab из nav возвращается в оверлей, не в спрятанный main/footer.
- Контраст 6 muted/disabled токенов затемнён до >=4.6:1 vs worst-case #dce6f1
  (hue сохранён, lightness-only): muted2/3/4, hint, disabledValue/Label.
- accent -> accentDeep как мелкий текст: номер секции оверлея (15px), счётчик
  "Мои отчёты" (10px).
- Семантика заголовков: sr-only h1 «Мера — оценка квартиры» + h2 (оверлей,
  ResultPanel, ObjectSummary, placeholder-панели).
- HeroBar «КОЭФ. ЛОКАЦИИ»: role=button + tabIndex + Enter/Space + aria-label.
- Фокус после успешной оценки переносится на результат-регион (role=region,
  tabIndex=-1, aria-labelledby); live-region «Оценка готова» сохранён.
- Инпут focus-ring усилен (2px solid accentDeep).
- combobox адреса: aria-live анонс количества подсказок; aria-required на
  адрес + площадь; декоративные inline-SVG помечены aria-hidden.
lekss361 merged commit 703f6f60c0 into main 2026-07-03 07:47:09 +00:00
lekss361 deleted branch fix/tradein-v2-a11y-round3 2026-07-03 07:47:10 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
admin

Admin UI / internal tools / pipeline tracker

  
analytics

Аналитика данных / matching

  
auth

Authentication / authorization / multi-tenancy

  
automation

Автоматизация / cron / scheduler

  
bug

Something isn't working

  
business

Коммерческая воронка / пилоты / pipeline / реестр ПО

  
chore

Operational housekeeping

  
ci

Forgejo Actions / build pipeline

  
compliance

152-ФЗ / реестр ПО / legal / TOS

  
data

Data sources, parsers, ETL

  
data-moat

Уникальные данные — sales tracker, CRM aggregation

  
docs

README / runbooks / onboarding

  
duplicate

This issue or pull request already exists

  
dx

Developer experience / build speed

  
enhancement

New feature or request

  
Fable 5 ревью

Находки полного мульти-агентного ревью (Claude Fable 5, 2026-06-09)

  
feedback/max

Feedback from Max (Senior partner)

  
generative

Generative Design — concept generation engine

  
GG-форсайт

Новое ТЗ от Ген Геныча (июнь 2026): Site Finder v2 — ИИ-прогноз продуктовой стратегии девелопера. Актуальный приоритет.

  
needs-discussion

Pending team sync / decision

  
needs-human

Human review/decision required

  
observability

Monitoring / metrics / logs / Sentry / Uptime

  
pause-bots

Kill-switch — all autonomous bots pause

  
performance

Latency / throughput / DB query tuning

  
priority/p0

Critical / production-blocker

  
priority/p1

High priority

  
priority/p2

Normal priority

  
priority/p3

Низкий приоритет (follow-up, tech-debt)

  
scope/backend

Backend Python — FastAPI, Celery, scrapers

  
scope/db

DB migrations, indexes, SQL

  
scope/devops

docker-compose, Caddy, workflows

  
scope/frontend

Frontend — Next.js, React, TypeScript

  
scope/qa

Tests, smoke, Playwright

  
scrapers

Скрапперы (Avito/Cian/Yandex/Rosreestr)

  
security

Security audit / secrets / vulnerabilities

  
site-finder

Site Finder scoring quality & data layer

  
stage/1

Stage 1: Generative MVP (нед 2-7)

  
stage/2

Stage 2: Site Finder MVP (нед 8-12)

  
status/blocked

Blocked — needs human intervention

  
status/done

Closed, completed

  
status/needs-analysis

Raw/fuzzy task filed by human — auto-analyst подхватывает: code-archeology + декомпозиция в actionable scope/* + status/ready (или split на под-issues)

  
status/needs-fix

Reviewer/QA вернул worker на доработку — worker сам чинит свой PR (fixup), НЕ требует human (в отличие от status/blocked)

  
status/qa

PR merged, awaiting smoke test

  
status/ready

Ready for pickup by worker bot

  
status/review

PR open, awaiting code review

  
status/wip

Work in progress (worker claimed)

  
tech-debt

Технический долг

  
tradein

Trade-In MVP проект

  
ux

User experience / responsive / conversion

  
week ревью 1

Week review 1 — находки multi-agent аудита проекта (2026-06-14)

  
wontfix

This will not be worked on

  
вторичка

Trade-In Estimator — проект оценки вторичного жилья

  
ИРД

Исходно-разрешительная документация — слой ограничений участка для analyze-таблицы

No labels
admin
analytics
auth
automation
bug
business
chore
ci
compliance
data
data-moat
docs
duplicate
dx
enhancement
Fable 5 ревью
feedback/max
generative
GG-форсайт
needs-discussion
needs-human
observability
pause-bots
performance
priority/p0
priority/p1
priority/p2
priority/p3
scope/backend
scope/db
scope/devops
scope/frontend
scope/qa
scrapers
security
site-finder
stage/1
stage/2
status/blocked
status/done
status/needs-analysis
status/needs-fix
status/qa
status/ready
status/review
status/wip
tech-debt
tradein
ux
week ревью 1
wontfix
вторичка
ИРД
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: lekss361/gendesign#2279
No description provided.
Delete branch "fix/tradein-v2-a11y-round3"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?