Customizer: remove redundant aria-label

[afercia]

Looks like this aria-label is no more necessary:

that div is no more focusable, there's no way screen readers can read out its aria-label, unless I'm missing something. Maybe a leftover from 4.2 when it was focusable and used to display the section description when clicked.

[afercia]

In the proposed patch:

• Customizer: remove no more necessary aria-label attribute from main section title

[afercia]

One more unnecessary aria-label attribute (and tabindex):

This one comes from render_screen_options() in screen.php. Reusing things is a good thing of course but both the aria-label and tabindex attribute used in render_screen_options() are strictly tied to the expandable panels used in the main admin screens and here in the Customizer are unnecessary. Open to suggestions :)

[westonruter]

Additional change: ​https://github.com/xwp/wordpress-develop/commit/d30fd7eb18d1e65b2a3603e3638c699a7980c977

[westonruter]

@afercia Take a look at 33182.2.diff​. It introduces an option to render_screen_options() to disable the wrap element. The option is true by default, but in the Customizer we can set it to false and supply our own wrapper.

[afercia]

@westonruter: makes sense to me :)

Side note: testing the Screen Options and Help panels, looks like NVDA and JAWS don't read out the panels aria-label attributes, respectively: "Screen Options Tab" and "Contextual Help Tab". Only ChromeVox does. I think they would need some ARIA roles. Also, wondering if they're really useful. By the way this is out of the scope of this ticket. Planning to do some more testing and maybe open a new ticket, looks like this is an admin area where there's room for improvements, see #21583 and the original accessibility related ticket #21326.

In the screenshot below: the aria-label doesn't get announced (NVDA)

[westonruter]

@wonerboymusic: Any concerns with 33182.2.diff​ which adds an $options argument to WP_Screen:: render_screen_options ()?

[afercia]

Replying to westonruter:

@wonerboymusic: Any concerns with 33182.2.diff​ which adds an $options argument to WP_Screen:: render_screen_options ()?

@wonderboymusic ! just to trigger the notification :)

[westonruter]

In 34093:

Customize: Remove redundant aria-label attributes.

Adds an $options array argument to WP_Screen::render_screen_options() to allow the div#screen-options-wrap element to be omitted when wrap value is false.

Props afercia, westonruter.
Fixes #33182.