Thanks for all the feedback – I’ll address a few points;
Quote:
I like it. I like the header graphic. Nice trees. did you make that? The only part I am not sure of is the main content. there is alot of text there, and the font size might be alittle small with that background color. The black text blends in alittle with that color. So a bigger font might be better. There needs to be more pictures. Even if they are just decoration. for example. Look at the header of this site. There is a tag on the left. It doesn’t really have anything to do with css, It’s decoration. But it ads to the site very well. Overall I like you design. I like the navigation as well.
Yeah I made the header, the trees are actually very simple. I didn’t make the actual logo, but I did play with the colors/textures, its been their running logo for years now.
I see what you mean about the font, for usuability purposes I think I should definitely bump it up a size or two.
All that is just placeholder text, obviously, the real main page will have a picture or two with a fancy border or something. However, I don’t think I’ll be adding any pictures to the main, page-to-page design.
Quote:
I think this is a wonderful design. I love the colors, the illustrations and the entire layout. Don’t add more pictures. You have a perfect balance as it is. Everything is so clean, and well organized, very very nice! Only suggestion I have is run your code through the validator. Your HTML has a few errors – you have your sidebar formated with the link around the image and h3, which isn’t valid. I suggest tweaking it and putting that sidebar into a list, and it have formatted like:
I realize that you probably are using the h3 for SEO purposes, but the strong tag is a good SEO keyword highlighter too (maybe not as much as an h3) but for the sake of code validity I’d rather be standards compliant.
Also, you’ve used the div id "footer-wrap" twice. You can only use an id once per page. I’m sure you already know that though ;P
I’m also wondering why you chose to break your CSS up into so many serparate files? I realize each has it’s own distinct place/styleset for the page, but each is so short, it seems it would be easier to contain all into one file and just properly comment where in the file the section is located.
But overall, I really like it! This is one of a best designs I think I’ve seen recently.
Thanks for looking through my code! I wasn’t quite finished so I haven’t run it through a validator, but thanks for the tip with that menu, I don’t know why I didn’t make it a list in the first place. Can I still make the entire element a clickable block that works cross-browser?
That wrap id was just an accident, nice catch.
About the CSS files – I built each peice of the site independently, so it usually had its own stylesheet or two with it. Plus, I like the organized aspect, its real easy to find any element. Also, its real easy to manage a whole folder of stylesheets in textmate.
Quote:
Just out of curiosity, for the menu bar, have you tried having all the hovers actually change color throughout the entire menu bar instead of merging half way down?
Are you talking about the right side navigation? The background highlight is the same on each section, just goes to white, but the gradient makes it look more pronounced at the top.
Too bad this is the only thing I have lined up, and I’ll be done with it soon.
/recently unemployed web designer
section title
