-
Notifications
You must be signed in to change notification settings - Fork 198
[Enhancement][a11y][Footer]: Добавить role="contentinfo" #4692
Copy link
Copy link
Labels
a11yДоступность (цифровая)Доступность (цифровая)a11y:auditПроблемы, найденные в ходе аудита невизуальной доступности VKUIПроблемы, найденные в ходе аудита невизуальной доступности VKUIcmp:footercmp:grouptype:enhancement
Milestone
Description
Из аудита невизуальной доступности VKUI:
Нужно тегу
<footer />задатьrole="contentinfo". В принципе теги семантических областей лучше дублировать ролями, потому что в некоторых ситуациях они просто не считываются.
Поскольку Footer у нас используется как минимум внутри Group и role="contentinfo" повторяется, то каждый Footer требует обязательного aria-label. Соответственно, нужен какой-то временный пропс для Group, использующийся вместе с description.
UPD(@mendrew): это не верно, в Group Footer не используется, там используется Footnote, так что такой проблемы нету.
VKUI/packages/vkui/src/components/Group/Group.tsx
Lines 152 to 154 in 0f313a6
| {hasReactNode(description) && ( | |
| <Footnote className={styles['Group__description']}>{description}</Footnote> | |
| )} |
Что я вижу как решение:
Footer: добавляем дефолтнуюrole="contentinfo", даем возможность переопределять ее вместе сComponentGroup: создаем подкомпонентGroup.Description, описываем, как его использовать и как сделать доступным [Feature]: Декомпозировать Group #5758Group: депрекейтим пропdescriptionв пользуGroup.Description.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
a11yДоступность (цифровая)Доступность (цифровая)a11y:auditПроблемы, найденные в ходе аудита невизуальной доступности VKUIПроблемы, найденные в ходе аудита невизуальной доступности VKUIcmp:footercmp:grouptype:enhancement
Type
Projects
Status
✅ Done