[wp-trac] [WordPress Trac] #26160: Proposed HTML changes for the post formats metabox in the Add

[WordPress Trac]

#26160: Proposed HTML changes for the post formats metabox in the Add
--------------------------+-----------------------------
 Reporter:  tsiger        |      Owner:
     Type:  enhancement   |     Status:  new
 Priority:  normal        |  Milestone:  Awaiting Review
Component:  Post Formats  |    Version:  trunk
 Severity:  normal        |   Keywords:  has-patch
--------------------------+-----------------------------
 I think there is room for improvement in the markup used to populate the
 post formats list in the metabox which currently looks like:

 [[Image(http://f.cl.ly/items/2s2c3w2q272w0K0I1e2k/Screen%20Shot%202013-11-22%20at%203.40.36%20PM.png)]]

 Instead of using a div as a container and br tags to keep things in place
 as a list I replaced the div with ul and put every post format item within
 a li and now looks like:

 [[Image(http://f.cl.ly/items/1R0T2M203w1F0T241u1v/Screen%20Shot%202013-11-22%20at%203.44.32%20PM.png)]]

 I think it's semantically better now (it's a list of items after all).
 Checked the appearance in all major browsers (RTL too) and looks the same.

 Also it is easier now for someone to target with CSS only a specific post
 format radio button and possible apply some extra styles to its container
 (li).

--
Ticket URL: <http://core.trac.wordpress.org/ticket/26160>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software