as stated in the title, this is my very first design/build of a Wordpress powered site: http://www.alliwagner.com/headleysite.jpg http://www.headleycommunity.org
Any and all feedback/criticism is greatly appreciated.
One thing I know I need to take care of off the bat is I need to figure out a way to add a link to the "blog" section in the navigation (which are actually just the pages styled). I havent been able to figure this out yet (my only solution has been to link the entire header to the index where the blog resides)
First Wp site? I am genuinely impressed, a rather regal look.
Ok, the first thing you should do is configure your permalinks /?cat=3 is not very search engine friendly or particularly helpful to users. You can do that from the wp admin panel under settings - permalinks, hit the radio button for custom structure and type something like /%postname%/
As regards adding a link to the blog, I presume you are using wp_list_pages for your nav? If so, create a new page (not post) name it "Blog" (or whatever you want to call your blog eg. "news", "latest" etc.) leave the rest blank and publish it. Now go back to settings - reading and there where it asks for the posts page, select your newly created page from the dropdown, save changes and presto it will now appear in the nav list.
I would suggest some kind of rollover effect for the main nav items and something to highlight the current page, even if it's only an underline. Perhaps make the footer stick to the bottom of the window. It looks great on the long pages but not so hot on the shorter ones when it appears half way up the screen.
There should be a backslash after %postname% as well as before. Here is a pretty good article on wp permalinks http://www.homebizpal.com/blogging/wordpress/understanding-wordpress-permalinks/ otherwise check that you have a .htaccess file and that it is write-able.
Ok, I've just run your xhtml through the W3 validator http://validator.w3.org/#validate_by_input. You have 4 errors on your home page and 9 on your single post pages so you might want to address those issues first.
I also noticed that your sidebar div is sitting inside your comments-container. As is your footer text which should surely be inside your footer div?
sidebar div weirdness fixed- check footer div weirdness fixed- check permalink restructured- check rollovers for nav- check (although I do want to rework them so they are all the same width)
still need the darn code to validate- boo still need to have current page highlighted- double boo (athough I saw a plugin offered that may make my life easier)
One last thing-- my "entries-nav" links are not working (newer entries and older entries nav) here is what my code looks like :
Try putting the "wp-credit" div above the the "footer-wrap" div but keep it inside the "footer".
You really need to fix those xhtml errors before you can be sure about your sticky footer. You have at least one extra closing div in there. Your sidebar is still inside the "comments-container" on the single posts page. So you need to fix index.php, page.php (if you are using it) and single.php. And validate your code!
Re. "entries-nav" Why the ul? Those span tags around the pipe character can't go between two list items, thats one of your errors right there. Better make the whole thing a paragraph and stick a line break in there if you want them on different lines. This should make the links work:
<?php /* This is commented, because it requires a little adjusting sometimes. You'll need to download this plugin, and follow the instructions: http://binarybonsai.com/archives/2004/08/17/time-since-plugin/ */ /* $entry_datetime = abs(strtotime($post->post_date) - (60*120)); echo time_since($entry_datetime); echo ' ago'; */ ?> <?php the_time('l, F jS, Y') ?> at <?php the_time() ?><br /> <?php the_category(', ') ?>
<?php if (('open' == $post-> comment_status) && ('open' == $post->ping_status)) { // Both Comments and Pings are open ?> <a href=\"#respond\">Comment</a>
<?php } elseif (!('open' == $post-> comment_status) && ('open' == $post->ping_status)) { // Only Pings are Open ?> Responses are currently closed, but you can <a href=\"<?php trackback_url(); ?> \" rel=\"trackback\">trackback</a> from your own site.
<?php } elseif (!('open' == $post-> comment_status) && !('open' == $post->ping_status)) { // Neither Comments, nor Pings are open ?> Both comments and pings are currently closed.
<?php } edit_post_link('Edit this entry','','.'); ?> </div> </p>
as stated in the title, this is my very first design/build of a Wordpress powered site:
http://www.alliwagner.com/headleysite.jpg
http://www.headleycommunity.org
Any and all feedback/criticism is greatly appreciated.
One thing I know I need to take care of off the bat is I need to figure out a way to add a link to the "blog" section in the navigation (which are actually just the pages styled). I havent been able to figure this out yet (my only solution has been to link the entire header to the index where the blog resides)
Thanks for your time in advance friends!
Allison
Ok, the first thing you should do is configure your permalinks /?cat=3 is not very search engine friendly or particularly helpful to users. You can do that from the wp admin panel under settings - permalinks, hit the radio button for custom structure and type something like /%postname%/
As regards adding a link to the blog, I presume you are using wp_list_pages for your nav? If so, create a new page (not post) name it "Blog" (or whatever you want to call your blog eg. "news", "latest" etc.) leave the rest blank and publish it. Now go back to settings - reading and there where it asks for the posts page, select your newly created page from the dropdown, save changes and presto it will now appear in the nav list.
I would suggest some kind of rollover effect for the main nav items and something to highlight the current page, even if it's only an underline.
Perhaps make the footer stick to the bottom of the window. It looks great on the long pages but not so hot on the shorter ones when it appears half way up the screen.
All in all, nice job ;)
I set it up for /%postname% as suggested, however when I do this the site falls apart
How do I go back and set names (or slugs I guess?) for each page that serves as the sites navigation etc?
The names of the pages come from the page titles.
Hope that helps.
I also noticed that your sidebar div is sitting inside your comments-container. As is your footer text which should surely be inside your footer div?
I'll address all those issues today!
Your time is so very appreciated :)
footer div weirdness fixed- check
permalink restructured- check
rollovers for nav- check (although I do want to rework them so they are all the same width)
still need the darn code to validate- boo
still need to have current page highlighted- double boo (athough I saw a plugin offered that may make my life easier)
One last thing-- my "entries-nav" links are not working (newer entries and older entries nav)
here is what my code looks like :
also, I implemented a sticky footer technique but unfortunately my moniter is too small to check-- is it working? :)
thanks!
You really need to fix those xhtml errors before you can be sure about your sticky footer. You have at least one extra closing div in there. Your sidebar is still inside the "comments-container" on the single posts page. So you need to fix index.php, page.php (if you are using it) and single.php. And validate your code!
Re. "entries-nav" Why the ul? Those span tags around the pipe character can't go between two list items, thats one of your errors right there. Better make the whole thing a paragraph and stick a line break in there if you want them on different lines.
This should make the links work:
<?php previous_post_link('« %link') ?><?php next_post_link('%link »') ?>
Check your nav in Firebug. Wordpress does it for you.
Just add the CSS.
I went through index.php, single.php, and page.php and I couldn't find anything (but I also dont really know php so I'm not too sure what to look for)
page.php:
#date {padding: 0px 0 5px 0;
font: italic 14px Georgia, \"Times New Roman\", Times, serif;
}
single.php:
index.php:
sweet I will!
Try switching these two lines :
So that it reads:
and your single post page is still not validating.