Printable Version of Topic

Click here to view this topic in its original format

HTMLHelp Forums _ Site Review Requests _ Dougmanic's site

Posted by: dougmanic Oct 5 2006, 09:20 AM

What do you guys think of Dougmanic's? Does it look good?

http://www.dougmanic.po.gs

Posted by: jimlongo Oct 5 2006, 11:02 AM

That's very nice Doug. I quite like the clean look and lack of fancy nonsense that plagues much of the web. Some great photos as well!

Posted by: Cameron Oct 5 2006, 01:19 PM

Very nice! I love the clean-looking page layout. It really adds a nice touch, plus, since it is mostly text and simple things like that and not so many pictures, it will load more quickly on dial-up or other slow browsers.

Posted by: Christian J Oct 5 2006, 02:28 PM

The validator reports this problem: http://www.htmlhelp.com/cgi-bin/validate.cgi?url=http://www.dougmanic.po.gs/

And although the validator doesn't complain, one shouldn't use SGML comments in SCRIPT or STYLE elements in proper XHTML: http://developer.mozilla.org/en/docs/Properly_Using_CSS_and_JavaScript_in_XHTML_Documents

CODE

It's good to add a space before the ending slash, IIRC older browsers handle it better then.

The HTML structure can be improved. E.g., instead of

CODE

<h4>Welcome to <i>Dougmanic</i>'s</h4>

you should use a H1 heading, and instead of the <I> element you might use http://www.htmlhelp.com/reference/html40/phrase/cite.html(?). Then style these elements with CSS to look the way you want.

The subheaders look strange without CSS:

CODE

Going to the TabsSeptember 28, 2006[ Website ]

--why not add some white-space?

In your style sheet it's good to specify color and background together.

A fluid page layout is more user-friendly than one with a fixed width.

Posted by: Frederiek Oct 5 2006, 03:37 PM

The navigation links catch the hover, but you have to click on the text link itself to be taken to that link. There are many samples on the internet on how to extend link response to the entire ""button" with CSS (see at http://www.d.umn.edu/itss/support/Training/Online/webdesign/css.html).

And Arial is a "sans-serif" font, though you specified Arial together with the generic "serif" font in your CSS.

Posted by: SkyB0lt Oct 9 2006, 06:22 PM

Dude, I am new to this and can't wait to make such clean looking sites! Also I am going to pick your brain when I need help

Shawn