We kindly ask that all forum users respect the following:
We would appreciate it if you could be patient. Please help each other and most of all take some time to search for the answer to your question! It is very rare that we get a new question so invariably the answer will be here somewhere! Thank you!

I'd use a regular img tag for the image. When possible, you want to avoid using inline css. It's slow to render, especially so if it includes lots of background images (and isn't cascading).

If this design is going to be the default for installations, consider, on the homepage for example, changing prices to a non-bold, black, or something else on the gray-scale, font. Currently the prices are difficult to read and their color competes with the buy button.

Couldn't agree more and it is an aim of this release to remove all the deprecated items - though it looks like we might have overlooked one or two - the td's should absolutely not be in there. I appreciate that there are a couple of items that still need looking at - for example in this release you will notice that we have both horizontal and vertical menus (to serve as an example that it is possible). Most people will remove the duplicated element, although we will replace with classes. Also we do have some CSS3 elements that won't validate at the moment (frustrating).

Don't fear out of the box it will be as squeaky as we can get it.

Also note that the js files and css files have not yet had their final strip down and cleanse.

there's no easy way to give these two their own formatting:
RRP/Our Price.

consider giving each it's own class.

also, they should be on seperate lines. currently reads
RRP: £55.00 Our
Price: £48.00

These are variables held in /private/shop_messages.php so you do have access to wrap them in whatever you like. However this has been noted and I have wrapped them in this release. Separate lines are a matter of preference and attainable via editing /private/shop_messages.php.

are both classnames, viewproduct-thumb-image and thumb-image, necessary?

There are certainly a couple of places where the css could be shrunk a little, however some parts deliberately have separate code to allow people to apply separate classes without having to think too hard (though you are able to completely replace our xhtml/css with yours). The featured item part you mention has been addressed though as it really served no purpose.

The thumb image css is in part due to the new sale labels which require positioning and z-indexing. Though there is probably a way to amalgamate these two classes - it escapes me at the moment.

Quote:

Originally Posted by jesseo

displayproduct.php
use - and not _. The underscore is difficult to read/see: displayproduct-desc, intro_copy

Not sure where you are referring to? All the css id and classes use - ?

Quote:

Originally Posted by jesseo

cart (you can copy and paste source directly into validator.w3.org)
<select name="selptid" onChange="do_postage()"> -
there is no attribute "onChange"

I had already blatted that somewhere else - it's simply capitalization. Changed to lowercase c and that should be ok!

Thank you for taking the time to look over the code - it is much appreciated!