Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Enhancement][a11y] Доступность ActionSheet #3044

Closed
eugpoloz opened this issue Aug 23, 2022 · 1 comment · Fixed by #7394
Closed

[Enhancement][a11y] Доступность ActionSheet #3044

eugpoloz opened this issue Aug 23, 2022 · 1 comment · Fixed by #7394
Assignees
Labels
a11y:audit Проблемы, найденные в ходе аудита невизуальной доступности VKUI a11y Доступность (цифровая) cmp:action-sheet type:enhancement
Milestone

Comments

@eugpoloz
Copy link
Contributor

eugpoloz commented Aug 23, 2022

Компоненты:

Аудит невизуальной доступности:

  • проверить общую доступность всплывающих окон

Ко всем модальным окнам, Layouts и подобным всплывающим объектам, в том числе не упомянутым в этом документе, относятся следующие рекомендации, сейчас не выполняемые:

  1. Содержимое модального окна должно быть обернуто в контейнер с ролью dialog, и этот контейнер должен иметь лейбл с названием модалки.
  2. Если крестик для закрытия доступен в верхней области модалки, а не только в самом конце, эта кнопка должна быть доступна для скринридера.
  3. Модалка должна закрываться и по нажатию на кнопку, и по нажатию на escape на клавиатуре.
  4. Важно проверить на бою модалку с мобильного устройства с включенным Voiceover/Talkback, возможно ли горизонтальными свайпами одним пальцем, перемещаясь по содержимому модалки, выйти за ее пределы, тем самым навигируясь по странице под модальным окном. Если возможно, нужно ограничить эту модалку.
  5. При нажатии на странице на кнопку, открывающую модальное окно, фокус должен устанавливаться на первый элемент модального окна - на заголовок или кнопку закрытия.
  6. При закрытии модального окна эскейпом, по нажатию на кнопку закрытия или при выборе одного из вариантов, предложенных модальным окном, фокус должен ставиться на кнопку, открывшую модальное окно. Если такой кнопки уже нет (удален пост/подгрузились новые данные/etc), фокус нужно ставить на появившееся на месте исчезнувшей кнопки контейнер (сообщение об успешном удалении поста с предложением восстановить) или на первый подгрузившийся элемент.
  • ActionSheet title

Почему-то эта надпись обернута в <header>, хотя, кажется, что этот тег сюда не подходит. "Вы действительно хотите удалить это видео из Ваших видео?"

  • ActionSheet subtitle

метка на кнопке, желательно, должна быть едина, а не дробить кнопку на две строки

  • ActionSheet selectable

Все элементы для выбора - это простые спаны. Нужно посмотреть этот элемент на бою, потому что сейчас не иллюстративно.

  • исправить пример в storybook

Перед списком действий есть элемент:
заголовок уровень 4 кнопка Открыть
Кажется, что он не должен являться заголовком.

Кроме того, после нажатия на него список скрывается, а при повторном - появляется. В табордер, который в списке зациклен, эта кнопка также не попадает

@eugpoloz eugpoloz added type:enhancement cmp:action-sheet a11y Доступность (цифровая) labels Aug 23, 2022
@eugpoloz eugpoloz added this to VKUI Aug 23, 2022
@eugpoloz eugpoloz moved this to 🗃 Backlog in VKUI Aug 23, 2022
@inomdzhon inomdzhon added this to the a11y. Документация по доступности компонентов milestone Dec 29, 2022
@eugpoloz eugpoloz added the a11y:audit Проблемы, найденные в ходе аудита невизуальной доступности VKUI label Apr 10, 2023
@eugpoloz eugpoloz removed this from the a11y. Документация по доступности компонентов milestone Jun 13, 2023
@SevereCloud SevereCloud moved this from 🗃 Backlog to 🔜 To do in VKUI Mar 17, 2024
@EldarMuhamethanov EldarMuhamethanov self-assigned this Aug 13, 2024
@inomdzhon inomdzhon added this to the v6.5.1 milestone Aug 20, 2024
@github-project-automation github-project-automation bot moved this from 👀 In Review to ✅ Done in VKUI Aug 20, 2024
@vkcom-publisher
Copy link
Contributor

v6.5.1 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a11y:audit Проблемы, найденные в ходе аудита невизуальной доступности VKUI a11y Доступность (цифровая) cmp:action-sheet type:enhancement
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

4 participants