Форум

danik.js · #21 (**permalink**) 12.01.2015, 15:46

Описание:

Цитата:

It is small cross-browser script for modal windows or popup windows.

Артикль пропущен. Да и не нужно тут предложение с подлежащим/сказуемым. Просто Small ...

Цитата:

NotUseAnimate - if true, modal window not used animation

function modal

Конструкторы с большой буквы - не вкурсе чели?

throw "Incorrect parameter 'window', expected Element";

Помоему это TypeError. И вроде аргумент, а не параметр.

Цитата:

closeHadnler

Цитата:

onClickClose

Цитата:

showAnimateWin

Цитата:

getSupported

Имена методов/событий - то show, то open, то hide, то close.
isHTMLElement вернет true даже для SVGElement (у них же есть style?) или просто Element.
К чему проверка style?

cyber · #22 (**permalink**) 12.01.2015, 17:02

danik.js, зоопарк с функциями и именами поправлю, я просто сначала поправил функционал.

Сообщение от danik.js

HTMLElement вернет true даже для SVGElement (у них же есть style?) или просто Element.

А как тогда выделить именно html element

cyber · #23 (**permalink**) 12.01.2015, 17:41

getSupported - это не буливая функция, там возвращает по строку с поддерживаемым методом, поэтому не is*

melky · #24 (**permalink**) 12.01.2015, 17:59

Сообщение от danik.js

Конструкторы с большой буквы - не вкурсе чели?

Сообщение от danik.js

NotUseAnimate

А имя темя не смутило

обычно не используют вообще булевы с "not" - слишком громоздко получается

в конкреном случае можно заюзать и useAnimate или вообще animate (вверх простоты)

cyber · #25 (**permalink**) 12.01.2015, 18:01

Сообщение от melky

А имя темя не смутило

Такого имени уже нет, только в комментарии забыл поменять

Сообщение от melky

обычно не используют вообще булевы с "not" - слишком громоздко получается

в конкреном случае можно заюзать и useAnimate или вообще animate (вверх простоты)

Тогда казалось так удобнее, сейчас поменяю, сделаю по умолчанию true и все)