Как стать автором
Поиск
Написать публикацию
Обновить

Комментарии 20

Взял на вооружение. Спасибо! :)
Всего голосов 4: ↑1 и ↓3-2
В chromium 18 в демо вижу оригинальные чекбоксы и радио справа от текста. Из них (оригинальных) кликаются только радио.
Комментарий пока не оценивали0
Оригинальные стоят только для того чтоб показать что все работает. Кликать по ним нет необходимости. Можно добавить overflow:hidden; в стиль для радио и чекбокс оберток.
Комментарий пока не оценивали0
bind -> live
А стандартный костыль с прозрачным элементом и подложкой в виде картинки нельзя было здесь применить?
Комментарий пока не оценивали0
Вы извините, но это кошмар.
if (!$(this).prop('disabled')) {
	$(this).parent().mousedown(pushed);
	$(this).parent().mouseup(check);
	$(this).parent().removeClass('disabled');
} else {
	$(this).parent().addClass('disabled');
	$(this).parent().unbind('mousedown', pushed);
	$(this).parent().unbind('mouseup', check);
}


Переписать бы код.
Всего голосов 5: ↑4 и ↓1+3
типа так $(this).parent().mousedown(pushed).parent().mouseup(check).parent().removeClass('disabled');?
Всего голосов 1: ↑1 и ↓0+1
ну тоесть без парент:)
Комментарий пока не оценивали0
$(this).parent() выносим в переменную.
Всего голосов 2: ↑2 и ↓0+2
if (!$(this).prop('disabled')) {
  $(this).parent().mousedown(pushed).mouseup(check).removeClass('disabled');
} else {
  $(this).parent().addClass('disabled').unbind('mousedown', pushed).unbind('mouseup', check);
}

Думаю так лучше будет т.к. вызов $(this).parent() 1 раз происходит в любом случае.
Комментарий пока не оценивали0
Все $(this) вынести в $this не помешает, зачем вызывать jq каждый раз. Цепочки само собой.
Всего голосов 2: ↑2 и ↓0+2
пасиба постараюсь исправить
Комментарий пока не оценивали0
Получать каждый раз необходимые элементы заново типичное порождение jQuery :) Просто пользователи jQuery разработчиками не являются, отсюда и такой код.
Всего голосов 2: ↑1 и ↓10
Немного больше понимания самого js и такого уже не бывает. Все же учатся на чем-то.
Всего голосов 1: ↑1 и ↓0+1
Для многих JS это jQuery, о чем речь вообще )
Всего голосов 3: ↑2 и ↓1+1
Очень жаль что люди досих пор пишут плагины, а не юишные виджеты. Вроде разница в коде между виджетами и плагинами не большая, но уже намекает на то что человек таки прочел мануал. Из плюсов виджетов стандартное API из коробки, темизация(если автор не поленился реализовать ее поддержку)

PS Прошу прощения за офтоп.
Всего голосов 3: ↑3 и ↓0+3
Мне вот что интересно, почему все оборачивают в span, а не в label?
Комментарий пока не оценивали0
НЛО прилетело и опубликовало эту надпись здесь
странно, что
.attr('checked', false)
.attr('checked', 'checked')
.is(':checked')

но
.prop('disabled')


может так:?
.prop('checked', false)
.prop('checked', true)
.prop('checked')
Комментарий пока не оценивали0
$('input[name=' + element.attr('name') + ']')

name у элементов может быть и посложнее qname, в т.ч. содержать скобки
Комментарий пока не оценивали0
спасибо
Комментарий пока не оценивали0
Зарегистрируйтесь на Хабре, чтобы оставить комментарий

Публикации

Показать лучшие за всё время

Истории

Ваш аккаунт

Разделы

Информация

Услуги

Техническая поддержка
© 2006–2024, Habr