PDA

Click to See Complete Forum and Search --> : Please review my rating site.

bflatman
05-01-2004, 12:30 PM
MakeOrBreakUs.com (http://www.makeorbreakus.com)

I would really appreciate it if any one would try out the registration system and pic upload system for me. Thank you!
:)
bflatman
05-01-2004, 01:29 PM
I would appreciate any feed back on design ideas (not my strong point).
bflatman
05-01-2004, 08:12 PM
Please!
Jona
05-01-2004, 08:42 PM
Please, don't reply to your own post; someone will eventually reply, most likely.

I ran your site through a CSS validator (http://jigsaw.w3.org/css-validator/validator?profile=css2&warning=2&uri=http%3A//www.makeorbreakus.com/), and found errors that should be fixed. I also ran it through an HTML validator (http://validator.w3.org/check?uri=http%3A%2F%2Fwww.makeorbreakus.com%2F&charset=utf-8+%28Unicode%2C+worldwide%29&doctype=HTML+4.01+Strict&verbose=1) and found six errors, aside from the fact that you have no DTD and no charset specified.

The site was well designed in CSS, although it could be much better, it is much better than having an ugly table layout. Regardless, however, the site does have some problems when viewed in a text-only browser (http://www.delorie.com/web/lynxview.cgi?url=http%3A%2F%2Fwww.makeorbreakus.com%2F). That also should be fixed.
bflatman
05-04-2004, 11:23 AM
Thank you for your review, could you please tell me what DTD and no charset specified are. This is my first site so a bit of a noob sorry.
Jona
05-04-2004, 02:08 PM
The Document Type Definition serves the following main functions:

* Specifies which elements can occur in the document
* Specifies content models for elements
* specifies the attribute-list Declarations for the elements. This includes specifying attribute names, possible attribute values and their attribute defaults
* defines general entitys for use within the document
* defines character references for use within the document


Basically it tells the browser what code it should be translating and how it should translate it. In HTML, you want to use the latest version, so you use an HTML 4.01 DTD. However, there are three different kinds of HTML 4.01 DTD's -- one for framesets (http://www.w3.org/TR/html4/frameset.dtd), one for transitional code (lenient version of HTML, allows for common errors and mistakes, see http://www.w3.org/TR/html4/transitional.dtd), and one for strict code (http://www.w3.org/TR/html4/strict.dtd). Unless you're using a frameset, which you shouldn't be using, you should use the HTML 4.01 Strict DTD. It is case-sensetive and should be at the very beginning of the code on every one of your pages:


<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01//EN"
&nbsp;"http://www.w3.org/TR/html4/strict.dtd">


The <!DOCTYPE part specified that it is a DTD, the "HTML PUBLIC" part means that it is HTML code, the "-//W3C//DTD HTML 4.01//EN" means to use the English version of the W3C's recommended HTML 4.01 DTD. The last part which specified the address of the DTD is not required but is good to ensure that the browser isn't using an outdated DTD version, which could cause some problems.