#43522 closed enhancement (fixed)
Add aria-current to output of Walker_Page
| Reported by: | chetan200891 | Owned by: | SergeyBiryukov |
|---|---|---|---|
| Priority: | normal | Milestone: | 5.1 |
| Component: | Themes | Version: | 4.8 |
| Severity: | normal | Keywords: | dev-feedback has-patch |
| Cc: | Focuses: | accessibility |
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.
8 years ago
This ticket was mentioned in Slack in #accessibility by afercia. View the logs.
8 years ago
#3
@
8 years ago
- Component General → Themes
- Keywords needs-patch added
- Milestone Awaiting Review → 5.0
- Version trunk
This ticket was mentioned in Slack in #accessibility by afercia. View the logs.
8 years ago
#6
@
8 years ago
- Milestone 5.0 → 4.9.9
Discussed during today's accessibility bug scrub and we'd like to propose this ticket for 4.9.9 consideration.
#10
@
8 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.
8 years ago
#12
@
8 years ago
- Milestone 5.0.3 → 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).
#15
@
5 years ago
- Keywords needs-patch added; has-patch 2nd-opinion removed
- Version → 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.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
Looks like this has a patch, so updating the keywords.