[wp-trac] [WordPress Trac] #31326: Edit comment screen: misplaced-missing labels
WordPress Trac
noreply at wordpress.org
Sat Mar 21 21:29:57 UTC 2015
#31326: Edit comment screen: misplaced-missing labels
-----------------------------------+--------------------------------
Reporter: afercia | Owner:
Type: defect (bug) | Status: new
Priority: normal | Milestone: 4.2
Component: Comments | Version: 4.1
Severity: normal | Resolution:
Keywords: has-patch ui-feedback | Focuses: ui, accessibility
-----------------------------------+--------------------------------
Changes (by DrewAPicture):
* keywords: has-patch => has-patch ui-feedback
Comment:
Replying to [comment:7 afercia]:
> Replying to [comment:5 rianrietveld]:
> > Jeff de Wit
> > Labels for those fields read out fine in Chrome. The links are a bit
context-less now though. While I personally understand the connection
between the Email field and the "send email" link (and similarly, the URL
field and the "visit site" link), not everyone might while just listening
to it as they tab through ("Send email to where?"). Maybe include the
email address in the link (as screen reader text or otherwise)?
> > Not too sure what to do about the visit site link, especially since
it'll take you to the commenter's website (whatever it is they've filled
in). It seems redundant to read it out loud again, but maybe refer back to
the URL field?
>
> Maybe just remove them? Need feedback from the UI team.
I'd also be interested to hear from the UI team on just removing those
links, the email link is just a mailto link and visit site link is I
suppose nice-to-have though odd, especially in the context of having to
move it below the input to fix the labels.
If we decide to keep the links, we should camel-case the text now that
they've lost their context of being part of the pseudo-label.
* See [attachment:31326_no_links.png] for what it would look like with the
links removed.
* See [attachment:31326_casing.png] for what it would look like with the
link text camel-cased.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/31326#comment:9>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list