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:#ffffff url("http://www.sxc.hu/pic/m/t/th/thea0211/1197407__2.jpg"); font-size:14px; font-family:arial; } #content{ background:#ffffff; width:960px; margin:0 auto; padding:20px; overflow:hidden; } #nav1{ position:absolute; top: 70px; } #nav p{ display:inline; } #nav img{ display:none; } #nav{ background:#ffffff; margin:0 auto; width:960px; } h1{ background:#ffffff; 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: [WSG] my latest version of my page > hi. > well replaced the image for the rollovers. > take a look at http://startrekcafe.alacorncomputer.com > cheers Marvin. > ps: any feedback, good, bad or ugly. > > > > > ******************************************************************* > List Guidelines: http://webstandardsgroup.org/mail/guidelines.cfm > Unsubscribe: http://webstandardsgroup.org/join/unsubscribe.cfm > Help: memberh...@webstandardsgroup.org > ******************************************************************* > ******************************************************************* List Guidelines: http://webstandardsgroup.org/mail/guidelines.cfm Unsubscribe: http://webstandardsgroup.org/join/unsubscribe.cfm Help: memberh...@webstandardsgroup.org ******************************************************************* ************************************************************** Guidelines: http://webstandardsgroup.org/mail/guidelines.cfm Unsubscribe: http://webstandardsgroup.org/join/unsubscribe.cfm Help: memberh...@webstandardsgroup.org ************************************************************** ******************************************************************* List Guidelines: http://webstandardsgroup.org/mail/guidelines.cfm Unsubscribe: http://webstandardsgroup.org/join/unsubscribe.cfm Help: memberh...@webstandardsgroup.org *******************************************************************