added main benchmark to all pages, labeled all footer buttons, labele… #1005
Conversation
…d primary nav and footer nav
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also, I think aria-labels should be lowercase. deferring to @akmadian for further suggestions.
@@ -71,14 +71,14 @@ | |||
</div> | |||
</div> | |||
<div class="column"> | |||
<aside class="donate-section"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess it would be better to leave the markup of this tag to be aside itself since it follows the rules of good HTML semantics and also helps in improving accessibility.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I changed that because OP's csv file reported this issue:https://dequeuniversity.com/rules/axe/3.3/landmark-complementary-is-top-level
I've seen aside used in main all the time, but this aside seems to be contained in a column
I am not that familiar with aria-labels, usually framework handles it, glad you point out my inconsistency, I will change them lower cases and maybe tighten up the naming as well.
@ayanchoudhary could you re-review this this week? I'll merge once you approve; thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good :) @zackkrida I guess this is ready to be merged
Fixes
Fixes #999 by @brenoferreira
Description
Change the main divs in all pages to the
tag for benchmarks.Added aria labels to all footer icons.
Labeled the header as primary and footer as footerlinks for uniqueness.
Change the nested in the to
Technical details
Tests
Screenshots
Checklist
Update index.md
).master
branch of the repository.visible errors.
Developer Certificate of Origin
Developer Certificate of Origin