Skip to content
This repository has been archived by the owner on Feb 22, 2023. It is now read-only.

Default layout should not nest footer inside main #2179

Closed
obulat opened this issue Feb 14, 2023 · 0 comments · Fixed by #2180
Closed

Default layout should not nest footer inside main #2179

obulat opened this issue Feb 14, 2023 · 0 comments · Fixed by #2180
Labels
♿️ aspect: a11y Concerns related to the project's accessibility 🛠 goal: fix Bug fix 🟧 priority: high Stalls work on the project or its dependents

Comments

@obulat
Copy link
Contributor

obulat commented Feb 14, 2023 •
edited
Loading

Description

When the footer is nested inside the main element, it loses its aria role, and NVDA does not communicate the visitor that the element has contentinfo landmark.

Original description by @alexstine

There is no contentinfo landmark or footer HTML5 tag on the home page. Please add this above the language translate widget as this is footer content.
@alexstine alexstine mentioned this issue Feb 22, 2023
Closed
15 tasks
@obulat obulat changed the title There is no contentinfo landmark or footer HTML5 tag on the home page. Please add this above the language translate widget as this is footer content. Default layout should not nest footer inside main Feb 14, 2023
@obulat obulat added 🛠 goal: fix Bug fix ♿️ aspect: a11y Concerns related to the project's accessibility 🟧 priority: high Stalls work on the project or its dependents labels Feb 14, 2023
@obulat obulat mentioned this issue Feb 14, 2023
Merged
7 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
♿️ aspect: a11y Concerns related to the project's accessibility 🛠 goal: fix Bug fix 🟧 priority: high Stalls work on the project or its dependents
Projects
None yet
Milestone
No milestone
1 participant
@obulat