[wp-trac] [WordPress Trac] #35210: Add notification area to Customizer
WordPress Trac
noreply at wordpress.org
Thu Sep 1 08:26:28 UTC 2016
#35210: Add notification area to Customizer
-------------------------------------------------+-------------------------
Reporter: westonruter | Owner:
Type: enhancement | Status: new
Priority: normal | Milestone: 4.7
Component: Customize | Version:
Severity: normal | Resolution:
Keywords: needs-patch ui-feedback ux-feedback | Focuses: ui,
| javascript
-------------------------------------------------+-------------------------
Comment (by afercia):
Replying to [comment:81 westonruter]:
> [[Image(https://core.trac.wordpress.org/raw-attachment/ticket/35210
/rough-global-notifications-mock.png, width=300)]]
From an accessibility perspective, I'm more concerned about the order of
elements in the markup. Ideally, the control that toggles the
notifications list should be immediately followed by the... notifications
list, meaning there should be no other elements in between.
I'd also recommend to consider a long standing issue in this top part of
the Customizer sidebar, again about the order of controls in the source
which is:
- Save (& Publish) button
- Preview/Customize toggle (only in the responsive view)
- Close X link (this one receives the initial focus)
Since the initial focus goes to the X link, the notification red circle
would be "skipped" for keyboard and screen reader users and very hard to
find. Currently, this happens with the Save button. Users need to tab
backwards in order to find it, but their starting point is the X link so
why they should even think they have to do so? I'd agree this is related
but also a separate issue and should probably go in a separate ticket :)
Other minor things for accessibility would be using aria-label and aria-
expanded on the toggle control (and it needs to be a button), these are
not a big deal I guess.
Lastly, I'd recommend to carefully consider if there's enough available
space. Especially in the responsive view, where the Preview/Customize
toggle appears, and considering translations could use longer strings, I'm
not so sure there's enough space. There's also a spinner there :)
[[Image(https://cldup.com/pAGVBgaQIs.png)]]
--
Ticket URL: <https://core.trac.wordpress.org/ticket/35210#comment:82>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list