Hello all,
I have been learning HTML and CSS for the past 2 months and now had to build a site to conclude the course
I would appreciate it if you take some time to review it - thanks

http://www.geocities.com/melitikus

Browsers don't do a very good job of resizing images. If you're going to display banner.jpg as 955x174, then resize it to 955x174 and use the resized version. Don't use a 1167x247 version and use HTML to get the browser to resize it.

It's a fixed-width layout. 955px is going to be too wide for a lot of display environments.

alt="" is not appropriate for the banner.jpg image. Since it includes an image of text, your should probably use the text itself: alt="Euro Coins Collection Malta"

The "menu" at commemorativecoins.htm doesn't work when JavaScript is disabled/unavailable, and seems rather pointless given how few commemorative coins Malta has issued. A simple list of links (once there are enough commemorative coins to justify more than one page) would be more usable and more accessible.

Don't use px for font sizes. Use 100% for body text, larger percentages for headings and the like, and (slightly) smaller percentages for legalese and similar fine print.

What if the customer wants to order two or more items? Your drop-down list only allows them to select one item.

thanks for your comments

I could not open your site.What is the problem?

Now review my sites please!!

http://www.hook4free.com

http://www.skyline-asia.net

http://www.dl.kg

http://www.ferrsarivsporsche.com

And www.dl.kg yields:

Our online validator reports some markup errors in the others:
http://htmlhelp.com/cgi-bin/validate.cgi?u...mp;warnings=yes
http://htmlhelp.com/cgi-bin/validate.cgi?u...mp;warnings=yes

The "Customize your homepage" elements splat on top of your content in the upper right corner of www.hook4free.com when JavaScript is disabled/unavailable.