[wp-trac] [WordPress Trac] #24932: HTML5 support for specific features doesn't work.

WordPress Trac noreply at wordpress.org
Sat Aug 31 01:20:50 UTC 2013


#24932: HTML5 support for specific features doesn't work.
------------------------------+-----------------------
 Reporter:  nathanrice        |       Owner:  nacin
     Type:  defect (bug)      |      Status:  accepted
 Priority:  normal            |   Milestone:  3.6.1
Component:  Themes            |     Version:  3.6
 Severity:  normal            |  Resolution:
 Keywords:  has-patch commit  |
------------------------------+-----------------------

Comment (by nacin):

 [attachment:24932.diff] makes two functionality changes:
  * Calling `add_theme_support( 'html5', array( 'comments-list' ) );
 add_theme_support( 'html5', array( 'comment-form' ) );` should
 '''merge'''. If a theme wants to ''replace'', they need to do a
 `remove_theme_support( 'html5' )` first. This is a change in behavior that
 we can really only get away with now, in 3.6.1. Reasoning is the mess that
 is #22080.

  * `add_theme_support( 'html5' )` (no arguments) no longer works. This was
 never supposed to work, as it would prevent us from adding future types in
 the future. `current_theme_supports( 'html5' )`, however, does continue to
 work, and will return `true` if *any* feature is supported. This might
 help a plugin author decide what markup to serve, but core should never
 check for this.

 In both cases, this is how the functionality was designed. It just wasn't
 implemented correctly.

 Also adds some unit tests.

--
Ticket URL: <http://core.trac.wordpress.org/ticket/24932#comment:4>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list