[wp-trac] [WordPress Trac] #29699: add_theme_support( 'screen-reader-text' );

[WordPress Trac]

#29699: add_theme_support( 'screen-reader-text' );
-------------------------+----------------------------
 Reporter:  GaryJ        |       Owner:
     Type:  enhancement  |      Status:  closed
 Priority:  normal       |   Milestone:
Component:  Themes       |     Version:
 Severity:  normal       |  Resolution:  wontfix
 Keywords:  has-patch    |     Focuses:  accessibility
-------------------------+----------------------------
Changes (by nacin):

 * status:  new => closed
 * resolution:   => wontfix
 * milestone:  Awaiting Review =>


Comment:

 I'm going to close this out. After reading the whole thread, I think
 obenland makes some excellent points, including:

 > I would suggest to start adding the class in a place that is not very
 visible and start beating the drum with the release of 4.2, doing that
 theme author education that you agreed we should do. We can then continue
 to add it in more visible places over time.

 > Also, using theme support to denote support for a specific class feels
 like using a sledgehammer to crack a nut.

 Adding it to theme check and making it required in the theme directory
 sounds good. If we find later that we need something akin to the proposal
 here to declare that the theme supports things, due to the impact it may
 have, then we can reconsider it. But some relevant text showing up on a
 theme is by itself probably not the end of the world.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/29699#comment:33>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform