Attempt to validate

Phixon · July 24th, 2007, 07:32 AM

Ok I have been trying to validate this web template for a while now, and its my first attempt at it. First I got it down to about 2 errors and then when it came to adding a doc type at added about 3million. What doc type do i need?

Code:

www.juxdesigns.com/hair

MikeHopley · July 24th, 2007, 07:39 AM

For a start, you are missing the opening <html> tag! This belongs after your Doctype.

You'd have a better chance of validating if you moved to HTML 4.01. XHTML requires all tags close, and you are not doing this at present. My recommendation:

Switch to HTML 4.01 Transitional, and make it validate.
Then try to remove all the presentational stuff to CSS.
Then switch to HTML 4.01 Strict, and make it validate.
Only then consider whether you want to use XHTML.

Phixon · July 24th, 2007, 07:43 AM

woops. must have accidently removed the html tag when messing around. So whats the first bit of code i need to add (the doc type code)

Phixon · July 24th, 2007, 08:02 AM

I have updated the file. Can you please tell me how to sort the last 2 errors please

matelot · July 24th, 2007, 08:29 AM

Search the forums and you'll find plenty of references. Google is also your friend.

But as a starter, try this: http://www.webforumz.com/vbarticles....e&articleid=29

It's a good article on the whys and wherefores of Doctype declarations.

Hope that helps.

matelot · July 24th, 2007, 08:31 AM

Quote:

Originally Posted by Phixon

I have updated the file. Can you please tell me how to sort the last 2 errors please

Can you at least post your errors so that we can see them?

Cheers

Phixon · July 24th, 2007, 08:32 AM

Quote:

Error Line 10 column 27: there is no attribute "HEIGHT".<table width="100%" height="100%" border="0" cellpadding="0" cellspacing="0"> You have used the attribute named above in your document, but the document type you are using does not support that attribute for this element. This error is often caused by incorrect use of the "Strict" document type with a document that uses frames (e.g. you must use the "Transitional" document type to get the "target" attribute), or by using vendor proprietary extensions such as "marginheight" (this is usually fixed by using CSS to achieve the desired effect instead).
This error may also result if the element itself is not supported in the document type you are using, as an undefined element will have no supported attributes; in this case, see the element-undefined error message for further information.
How to fix: check the spelling and case of the element and attribute, (Remember XHTML is all lower-case) and/or check that they are both allowed in the chosen document type, and/or use CSS instead of this attribute. If you received this error when using the <embed> element to incorporate flash media in a Web page, see the FAQ item on valid flash.

Error Line 12 column 46: there is no attribute "BACKGROUND"....lign="right" valign="top" background="images/rep_1.jpg" style="background-rep

I've been searching around but not much luck

MikeHopley · July 24th, 2007, 08:46 AM

I'm not much good with tables, but...

Try shifting the "height" and "background" attributes into CSS properties.

You have:

Code:

<table width="100%" height="100%"  border="0" cellpadding="0" cellspacing="0">

Replace this with:

Code:

<table width="100%" border="0" cellpadding="0" cellspacing="0" style="height: 100%;">

You also have:

Code:

<td align="right" valign="top" background="/images/rep_1.jpg" style="background-repeat:repeat-x; background-position:bottom ">

Replace this with:

Code:

<td align="right" valign="top" style="background: url('/images/rep_1.jpg') bottom repeat-x">

matelot · July 24th, 2007, 08:49 AM

Quote:

Error #1 means that your Doctype (HTML4.01 Trans) doesn't support the height attribute, so take it out. When you start designing without tables, you'll use CSS to set height on elements as required.

Error #2. You've set an image (rep_1.jpg) as a background in a TD, then tried to style it with background-repeat without declaring the image in the style. And you don't want it referenced twice.

Try style="background: url(images/rep_1.jpg) repeat-x top left"

I haven't tested it, but see how it goes.

Phixon · July 24th, 2007, 09:00 AM

Every time i remove a height attribute it finds another. Do I just remove all on the page?

MikeHopley · July 24th, 2007, 10:24 AM

Quote:

Originally Posted by Phixon

Every time i remove a height attribute it finds another. Do I just remove all on the page?

Not all the heights. You should keep the heights on images (<img>).

All the <table>, <tr>, and <td> elements, however, need fixing. Either remove the height, or shift it to style="height:100%;" as before.

matelot · July 24th, 2007, 10:25 AM

Quote:

Originally Posted by Phixon

Every time i remove a height attribute it finds another. Do I just remove all on the page?

The validator will find an error e.g. a height attribute in a table. Logically, if that attribute is not supported, then pre-empt the validator and remove similar occurrences.

You'll soon get the hang of it: we've all been through it.