-- Problems --
Next/Previous post links rendered with no link or text if there was no next/previous post and with a doubled outer border around the link that existed if one was missing.
Functions previous_post_link(), next_post_link() echo the link (don't return a value--that took an hour or two to discover), so are unsuitable for retrieving the link and testing the value.
-- Changes --
Used get_previous_post()/get_next_post() to detect if prev/next post exists. Only include prev/next links if corresponding post exists.
Used get_permalink() to get href to avoid issues with previous_post_link() and next_post_link(). Also makes the HTML easier to read and modify IMO.
Simplified container structure to avoid double border.
Put Previous on the left, Next on the right. Most languages are left-to-right. In LTR languages, previous "page" is on the left. This agrees with the original arrow directions which pointed left for Previous and right for Next.
Moved right-arrow on next to right edge of link. (Now <-Previous Next-> instead of <-Previous ->Next.)
Added pull-left, pull-right to links. (Looks better, IMO.)
-- Note to Konstantin --
For my child theme, I plan to generate prev/next in a function called from single.php. I want to show the post title instead of "Next"/"Previous" and need to support "cousins" navigation (move to prev/next under a common ancestor category--similar to what the next_page_not_next_post plugin does).
My first step will be to replicate the existing behavior in a function. I can update code here with that (so child themes can override the function in functions.php instead of copying and modifying single.php). Let me know if you want me to update git when I have that working. Also, let me know if you have a preference for where the function goes (functions.php, somewhere in ./inc, etc.).
(PS: This theme is the only reason I have a user id on git. ^_^ )
-- Problems -- Next/Previous post links rendered with no link or text if there was no next/previous post and with a doubled outer border around the link that existed if one was missing.
Functions previous_post_link(), next_post_link() echo the link (don't return a value--that took an hour or two to discover), so are unsuitable for retrieving the link and testing the value.
-- Changes -- Used get_previous_post()/get_next_post() to detect if prev/next post exists. Only include prev/next links if corresponding post exists.
Used get_permalink() to get href to avoid issues with previous_post_link() and next_post_link(). Also makes the HTML easier to read and modify IMO.
Simplified container structure to avoid double border.
Put Previous on the left, Next on the right. Most languages are left-to-right. In LTR languages, previous "page" is on the left. This agrees with the original arrow directions which pointed left for Previous and right for Next.
Moved right-arrow on next to right edge of link. (Now <-Previous Next-> instead of <-Previous ->Next.)
Added pull-left, pull-right to links. (Looks better, IMO.)
-- Note to Konstantin -- For my child theme, I plan to generate prev/next in a function called from single.php. I want to show the post title instead of "Next"/"Previous" and need to support "cousins" navigation (move to prev/next under a common ancestor category--similar to what the next_page_not_next_post plugin does).
My first step will be to replicate the existing behavior in a function. I can update code here with that (so child themes can override the function in functions.php instead of copying and modifying single.php). Let me know if you want me to update git when I have that working. Also, let me know if you have a preference for where the function goes (functions.php, somewhere in ./inc, etc.).
(PS: This theme is the only reason I have a user id on git. ^_^ )