Показать сообщение отдельно
  #27 (permalink)  
Старый 12.12.2009, 07:20
Аватар для x-yuri
Отправить личное сообщение для x-yuri Посмотреть профиль Найти все сообщения от x-yuri
 
Регистрация: 27.12.2008
Сообщений: 4,201

Сообщение от Kolyaj
Я еще как-то не решил для себя, стоит различать string от String и т.п. или не стоит
а ты знаешь, в каких случаях это может пригодиться?

пока небольшие замечания
core.js
· вроде extend более распространенное название для функции apply, тем более что так же названа Function.prototype.apply
·
$$: var elements = $(options.parent || document).getElementsByTagName(selector.split('.')[0] || '*');
createSelectorFilter: tagName = selectorParts[0].toUpperCase() || '*'

дублирование кода
·
getDocumentScroll:     return [
        win.pageXOffset || doc.documentElement.scrollLeft || doc.body.scrollLeft || 0,
        win.pageYOffset || doc.documentElement.scrollTop  || doc.body.scrollTop  || 0
    ];

я бы возвращал объект с полями x, y, и в других функциях тоже
Ответить с цитированием