[wp-trac] [WordPress Trac] #34923: Introduce basic content authorship in the Customizer
WordPress Trac
noreply at wordpress.org
Sun Jul 31 14:19:24 UTC 2016
#34923: Introduce basic content authorship in the Customizer
-------------------------------------+-------------------------------------
Reporter: westonruter | Owner: westonruter
Type: enhancement | Status: accepted
Priority: normal | Milestone: Future Release
Component: Customize | Version: 4.3
Severity: normal | Resolution:
Keywords: has-patch ux-feedback | Focuses: ui, accessibility,
needs-testing has-screenshots | javascript
-------------------------------------+-------------------------------------
Comment (by afercia):
tested a bit the last patch and I'd like to thank you for implementing
some accessibility features like audible messages. Noticed a few things
that could be improved though.
1
When creating a new page, after pressing the "Add" button, focus is moved
back to the input field. With some screen readers (e.g. NVDA) this
prevents the message "Menu item added" to be announced because the
announcement of the input field immediately interrupts the previous polite
message. I'd recommend to don't move focus back to the input field.
[[Image(https://cldup.com/A5rXNLEWEU.png)]]
2
Users can add more than one page and when saving the newly added pages get
published all together. However, seems that the `wp.a11y.speak()` message
gets just the last page. See in the screenshot below, where I've added and
published the "Music" and "Movies" pages but just "Movies" is used for the
aria live message. Haven't looked at the code though, it could be that
there are multiple, consecutive, messages and just the last one gets
announced.
[[Image(https://cldup.com/ugOj9R_MxN.png)]]
Also, the message should not contain the last part "Edit it here etc...".
Maybe, announcing the number of published pages would be more useful,
something like "2 new pages published".
3
The published pages get a blue border to give a visual indication they're
"new". It would be great to add also some hidden text or something in the
markup in order to have something in the markup that can be read out by
assistive technologies to announce those pages as "new":
[[Image(https://cldup.com/_uNIgxyR0C.png)]]
4
When opening a new page menu item, there's a notification that uses the
same blue border already used for the menu item. Maybe it's just me but I
find this a bit confusing, maybe UI/design people could give some feedback
here.
The link to edit the page should make sense even when read out of context.
I'd strongly recommend to avoid "here" links. Worth considering in the
admin these "edit" links are expanded with an `aria-label` attribute to
add the page name. Maybe the whole notification text should be reviewed a
bit to highlight the most important thing: the page was just published
'''and it's an empty page'''.
[[Image(https://cldup.com/pMAOuKzMVz.png)]]
--
Ticket URL: <https://core.trac.wordpress.org/ticket/34923#comment:60>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list