#3237 closed defect (bug) (fixed)
Use wp_link_pages() in index.php and archive.php
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 1.5 | Priority: | normal |
Severity: | Version: | 1.5 | |
Component: | Templates | Keywords: | |
Cc: | hugoashmore@… |
Description
Review feedback: "wp_link_pages should also be included the following template files:
- index.php
- archive.php"
http://codex.wordpress.org/Theme_Development#Template_File_Checklist
Attachments (3)
Change History (8)
#2
@
13 years ago
Files patched as per example SC.
Default-ish wp_link_pages() args passed as in 2010 i.e before => <div class="page-links">
No styling applied class .page-links acquires existing bp-default styles for:
.edit-link, .page-links { clear: both; }
As this is a list of links the markup really should reflect that by using a ul construct. On brief perusal of codex I don't think it's possible to pass 'li' element args easily if at all, but will recheck but it would leave an issue with what to do with anonymous inline text box ( the bear naked cdata 'Pages' existing in the div)
#3
@
13 years ago
Thanks for the patch, hnla!
I've taken the <strong> out and put it into CSS, and I've found where we've used wp_link_pages() in some of the existing templates, and added the same markup so it's identical across all the templates.
Adds wp_link_pages() to index.php & archive.php