WordPress.org

Make WordPress Core

Opened 3 years ago

Closed 3 years ago

#16224 closed defect (bug) (fixed)

format-standard class unnecessary for pages

Reported by: iandstewart Owned by:
Milestone: 3.1 Priority: normal
Severity: normal Version: 3.1
Component: General Keywords: has-patch
Focuses: Cc:

Description

If post formats are meant only for posts it seems unnecessary to include format-standard in the post classes for pages.

Attachments (2)

post-format-classes.diff (515 bytes) - added by iandstewart 3 years ago.
post-format-classes-2.diff (758 bytes) - added by iandstewart 3 years ago.

Download all attachments as: .zip

Change History (6)

comment:1 follow-up: nacin3 years ago

  • Milestone changed from Awaiting Review to 3.1

We should probably instead use if ( post_type_supports( $post->post_type, 'post-formats' ) ).

comment:2 in reply to: ↑ 1 westi3 years ago

Replying to nacin:

We should probably instead use if ( post_type_supports( $post->post_type, 'post-formats' ) ).

Agreed

comment:3 iandstewart3 years ago

Updated patch to exclude any post type that doesn't use post formats.

comment:4 nacin3 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [17292]) Only add the post format post classes when the post type supports post formats. posts posts posts posts. props iandstewart, fixes #16224.

Note: See TracTickets for help on using tickets.