Показать сообщение отдельно
  #20 (permalink)  
Старый 25.02.2013, 13:58
Аспирант
Отправить личное сообщение для master_alf Посмотреть профиль Найти все сообщения от master_alf
 
Регистрация: 08.04.2010
Сообщений: 34

Сообщение от danik.js Посмотреть сообщение
1) body в селекторе ни к селу ни к городу. Бывают картинки вне body чтоли?
Согласен, fail.
Сообщение от danik.js Посмотреть сообщение
2) Раз jQuery, то и оформи как плагин к jQuery. Чтобы вызов был через $('div img').makeGray()
Погляжу как такое делается.
Сообщение от danik.js Посмотреть сообщение
Срём в глобал? Не канает для публичного плагина. Срочно обернуть в анонимную функцию.
Как только прочту, что такое анонимная функция - поправлю как рекомендвано. с JS сталкиваюсь постольку-поскольку...
Сообщение от danik.js Посмотреть сообщение
А что если у юзера был вызван jQuery.noConflict() ?
Передать в анонимную функцию - обертку ссылку на jQuery, а внутри уже использовать знак доллара. Смотри как это сделано в других плагинах. Вообще, раз ты делаешь плагин, хоть бы не поленился и изучил как их делают другие.
Как сказал выше - в JS слаб, jQuery умею использовать, но не писать под него плагины... погляжу ну другие обязательно.

Сообщение от danik.js Посмотреть сообщение
Это самый последний вариант решения, когда подругому - ну никак. В данном же случае можно просто навесить обработчик onload на картинки и ждать их загрузки.
Согласен, такой вариант в голову не приходил. Перепишу.

Сообщение от danik.js Посмотреть сообщение
Как будто два разных человека писали код. addClass предпочтительней.
За исключением пересчета цвета для канваса - всё писал я. А разные методы т.к. по-разному думал в эти моменты. Что вспомнил, то и использовал...
Сообщение от danik.js Посмотреть сообщение
Где используется функция createCanvas ?
Fail осталась не вычещенной с момента сборки всего воедино.
Сообщение от danik.js Посмотреть сообщение
Зачем это? jQuery сама фиксит opacity в старых ишаках.
Это не фикс прозрачности, а её динамическое изменение. Этим и достигатся плавность перехода в IE.
Сообщение от danik.js Посмотреть сообщение
Перед .animate() нужно делать .stop() чтоб не образовывалось очереди.
Забыл добавлю.
Сообщение от Deff Посмотреть сообщение
На большинстве халявных сайтах и форумов, нет загрузки изо на текущий домен
Пример Укоз, mybb.ru
Не подумал - добавлю.
Сообщение от danik.js Посмотреть сообщение
В WebKit и вроде 10м ишаке (не уверен) есть поддержка css mask, нужно проверять поддержку и по возможности использовать.
Почитаю что такое css mask. Пока я не понял о чем ты сказал.
Сообщение от danik.js Посмотреть сообщение
$(newImg).attr('class', 'gray')

Слишком общий css-класс. Он может быть определен у юзера. Вообще, зачем тут css-классы назначать?
С названием класса согласен. Хотел изменить на уникальные... но, опять же, забыл. А классы назначал как признак того, что данное изображение - серое. Что бы с него не делать копий. Т.к. при моем способе вызова функции grayScale() - были возможны дубликаты и уже серых картинок. Но с img.onload - проблема уйдет.
Ответить с цитированием