Andy said:
Nothing wrong with most of that - you use href appropriately as a
fallback if the JavaScript event handler isn't available. It does the
right things with return values etc.
I would suggest though using <br> instead of <p> to keep the HTML
valid.
I did in my page about the thumbnailing that
I copied it from:
http://www.mbstevens.com/howtothumb
I saw the <br> on my page and thought, a <p>
would work better. I didn't remember my own
research!
I'd also give a useful title as the caption beneath the image, not an
instruction to go clicking on things. If you need this, do it once per
page.
That's not so much an instruction as a
warning that the thing will open a new
window. Visitors should be warned, at the
very least, if you are about to do something
so rude as opening a new window. Of course,
you shouldn't do it at all!
If the popup window is to have a size, then try to set this dynamically
based on the actual size of the image. You can do this either server or
client side.
Finding the size of images on the fly might
eat time. I think that this size should be
known in advance, either stored by the server
side program in a base or passed to the
server side program by the call from HTML.
For client side, you should just know the
size of the image in advance and code it in.
If it's on your site you can find it out in
advance easily.