Opened 2 years ago

Closed 2 years ago

#15891 closed enhancement (wontfix)

TwentyTen: Show navigation <div> when needed

Reported by: zeo Owned by:
Priority: normal Milestone:
Component: Themes Version:
Severity: normal Keywords: has-patch 2nd-opinion
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 2 years ago.

Download all attachments as: .zip

Change History (5)

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

zeo — 2 years ago

comment:2   zeo — 2 years ago

Refresh.

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

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

comment:3   nacin — 2 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   nacin — 2 years ago

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