Hmmm...
Too much color. What is with the background graphic in the body and header? Unecessary.
The header is a total disaster. Ugly fonts, disorganized (need to use a grid). The footer is smashed in there (let it breathe a little!). Matching a script/display font (with an outline even) is near impossible - try to compliment it if possible with a sans-serif.
Index page...grids, grids, grids. Your content area doesn't really follow a grid or it's because your images don't really follow any kind of concept. Having images is good, but why not use them in a way that allows them to do their job and not seem like randomness.
If I look closely at the images I can see the etching. The wine glass image on the right bottom is really not good and even lands over the text (in Safari 1.2). Somebody should take better pictures however - I think this is really the bigger issue here.
Most of the fonts are the same size and the title fonts are centered. Poor choice of colors because you get blending and it's not exactly easy to read for alot of people. What's with all the horizontals? Too much going on - extremely distracting to the eye. Get rid of the purple background color and go to white...
There's better javascript for your popup windows out there - it shouldn't need to resize after it opens the window.
I'm not going to mention your html except that I looked at it and didn't even bother trying to organize it in my head...
The gallery - grid...similar shapes...more order...less distraction.
Reduction...simplicity is needed here. Simplicity is harder than people think in the early stages.
It could be too late, but if I were your course instructor I would tell you that you could only do your next draft in plain black and white and that everything had to physically be able to breathe fresh air.
ps: dug it out...here is some better javascript that does work on all browsers...
function OpenWindow(mypage, myname, w, h, scroll) {
var winl = (screen.width - w) / 2;
var wint = (screen.height - h) / 2;
winprops = 'height='+h+',width='+w+',top='+wint+',left='+winl+',scrollbars='+scroll+',resizable'
win = window.open(mypage, myname, winprops)
if (parseInt(navigator.appVersion) >= 4) { win.window.focus(); }
}
Implement it this way...
<a href="page.php" onClick="OpenWindow(this.href,'popup','475','525','yes');return false;">Hello</a>