PDA

View Full Version : Resolved W3C Validator errors



theremotedr
05-12-2015, 07:38 PM
Hi,
Are you able to please take a look at these 3 errors after using the W3C Validator.
Here is the report.
http://validator.w3.org/check?uri=http%3A%2F%2Fwww.theremotedoctor.co.uk%2Fhonda_programming.html&charset=%28detect+automatically%29&doctype=Inline&group=0&st=1&user-agent=W3C_Validator%2F1.3+http%3A%2F%2Fvalidator.w3.org%2Fservices

Page in question http://www.theremotedoctor.co.uk/honda_programming.html

After working my way through a few errors i am now stuck with the following & not sure what the correct answers would be
Line 10,26 & 35

Thanks very much.

jscheuer1
05-13-2015, 12:43 AM
Line 10 it's (,chrome=1 is) deprecated, but it's conceivable some people are still using the chrome frame plug-in for IE (support has stopped but there is still a stable version in use for IE up to version 9). So, if you want to support that (I have no idea what it is, Google - 'Chrome Frame' for details) you can still use that tag and ignore the error - it's not hurting anything. If you want to eliminate the error report, make the tag look like so:


<meta http-equiv="X-UA-Compatible" content="IE=edge">

IE 9 and less will be fine either way, and IE 10 and up, and all other browsers ignore it.

Line 26 is actually wrong, not an error as far as I and many others who have examined this issue are concerned. But it's possible it's technically an error. Either way it hurts nothing. If you want it to go away change:


<script type="text/javascript" src="js/selectionscroll/bookmarkscroll.js">

/***********************************************
* Scrolling HTML bookmarks- Dynamic Drive DHTML code library (www.dynamicdrive.com)
* This notice MUST stay intact for legal use
* Visit Project Page at http://www.dynamicdrive.com for full source code
***********************************************/

</script>

to:


<script type="text/javascript" src="js/selectionscroll/bookmarkscroll.js">

//***********************************************
//* Scrolling HTML bookmarks- Dynamic Drive DHTML code library (www.dynamicdrive.com)
//* This notice MUST stay intact for legal use
//* Visit Project Page at http://www.dynamicdrive.com for full source code
//***********************************************/

</script>

Line 35 - There's a technically illegal character in the URL used for the href value of the link tag. It's the | character, which can be replaced by:

%7C

and then validate and still work as the intended href. Change:


<link href='http://fonts.googleapis.com/css?family=Droid+Serif|Ubuntu' rel='stylesheet' type='text/css'>

to:


<link href='http://fonts.googleapis.com/css?family=Droid+Serif%7CUbuntu' rel='stylesheet' type='text/css'>

theremotedr
05-13-2015, 05:21 AM
All the above have now be change and across the site its much better.
I have two more requests below please.

http://validator.w3.org/check?uri=http%3A%2F%2Fwww.theremotedoctor.co.uk%2Ffor_sale_slideshow.html&charset=%28detect+automatically%29&doctype=Inline&group=0&user-agent=W3C_Validator%2F1.3+http%3A%2F%2Fvalidator.w3.org%2Fservices

http://validator.w3.org/check?uri=http%3A%2F%2Fwww.theremotedoctor.co.uk%2Findex.html&charset=%28detect+automatically%29&doctype=Inline&group=0&user-agent=W3C_Validator%2F1.3+http%3A%2F%2Fvalidator.w3.org%2Fservices

keyboard
05-13-2015, 10:41 AM
Link #1

Error #1
The language attribute of the <script> element has been depricated.
Change line #184 from

<script language="JavaScript1.2">
to

<script type="text/javascript">

Error #2
Not sure about this one

Error #3
The div #main .wrapper is closed before its child section element is closed.
Change


</aside><!-- #end left sidebar -->

</div><!-- #end div #main .wrapper -->

to


</aside><!-- #end left sidebar -->
</section>
</div><!-- #end div #main .wrapper -->


Error #4
Will be fixed once #3 is done

Warning #1
Should be fixed once #3 is done

Link #2

Warning #1
This block of code

<!-- content area -->
<section id="content" class="wide-content">
<div class="grid_4">
<img src="m-images/skp900-advert.jpg" alt="skp-900" width="287" height="148" />
<p>Super OBD SKP-900 Key Programmer, Updated each month to keep inline with the modern technology that is currently in use today.</p>
</div>

<div class="grid_4">
<img src="m-images/x100-advert.jpg" alt="x100+" />
<p>Diagnostic programmer which is a favourite with Honda vehicles. Connected to the OBD2 port it programmes Honda keys with ease.</p>
</div>

<div class="grid_4">
<img src="m-images/dr-card-advert.jpg" alt="remote dr card" />
<p>Based in Weston Super Mare i offer a mobile service for Honda key programming. I will also travel to the surrounding areas.</p>
</div>

</section><!-- #end content area -->
Doesn't contain any header tags. All section's are supposed to include a header.

jscheuer1
05-13-2015, 01:34 PM
Wow, that's such an old error I guess you never learned about it. The ilayer tag is deprecated. And, as it so happens so are the browsers that used it and any code meant to support them/it. So, to fix this on the slideshow page:


Line 253, Column 122: Element ilayer not allowed as child of element section in this context. (Suppressing further errors from this subtree.)

The tag can simply be removed. Get rid of this:


<ilayer id="slidensmain" width=&{slidewidth}; height=&{slideheight}; bgColor=&{slidebgcolor}; visibility=hide><layer id="slidenssub" width=&{slidewidth}; left=0 top=0></layer></ilayer>

It will not affect the slideshow because that part of the slideshow script which supports browsers like Netscape 4 and earlier (which are never used) is also never used.

theremotedr
05-13-2015, 01:44 PM
Now corrected.

I am just going through the site for others.

theremotedr
05-13-2015, 02:11 PM
Been through the site and have this only.

http://validator.w3.org/check?uri=http%3A%2F%2Fwww.theremotedoctor.co.uk%2Fhonda_key_programming_mobile_service.html%3Fscrollto%3Dselection&charset=%28detect+automatically%29&doctype=Inline&group=0&user-agent=W3C_Validator%2F1.3+http%3A%2F%2Fvalidator.w3.org%2Fservices

The second is about the heading,do i really need it ?
I deleted them to gain space on the pages.
If i put headings on some pages they will all but pushed down the page a tough.
I assume my choice is ok to do this but is it as simple as that,please advise.

jscheuer1
05-13-2015, 03:51 PM
That's a warning, not an error. Best practices generally uses headings for SEO and ease of use by humans by making it clearer what the site and the page is about. However it is already very clear what the site is about, and probably the page as well, let me check. The page's title says it all. You don't need headings for such a straight forward topic.

theremotedr
05-13-2015, 04:03 PM
Many thanks for your time & help.