[wp-trac] [WordPress Trac] #25459: Provide more meaningful links in Edit Post/Page
WordPress Trac
noreply at wordpress.org
Tue Dec 10 15:10:41 UTC 2013
#25459: Provide more meaningful links in Edit Post/Page
-----------------------------------------------+---------------------------
Reporter: grahamarmfield | Owner:
Type: enhancement | Status: new
Priority: low | Milestone: Future
Component: Accessibility | Release
Severity: minor | Version: 3.6.1
Keywords: i18n-change has-patch 2nd-opinion | Resolution:
-----------------------------------------------+---------------------------
Comment (by grahamarmfield):
Replying to [comment:44 neil_pie]:
> Taking into account some of the discussion here, I've provided an
advancement on the existing patch. This patch:
>
> * Tries to remove any notion of 'short' vs. 'long' text and keeps it in
the context of 'display on screen' vs 'present to screen readers'
Good idea
> * Adds some escaping to the HTML that we return
Good idea
> * Removes the $screen_reader_class as a parameter to be passed in and
adds it as a parameter that can be filtered
Isn't filtering a bit overkill? The direct parameter that defaults to
'screen-reader-text' is easier to use.
> * Suggests yet another function name, removing references to
'short','long' and 'a11y'
OK
> * Includes PHP Doc in accordance with
http://make.wordpress.org/core/handbook/inline-documentation-standards
/php-documentation-standards/
>
> I think it's important to be as broad as possible about what the
function is doing. While it's a real solution to an a11y problem, the
specifics of the function just provide the appropriate HTML to display one
text string on the screen and another to screen readers.
OK
--
Ticket URL: <http://core.trac.wordpress.org/ticket/25459#comment:47>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list