WordPress.org

Make WordPress Core

Opened 5 months ago

Last modified 5 months ago

#26160 new enhancement

Proposed HTML changes for the post formats metabox

Reported by: tsiger Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version: 3.7
Component: Post Formats Keywords: has-patch
Focuses: Cc:

Description

I think there is room for improvement in the markup used to populate the post formats list in the metabox which currently looks like:

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:

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).

Attachments (1)

meta-boxes.php.patch (2.0 KB) - added by tsiger 5 months ago.

Download all attachments as: .zip

Change History (3)

tsiger5 months ago

comment:1 tsiger5 months ago

  • Summary changed from Proposed HTML changes for the post formats metabox in the Add to Proposed HTML changes for the post formats metabox

comment:2 SergeyBiryukov5 months ago

  • Version changed from trunk to 3.7
Note: See TracTickets for help on using tickets.