Skip to content

patch: pr7378#7432

Merged
mendrew merged 1 commit into6.5-stablefrom
patch/pr7378
Aug 23, 2024
Merged

patch: pr7378#7432
mendrew merged 1 commit into6.5-stablefrom
patch/pr7378

Conversation

@mendrew
Copy link
Copy Markdown
Contributor

@mendrew mendrew commented Aug 23, 2024

…role=contentinfo by default (#7378)

Добавилено автоматическое выставление role=contentinfo если Component="footer".
Добавлен раздел по доступности с пояснениями когда и как использовать компонент с дефолтным тегом footer. Приведены примеры, где стараемся объяснить где есть смысл в footer а где лучше div.

Заметили, что, возможно, использовать нативный тег footer в этом компоненте Footer было излишне, потому что этот компонент не очень похож на семантическо значение footer.

    Обычно в футере находится справочная информация о сайте, копирайтинг, основная навигация, ссылки на социальные сети и другой похожий контент. doka

Поэтому довольно трудно написать рекомендации по доступности, когда пример использования по смыслу нарушает правила использования footer.
В нашем примере, мы кладём Footer вне Group, тем самым как бы устанавливая Footer для всей страницы, что явно не то, чего мы хотели бы добиться семантически в реальности.

В идеале, в v7 я бы убрал использование нативного тега footer, чтобы не ломать семантику страниц неправильным использованием, потому что, подозреваю, его чаще используют и будут использовать с декоративными целями, а не по смыслу, закладываемому в тег footer.
Изменения
@mendrew mendrew requested review from a team as code owners August 23, 2024 08:55
@mendrew mendrew changed the base branch from master to 6.5-stable August 23, 2024 08:55
@mendrew mendrew removed the request for review from a team August 23, 2024 08:56
@codesandbox-ci
Copy link
Copy Markdown

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

@mendrew mendrew merged commit 90a9b6e into 6.5-stable Aug 23, 2024
@mendrew mendrew deleted the patch/pr7378 branch August 23, 2024 09:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants