jQuery custom radio and checkbox

[Georgy1]

Взял на вооружение. Спасибо! :)

[CAJAX]

В chromium 18 в демо вижу оригинальные чекбоксы и радио справа от текста. Из них (оригинальных) кликаются только радио.

[n0r8]

Оригинальные стоят только для того чтоб показать что все работает. Кликать по ним нет необходимости. Можно добавить overflow:hidden; в стиль для радио и чекбокс оберток.

[Anonym]

bind -> live
А стандартный костыль с прозрачным элементом и подложкой в виде картинки нельзя было здесь применить?

[Elfrey]

Вы извините, но это кошмар.

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);
}

Переписать бы код.

[n0r8]

типа так $(this).parent().mousedown(pushed).parent().mouseup(check).parent().removeClass('disabled');?

[n0r8]

ну тоесть без парент:)

[JiLiZART]

$(this).parent() выносим в переменную.

[n0r8]

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 раз происходит в любом случае.

[Elfrey]

Все $(this) вынести в $this не помешает, зачем вызывать jq каждый раз. Цепочки само собой.

[n0r8]

пасиба постараюсь исправить

[serf]

Получать каждый раз необходимые элементы заново типичное порождение jQuery :) Просто пользователи jQuery разработчиками не являются, отсюда и такой код.

[Elfrey]

Немного больше понимания самого js и такого уже не бывает. Все же учатся на чем-то.

[serf]

Для многих JS это jQuery, о чем речь вообще )

[MaksSlesarenko]

Очень жаль что люди досих пор пишут плагины, а не юишные виджеты. Вроде разница в коде между виджетами и плагинами не большая, но уже намекает на то что человек таки прочел мануал. Из плюсов виджетов стандартное API из коробки, темизация(если автор не поленился реализовать ее поддержку)

PS Прошу прощения за офтоп.

[Borro]

Мне вот что интересно, почему все оборачивают в span, а не в label?

[dmitry_dvp]

странно, что

.attr('checked', false)
.attr('checked', 'checked')
.is(':checked')

но

.prop('disabled')

может так:?

.prop('checked', false)
.prop('checked', true)
.prop('checked')

[dmitry_dvp]

$('input[name=' + element.attr('name') + ']')

name у элементов может быть и посложнее qname, в т.ч. содержать скобки

[n0r8]

спасибо