Opened 2 years ago

Closed 2 years ago

#16224 closed defect (bug) (fixed)

format-standard class unnecessary for pages

Reported by: iandstewart Owned by:
Priority: normal Milestone: 3.1
Component: General Version: 3.1
Severity: normal Keywords: has-patch
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 2 years ago.
post-format-classes-2.diff​ (758 bytes) - added by iandstewart 2 years ago.

Download all attachments as: .zip

Change History (6)

iandstewart — 2 years ago

comment:1 follow-up: ↓ 2   nacin — 2 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   westi — 2 years ago

Replying to nacin:

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

Agreed

iandstewart — 2 years ago

comment:3   iandstewart — 2 years ago

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

comment:4   nacin — 2 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.