![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png){kind=logo}
Opened 7 years ago
Closed 7 years ago
#47390 closed defect (bug) (fixed)
Improve accessibility of forms elements within some "form-table" forms
| Reported by: | afercia | Owned by: | afercia |
|---|---|---|---|
| Priority: | normal | Milestone: | 5.2.3 |
| Component: | Administration | Version: | |
| Severity: | normal | Keywords: | has-patch has-screenshots needs-testing fixed-major |
| Cc: | Focuses: | accessibility, multisite, privacy |
Description
Follow-up to #46899.
In #46899, an attribute role="presentation" was added to most of the form-table used in the admin pages. These tables contain forms and in most of the cases they are used just for layout purposes. The role="presentation" helps in removing the table semantics where appropriate.
Note: this should be considered a temporary improvement, pending decision on the proposal to entirely remove layout tables from WordPress. See #16413, #18801, #39441, and the explorations made on https://github.com/wpaccessibility/settings-api-enhanced.
That said, there are a few cases where the semantics of these forms within tables can be improved. I'm working on a patch to address these cases.
Attachments (5)
Change History (11)
Note:
See TracTickets
for help on using tickets.
47390.diff does the following:
role="presentation"to the edit comment layout table<fieldset>elements<label>elements, especially on some network admin pagesclass="td-full"to table rows spanning to multiple cellsscope="row"attribute to the table headers inoptions-permalink.php: this table is better communicated as data tablearia-describedbyuser-edit.phppageRe: the "Privacy Settings" page, the select label should be the bold text in the left column. I'd like to propose to remove
Select an existing page:and just use the left column text as the<label>, also for consistency with all the similar settings tables in the admin. See attached screenshots.