#43522 closed enhancement (fixed)
Add aria-current to output of Walker_Page
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 5.1 | Priority: | normal |
Severity: | normal | Version: | 4.8 |
Component: | Themes | Keywords: | dev-feedback has-patch |
Focuses: | accessibility | Cc: |
Description
From the reference of ticket #43191, Also need to add area-current
to output of Walker_Page
.
I think below modifications can help.
if ( $page->ID == $current_page ) {
$atts['aria-current'] = 'page';
}
Attachments (2)
Change History (21)
This ticket was mentioned in Slack in #accessibility by chetan200891. View the logs.
7 years ago
This ticket was mentioned in Slack in #accessibility by afercia. View the logs.
7 years ago
#3
@
7 years ago
- Component changed from General to Themes
- Keywords needs-patch added
- Milestone changed from Awaiting Review to 5.0
- Version trunk deleted
This ticket was mentioned in Slack in #accessibility by afercia. View the logs.
6 years ago
#6
@
6 years ago
- Milestone changed from 5.0 to 4.9.9
Discussed during today's accessibility bug scrub and we'd like to propose this ticket for 4.9.9 consideration.
#10
@
6 years ago
- Keywords 2nd-opinion dev-feedback added
Patch looks fine on my side.
This ticket is triaged in milestone 5.0.3. It will needs commit and backport to the related branch.
This ticket was mentioned in Slack in #core by desrosj. View the logs.
6 years ago
#12
@
6 years ago
- Milestone changed from 5.0.3 to 5.1
Going to punt this to 5.1. It is outside of the scope for 5.0.3 (block
editor bugs, regressions, and major bugs).
#13
@
6 years ago
- Owner set to SergeyBiryukov
- Resolution set to fixed
- Status changed from new to closed
In 44416:
#15
@
4 years ago
- Keywords needs-patch added; has-patch 2nd-opinion removed
- Version set to 4.8
[44416] has an issue: the array key is named aria-current
in the code but referred to as aria_current
in the docs.
Looks like this has a patch, so updating the keywords.