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(…) {
…
}; <—