Repository navigation
Conversation
On the Network Setup screen, associate the "Network Title" and "Network Admin Email" descriptions with their inputs via `aria-describedby`. On the My Sites screen, change the site and "Global Settings" headings from `h3` to `h2` so the heading hierarchy doesn't skip a level. `h2` and `h3` share the same admin styles, so there is no visual change. See #66205.
|
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the Core Committers: Use this line as a base for the props when committing in SVN: To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
Test using WordPress PlaygroundThe changes in this pull request can previewed and tested using a WordPress Playground instance. WordPress Playground is an experimental project that creates a full WordPress instance entirely within the browser. Some things to be aware of
For more details about these limitations and more, check out the Limitations page in the WordPress Playground documentation. |
Change the section headings from h3 to h2 so they follow the page h1 without skipping a level. The elements share the same admin styles, so this has no visual impact. See #66205.
3c93126 to
c972a00
Compare
|
I added a follow-up commit for the additional heading-level issue found on the Network Setup page. The four section headings rendered across the two setup steps—“Addresses of Sites in your Network,” “Server Address,” “Network Details,” and “Enabling the Network”—now use I also ran the focused PHP coding standards check for |
Improves the accessibility of two network/multisite related admin screens: * My sites: Converts site name and "Global Settings" (available via filter) headings to `h2` to avoid skipping a heading level * Network setup: - associates fields with their descriptive text with `aria-describedby` attributes - converts section headings to `h2` to avoid skipping a heading level Developed in #13819 Props mitgiselle, johnjamesjacoby, afercia. Fixes #66205. git-svn-id: https://develop.svn.wordpress.org/trunk@64013 602fd350-edb4-49c9-b593-d223f7449a82
Improves the accessibility of two network/multisite related admin screens: * My sites: Converts site name and "Global Settings" (available via filter) headings to `h2` to avoid skipping a heading level * Network setup: - associates fields with their descriptive text with `aria-describedby` attributes - converts section headings to `h2` to avoid skipping a heading level Developed in WordPress/wordpress-develop#13819 Props mitgiselle, johnjamesjacoby, afercia. Fixes #66205. Built from https://develop.svn.wordpress.org/trunk@64013 git-svn-id: http://core.svn.wordpress.org/trunk@63173 1a063a9b-81f0-0310-95a4-ce76da25c4cd
Trac ticket: https://core.trac.wordpress.org/ticket/66205
Summary
Minor accessibility fixes on the network admin screens, as described in the ticket and follow-up review.
Network Setup (
network.php)The "Network Title" and "Network Admin Email" fields have description text that isn't programmatically associated with the inputs. This adds an
aria-describedbyon each input, pointing at anidon its description paragraph (sitename-descandemail-desc), matching the pattern used elsewhere in wp-admin.The page's
h1is also followed byh3section headings in both Network Setup steps. The "Addresses of Sites in your Network", "Server Address", "Network Details", and "Enabling the Network" headings are nowh2so the hierarchy doesn't skip a level.My Sites (
my-sites.php)The page's
h1is followed byh3headings, which skips a level. The per-site heading and the conditional "Global Settings" heading are nowh2. The ticket asks for the CSS to be adjusted so the font size doesn't change, buth2andh3share the same rule incommon.css(font-size: 1.3em; margin: 1em 0), so no CSS change is needed. Nothing else in the admin stylesheets targets.my-sites h3.Testing
Tested locally with the wordpress-develop Docker environment.
Network Setup
define( 'WP_ALLOW_MULTISITE', true );towp-config.php. Don't do this on a production site.aria-describedbythat matches theidof its description.h1, thenh2section headings, with no skipped levels.h2and anh3in the same container. Font size, margin, weight and colour are identical, so there's no visual change.My Sites
wp-admin/my-sites.php.h1, thenh2for each site, with no skipped levels.myblogs_optionsfilter and return some HTML when the context isglobal. It renders as anh2.h2and anh3in the same container. Font size, margin, weight and colour are identical, so there's no visual change.phpcspasses on both files. No tests were added, as these are markup-only changes and no existing tests cover these screens.