Hi Marvin
Good to see you have an interest in web design. This is a great industry
with a lot of friendly competitors.
The JavaScript:
At the end of each line you need to have semi-colon ;
Why the three changes to the same element in one call?
b1 element does not exist
The layout needs work you can find a good tutorial on building a layout
from scratch @
http://net.tutsplus.com/tutorials/design-tutorials/design-and-code-a-slick-website-from-scratch-%E2%80%93-part-i/
Using the principles in this guide should help you for a much more slicker
looking site.
Other notes:
You need to make your elements semantic i.e. put the prices in span tags
e.g. span class=price$3.99/span and address in
address/address tags
When you use tables you need to have a heading row which describes the
column content. This helps with accessibility
The Bananas in the navigation should be in the CSS as they are layout
and not content. Try the CSS below
#nav a{
display:block;
height: 60px;
background:url('../images/fruit_banana.jpg') no-repeat;
vertical-align:bottom;
}
Your site is not Valid XHTML as it states in the bottom right hand
corner
Joe's Fruit's address is Anytown
Contrast really bad, red on dark green when you mouse over a link is not
a good idea
On the staff page you have some wonderful imagery of fruit these should
be incorporated into the front page
The Mango image is missing
To get you started try this CSS instead (example @
http://createmy.com.au/temp/CropperCapture%5B18%5D.jpg). It is very basic,
once you make your XHTML more semantic you can do a lot more.
@import http://meyerweb.com/eric/tools/css/reset/reset.css;;
body{
background:#ff
url(http://www.sxc.hu/pic/m/t/th/thea0211/1197407__2.jpg;);
font-size:14px;
font-family:arial;
}
#content{
background:#ff;
width:960px;
margin:0 auto;
padding:20px;
overflow:hidden;
}
#nav1{
position:absolute;
top: 70px;
}
#nav p{
display:inline;
}
#nav img{
display:none;
}
#nav{
background:#ff;
margin:0 auto;
width:960px;
}
h1{
background:#ff;
margin:0 auto;
width:960px;
font-size:1.6em;
height:3em;
display:block;
padding:20px;
}
h2{
font-size:1.5em;
}
#content table{
width:660px;
float:left;
border-right:1px #e1e1e1;
margin-right:20px;
}
#content img{
float:right;
}
hr{
display:none;
}
Regards,
Dale
Dale Hurley
http://createmy.com.au
From: wsg@webstandardsgroup.org
To: wsg@webstandardsgroup.org
Date: 22/09/2009 05:00 AM
Subject:WSG Digest
*
WEB STANDARDS GROUP MAIL LIST DIGEST
*
From: Ted Drake tdr...@yahoo-inc.com
Date: Mon, 21 Sep 2009 11:50:21 -0700
Subject: RE: [WSG] my latest version of my page
Use semantic markup. The nav should be a list, not a stack of
paragraphs.
Ted
-Original Message-
From: li...@webstandardsgroup.org [mailto:li...@webstandardsgroup.org]
On Behalf Of Dave Westell
Sent: Saturday, September 19, 2009 8:37 AM
To: wsg@webstandardsgroup.org
Subject: Re: [WSG] my latest version of my page
Hi Marvin,
OK I now see you are using mouseover () mouseOut() effects...
Your .js script is missing .
http://www.alacorncomputer.com/javascript/fruit.js
Also your page does not validate, you have one too many closing divides
/div after #nav1
Dave...
- Original Message -
From: Marvin Hunkin startrekc...@gmail.com
To: wsg@webstandardsgroup.org
Sent: Saturday, September 19, 2009 7:51 AM
Subject: