WordPress.org

Make WordPress Core

Opened 4 years ago

Closed 4 years ago

#15891 closed enhancement (wontfix)

TwentyTen: Show navigation <div> when needed

Reported by: zeo Owned by:
Milestone: Priority: normal
Severity: normal Version:
Component: Themes Keywords: has-patch 2nd-opinion
Focuses: Cc:

Description

Show navigation <div> only when there's previous/next post (single view), attachment count more than 1 (attachment view).

Attachments (1)

15891.diff (4.0 KB) - added by zeo 4 years ago.

Download all attachments as: .zip

Change History (5)

comment:1 @nacin4 years ago

A naked get_adjacent_post() just looks for the previous post.

This seems unnecessary. Many people might style the site using the navigation (for example, clearing, borders, etc.) and very rarely will it ever be empty for both previous/next. Might be good for future default themes, but not now with custom CSS and child themes.

@zeo4 years ago

comment:2 @zeo4 years ago

Refresh.

I agree it's a rare case. Just sharing. It's your call.

Version 0, edited 4 years ago by zeo (next)

comment:3 @nacin4 years ago

  • Keywords 2nd-opinion added; dev-feedback removed
  • Type changed from defect (bug) to enhancement

Yes, for multiple pages. Not for previous/next.

comment:4 @nacin4 years ago

  • Milestone Awaiting Review deleted
  • Resolution set to wontfix
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.