Общие вопросы

Еще, я присмотрелся к этой замечательной функцией CheckBrowser() и возникла целая куча вопросов:

> function CheckBrowser() {
> this.agent = navigator.userAgent.toLowerCase();
> this.version = parseInt(navigator.appVersion);
>
> this.explorer = (this.agent.indexOf('msie') != -1);
> this.explorer4 = this.explorer && this.version == 4 && document.all;
> this.explorer5 = this.explorer && this.version == 5 && document.all;
> this.explorer6 = this.explorer && this.version == 6 && document.getElementById;

Почему игнорируеш DOM в MSIE5?
Не придется ли добавлять строчки в срипте при каждом выходе нового MSIE (ровно как и NN)?

> this.netscape = (this.agent.indexOf('mozilla') != -1) && !this.explorer;
> this.netscape4 = this.netscape && this.version == 4 && document.layers;
> this.netscape5 = this.netscape && this.version == 5 && document.getElementById; // mozilla? :)
> this.netscape6 = this.netscape && this.version == 6 && document.getElementById;

Что за netscape5 и чем он отличается от netscape6?
Зачем делиш неделимое? ;-)


> this.mozilla = this.netscape && (this.agent.indexOf('gecko') != -1) && document.getElementById;

Под это определение в NN6+ this.mozilla == true... может лучше сказать:
this.gecko = (this.agent.indexOf('gecko') != -1)
а?


> this.opera = (this.agent.indexOf('opera') != -1);

лучше искать свойство window.opera, а то я в мозилле могу дописать в userAgent'е 'opera' и получиться, что я работаю оперой ;-)


> this.dom = (this.mozilla || this.netscape6 || this.explorer6) && document.getElementById;
> }