Inconsistent use of (next|previous)_post_link and (next|previous)_posts_link functions in default themes
|Reported by:||solarissmoke||Owned by:||iandstewart|
There is an inconsistency in the way the functions next_post_link()/previous_post_link() (used in single.php) and next_posts_link()/previous_posts_link() (used in index.php and archive.php) are used in the default themes.
The current behaviour (in the default themes) is as follows:
- next_post_link() and previous_post_link() generate an anchor tag with post title plus either » or « outside the tag.
- next_posts_link() and previous_posts_link() generate an anchor tag with post title and » or « inside the anchor tag.
Since these functions are all used in the same visual space (div.navigation) the position of the »/« (inside out outside of the anchor tag) should be consistent. Personally I prefer having the additional text inside the anchor tag.
It just requires changing the arguments supplied to the functions in the themes. Proposed patch to follow.
Change History (9)
comment:1 @solarissmoke — 5 years ago
- Summary changed from Inconsistent link generation between (next|previous)_post_link and (next|previous)_posts_link functions to Inconsistent use of (next|previous)_post_link and (next|previous)_posts_link functions in default themes
comment:4 @solarissmoke — 5 years ago
- Cc solarissmoke removed
- Keywords has-patch removed
- Milestone 3.1 deleted
- Resolution set to wontfix
- Status changed from new to closed
- Component changed from Themes to WordPress.org
- Milestone set to WordPress.org site
- Resolution wontfix deleted
- Status changed from closed to reopened