docs: define accessible review interface #25

Merged
admin merged 3 commits from docs/search-review-accessibility into main 2026-08-26 21:11:35 +00:00
Owner

Фиксирует решения holistic UX-review для внутреннего интерфейса оценки поиска: доступная оценка, адаптивность, состояния и сохранение. Связано с #23.

Фиксирует решения holistic UX-review для внутреннего интерфейса оценки поиска: доступная оценка, адаптивность, состояния и сохранение. Связано с #23.
agent added 1 commit 2026-08-26 20:59:58 +00:00
Author
Owner

Независимый review: есть blocking findings. (1) Для dialog фокус требуется возвращать на кнопку-триггер «Открыть документ», а не на «карточку результата»: карточка обычно не фокусируема, и клавиатурный пользователь потеряет позицию. (2) Пустая выдача предлагает «Очистить фильтры», хотя планом UI фильтры не предусмотрены; оставьте «Изменить запрос» (YAGNI) либо опишите сами фильтры. (3) Ошибка поиска — динамическое нетривиальное сообщение, но не задана доступная семантика: предусмотрите заранее отрендеренный role=alert для ошибки и aria-busy/role=status для загрузки, иначе screen reader её может не объявить. Нативные radio+fieldset, сохранение с валидацией после нажатия и role=status после успеха соответствуют #23; закреплённая панель на широком экране — минимальное решение.

Независимый review: есть blocking findings. (1) Для dialog фокус требуется возвращать на кнопку-триггер «Открыть документ», а не на «карточку результата»: карточка обычно не фокусируема, и клавиатурный пользователь потеряет позицию. (2) Пустая выдача предлагает «Очистить фильтры», хотя планом UI фильтры не предусмотрены; оставьте «Изменить запрос» (YAGNI) либо опишите сами фильтры. (3) Ошибка поиска — динамическое нетривиальное сообщение, но не задана доступная семантика: предусмотрите заранее отрендеренный role=alert для ошибки и aria-busy/role=status для загрузки, иначе screen reader её может не объявить. Нативные radio+fieldset, сохранение с валидацией после нажатия и role=status после успеха соответствуют #23; закреплённая панель на широком экране — минимальное решение.
agent added 1 commit 2026-08-26 21:01:42 +00:00
Author
Owner

Повторный независимый review полного PR после a594aec завершён: blocking findings нет. Все три замечания устранены: фокус возвращается на кнопку-триггер, пустая выдача больше не предлагает отсутствующие фильтры, а загрузка/успех объявляются через стабильный role=status и ошибки — через role=alert. Новых accessibility, UX или layout-регрессий в плане не выявлено.

Повторный независимый review полного PR после a594aec завершён: blocking findings нет. Все три замечания устранены: фокус возвращается на кнопку-триггер, пустая выдача больше не предлагает отсутствующие фильтры, а загрузка/успех объявляются через стабильный role=status и ошибки — через role=alert. Новых accessibility, UX или layout-регрессий в плане не выявлено.
agent added 1 commit 2026-08-26 21:02:35 +00:00
Author
Owner

Финальный независимый review полного PR после 407dd2d завершён: blocking findings нет. CHANGELOG корректно фиксирует доступные состояния и адаптивное поведение. Ранее исправленные решения сохраняются: возврат фокуса на триггер, корректное пустое состояние, стабильный role=status и role=alert для ошибок. Changelog-коммит не принёс регрессий.

Финальный независимый review полного PR после 407dd2d завершён: blocking findings нет. CHANGELOG корректно фиксирует доступные состояния и адаптивное поведение. Ранее исправленные решения сохраняются: возврат фокуса на триггер, корректное пустое состояние, стабильный role=status и role=alert для ошибок. Changelog-коммит не принёс регрессий.
admin merged commit 182b87fff2 into main 2026-08-26 21:11:35 +00:00
Sign in to join this conversation.
No Reviewers
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: agent/akyldash#25