Показать сообщение отдельно
  #2 (permalink)  
Старый 29.10.2008, 19:46
Отправить личное сообщение для Octane Посмотреть профиль Найти все сообщения от Octane  
Регистрация: 10.07.2008
Сообщений: 3,873

this.browserVersion.indexOf('MSIE 6.0') != -1
и так выдаст true или false, для чего это записывать в виде
(this.browserVersion.indexOf('MSIE 6.0') != -1) ? true : false;

и скобки можно убрать.


Чтобы не писать много раз:
HelpeR.prototype…

можно использовать такую конструкцию:
HelpeR.prototype = {
  method1: function(…) {
   …
  },
  method2: function(…) {
   …
  },
  method3: function(…) {
   …
  }
};


Удобнее было бы, если скрипт обрабатывал только ссылки с указанным классом, например:
<a href="image.png" class="lightbox">Image</a>

а не все ссылки на странице.

А так же способ назначения обработчика событий, который вы используете сейчас:
HelpeR.prototype.processing = function(obj) 
{
…
    obj.onclick = function() 
	{
…
	  return false;
	}
  }
}

Лучше переписать с использованием addEventListener/attachEvent.

И совет: ставь точку с запятой там, где это возможно:
… = function(…) {
    …
}; <—
Ответить с цитированием