[wp-trac] [WordPress Trac] #36638: Detect broken URLs in the editor
WordPress Trac
noreply at wordpress.org
Wed Jun 29 08:32:33 UTC 2016
#36638: Detect broken URLs in the editor
-------------------------------------------------+-------------------------
Reporter: iseulde | Owner: azaozz
Type: task (blessed) | Status: assigned
Priority: normal | Milestone: 4.6
Component: Editor | Version:
Severity: normal | Resolution:
Keywords: needs-unit-tests needs-testing | Focuses: ui
needs-patch |
-------------------------------------------------+-------------------------
Comment (by afercia):
Replying to [comment:24 ocean90]:
> * The title attribute might be not the best idea in terms of a11y. Maybe
we should use `wp.a11y.speak()`?
Yep I'd recommend to don't add new title attributes. From an a11y
perspective when the link is broken there's no audible feedback and a
barely noticeable visible feedback (the title attribute "tooltip").
To get the error message, screen reader users should (unlikely) go through
the following process:
- after the link is inserted, press Alt+F8 to move focus to the link
toolbar
- explore the link toolbar using the arrow keys (not the tab key since the
displayed link is not focusable)
- finally, only when the screen reader is reading out the link displayed
in the toolbar they would get the error message in the title attribute,
see below:
[[Image(https://cldup.com/Ek5r3dTHfX.png)]]
I'd consider @ocean90 suggestion about using `wp.a11y.speak()` to convey
an audible warning message.
For sighted users also, the title attribute is barely noticeable and not
so useful on touch devices. It would be great to try to find a good idea
for a better visual feedback.
About colors, worth reminding the Editor background may be styled, for
example:
[[Image(https://cldup.com/OoCWmaYS2Q.png)]]
Lastly, I'd suggest to review the error messages wording. As an average
users I'd be probably be confused by a message saying something about
"host name" or "HTTP error 404". I'd just need to know the link is broken
:) As far as I see, the current error messages are:
{{{
'Invalid host name.'
'Not found, HTTP error 404.'
}}}
Maybe using a generic, simpler, sentence followed by the "technical" error
code could work?
--
Ticket URL: <https://core.trac.wordpress.org/ticket/36638#comment:25>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list