Глюк со сменой картинки

HTML:

<SCRIPT language="JavaScript" src="scr.js" type="text/javascript"></SCRIPT>



<IMG name="image1" src="images/but1off.gif" width="291px"

height="30px" border="0" alt="Your Image title" 

onMouseOver="return setImage(this.name , 'on');" 

onMouseOut="return setImage(this.name, 'off');">

JavaScript (scr.js):

function setImage (imgName, type) {

  if (document.images) {

      imgOff = new Object ();

      imgOff[imgName] = new Image();

      imgOff[imgName].src = "images/but1off.gif";

      imgOn = new Object ();

      imgOn[imgName] = new Image();

      imgOn[imgName].src = "images/but1on.gif";	 

      if (type == "on") {

          document.images[imgName].src = imgOn[imgName].src;

          return true;

      }

      else if (type == "off") {

          document.images[imgName].src = imgOff[imgName].src;

          return true;

      }

  }

  return false;

}

Да? А покороче нельзя чтобы хтмл особо не портить перед поисковиками?

Хм.. куда уж короче :) Файл scr.js - это внешний файл. Можно конечно поизвращаться с

и попробовать вынести также во внешний файл, но это бессмысленно :)

Кстати, Ваш код был более длинный, вот это

лишнее.

<script type="text/javascript">

  var img1 = new Image();

  var img2 = new Image();

  img1.src = 'img1.gif';

  img2.src = 'img2.gif';

</script>

...

<img src="img1.gif" onMouseOver="this.src=img2.src" onMouseOut="this.src=img1.src">

Короче вроде некуда.

Ну да, только без проверок.. а вообще если это внешний JS файл, то какая разница, какой у него размер? Разве поисковики будут загружать внешние JS скрипты? :)

А какие проверки здесь нужны?

Лично я считаю, что не будут загружать, только зачем лишний код нужен.

Проверку на существование document.images.. Хотя конечно можно и забить на это, я обычно предпочитаю оставлять поддержку для устаревших браузеров. В принципе этот код можно переписать и вынести во внешний файл также и onMouseOver="..." onMouseOut="...", прописав для рисунка только класс. Другой вариант - это сделать замену изображения с помощью стилей, однако поддерживать его будут также только новые браузеры (кстати, этот вариант мне больше по душе, т.к. работает и при выключенном JS у клиента).

Fuzzy, как-то вы все-таки усложняете код. Тут всего-то надо заменить src у картинки, а дополнительный код в начале страницы нужен лишь для того, чтобы не тормозило при первой подмене. Так к чему так много кода?

Так я пояснил вроде - для поддержки устаревших браузеров :) Однако не считаю это предельно важным, вообще в работе стараюсь использовать поменьше JS, т.к. многие его трюки выполняются с помощью CSS.

и всётаки я не врубился) +)

Какой код более универсальный то? =)