Независимый 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; закреплённая панель на широком экране — минимальное решение.
Повторный независимый review полного PR после a594aec завершён: blocking findings нет. Все три замечания устранены: фокус возвращается на кнопку-триггер, пустая выдача больше не предлагает отсутствующие фильтры, а загрузка/успех объявляются через стабильный role=status и ошибки — через role=alert. Новых accessibility, UX или layout-регрессий в плане не выявлено.
Повторный независимый review полного PR после a594aec завершён: blocking findings нет. Все три замечания устранены: фокус возвращается на кнопку-триггер, пустая выдача больше не предлагает отсутствующие фильтры, а загрузка/успех объявляются через стабильный role=status и ошибки — через role=alert. Новых accessibility, UX или layout-регрессий в плане не выявлено.
Финальный независимый 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 main2026-08-26 21:11:35 +00:00
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Фиксирует решения holistic UX-review для внутреннего интерфейса оценки поиска: доступная оценка, адаптивность, состояния и сохранение. Связано с #23.
Независимый review: есть blocking findings. (1) Для dialog фокус требуется возвращать на кнопку-триггер «Открыть документ», а не на «карточку результата»: карточка обычно не фокусируема, и клавиатурный пользователь потеряет позицию. (2) Пустая выдача предлагает «Очистить фильтры», хотя планом UI фильтры не предусмотрены; оставьте «Изменить запрос» (YAGNI) либо опишите сами фильтры. (3) Ошибка поиска — динамическое нетривиальное сообщение, но не задана доступная семантика: предусмотрите заранее отрендеренный role=alert для ошибки и aria-busy/role=status для загрузки, иначе screen reader её может не объявить. Нативные radio+fieldset, сохранение с валидацией после нажатия и role=status после успеха соответствуют #23; закреплённая панель на широком экране — минимальное решение.
Повторный независимый review полного PR после
a594aecзавершён: blocking findings нет. Все три замечания устранены: фокус возвращается на кнопку-триггер, пустая выдача больше не предлагает отсутствующие фильтры, а загрузка/успех объявляются через стабильный role=status и ошибки — через role=alert. Новых accessibility, UX или layout-регрессий в плане не выявлено.Финальный независимый review полного PR после
407dd2dзавершён: blocking findings нет. CHANGELOG корректно фиксирует доступные состояния и адаптивное поведение. Ранее исправленные решения сохраняются: возврат фокуса на триггер, корректное пустое состояние, стабильный role=status и role=alert для ошибок. Changelog-коммит не принёс регрессий.