Javascript-форум (https://javascript.ru/forum/)
-   Ваши сайты и скрипты (https://javascript.ru/forum/project/)
-   -   Модальное окно на чистом js (https://javascript.ru/forum/project/39359-modalnoe-okno-na-chistom-js.html)

danik.js 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 12.01.2015 17:02

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

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

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

cyber 12.01.2015 17:41

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

melky 12.01.2015 17:59

Цитата:

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

Цитата:

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

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

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

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

cyber 12.01.2015 18:01

Цитата:

Сообщение от melky
А имя темя не смутило

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

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

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

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

cyber 12.01.2015 18:23

Цитата:

Сообщение от danik.js
Вот только зачем, если на регулярках быстрее?

Нет, не быстрее проверял

это не поломает функции, но trim добавил, так как пробелы могут быть в начале и конце
Цитата:

Сообщение от danik.js
Пробелов может быть несколько идущих подряд. Тогда делать trim().


cyber 12.01.2015 19:13

Обновил до 1.0.5
https://github.com/cyberua/modalJS

cyber 12.01.2015 19:29

Обновил до 1.1
https://github.com/cyberua/modalJS

cyber 05.06.2015 22:49

Версия 2.0
https://github.com/cyberua/multiPopupJS

cyber 08.06.2015 01:44

Знаю что скрипт особо кроме меня не кому не нужен, но я тут учу инглишь что скажете по комментам, сильном много ошибок?)


Часовой пояс GMT +3, время: 08:39.