On 21/11/2005 19:17, Jeff wrote:
[snip]
[color=blue]
> <!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN"
> "http://www.w3.org/TR/html4/loose.dtd">[/color]
Unless you /need/ a Transitional feature, use a Strict document type
declaration.
[snip]
[color=blue]
> <script type="text/javascript">
> <!-- HIDE[/color]
Hide from what? Browsers that are old enough not to recognise a SCRIPT
element (even if they don't support client-side scripts) aren't used on
the Web, not least because they're all but useless now.
[color=blue]
> if (document.images)
> {
> document.images["order1"] = new Image(135,40);[/color]
The images collection should be used to /retrieve/ images already in the
document. Create a new object to retain image objects for preloading
purposes:
var preload = {};
/* Check for support of the Image constructor */
if(('object' == typeof Image) || ('function' == typeof Image)) {
/* Preload */
preload.order1 = new Image(135, 40);
preload.order1.src = 'images/ordernow1.jpg';
/* ... */
}
[snip]
[color=blue]
> <a href="http://www.mysite.com/order.html"
> onMouseover="return rollover('order2','images/ordernow2.jpg');"
> onMouseout="return rollover('order1','images/ordernow1.jpg');">[/color]
Your rollover function will attempt to look for IMG elements with a name
or id attribute value of 'order1' and 'order2' (depending on the event),
but no such element exists. This is why your code fails.
[color=blue]
> <img src="images/ordernow1.jpg">[/color]
This element should have an alt attribute (required, even if an empty
string), and must be identified so that the script can reference it.
Change the strings in the event listeners above to reflect this name:
<a href="..."
onmouseover="rollover('order', 'images/ordernow2.jpg');"
onmouseout="rollover('order', 'images/ordernow1.jpg');">
<img id="order" name="order"
alt="..."
src="images/ordernow1.jpg">
</a>
[snip]
Mike
--
Michael Winter
Prefix subject with [News] before replying by e-mail.
