Как убрать класс при нажатии?

Question

[eellazy]

Всем привет! Никак не могу убрать класс при нажатии. Сделал чтобы класс добавлялся, но при повторном клике он все равно остается.

Вот кусок кода

<div class="item__uslugi <?=$sct["CODE"]?>">
<label class="icr-check icr-label" for="checkbox-<?=$sct["ID"]?>">
   <span class="icr-item type_checkbox"></span>
   <span class="icr-hidden">
   <input class="icr-input "  type="checkbox" name="dop[]" value="<?=$sct["ID"]?>" id="checkbox-<?=$sct["ID"]?>">
   </span>
   <span class="icr-text"><?=$sct["PREVIEW_TEXT"]?> <?=$sct["PRICE"]?></span>
</label>
 </div>

Вот сам скрипт

$(document).ready(function() {
$('.icr-label').on('click', function() {
  $(this).addClass('checked');
});
});

Пробовал так но не получилось. Даже не присваивает класс

$(document).ready(function() {

$('.icr-label').on('click', function() {
  $(this).toggleClass('checked');
});
});

Comments

[Евгений Меркушев]

Ты какой класс убираешь .checked или тебе .icr-check надо убрать?)
.toggleClass() - включает/выключает класс, а .removeClass() удаляет

[eellazy]

Евгений Меркушев: .checked нужно убрать

[Евгений Меркушев]

eellazy: тогда должно работать с toggle.
А ты аяксом не подгружаешь этот html?)

Answer 1

[Максим Тимофеев]

Все работает https://jsfiddle.net/2s50jgf7/
Думаю что Вы ловите клик не на том элементе, который реально клик получает. Но без ссылки на сайт - только вуду магия поможет.

Comments

[eellazy]

в том то и дело что на том в инспекторе подсвечивается при клике. Но класс не добавляется

[Максим Тимофеев]

eellazy: Дайте ссылку что ли. Как по другому понять что не так. Сам код что Вы привели рабочий. Значит проблема где-то за его пределами.

[eellazy]

у вас добавляется класс только если по чеку нажать

[eellazy]

Максим Тимофеев: avtogear-nov.ru/trendvision-combo.html?clear_cache=Y Дополнительные услуги - справа. У вас та же проблема. Если кликнуть по чекбоксу - сработает. Если в другое место - нет. Нужно чтобы при клике в любом месте блока присваивался класс

[Максим Тимофеев]

eellazy: ну так строчный элемент же, конечно, добавьте ему display:block; и будет работать везде клик: https://jsfiddle.net/2s50jgf7/1/

[eellazy]

Максим Тимофеев: нет. Смотрите. При клике на любую часть блока рамочка не появляется. То есть не назначает класс. А при клике на чекбокс появляется

[Максим Тимофеев]

eellazy: Смените label на div

[eellazy]

но тогда чек не будет работать по клику в любой области

[Максим Тимофеев]

eellazy: Да, трабл с двойной сработкой клика, вот иллюстрация: https://jsfiddle.net/2s50jgf7/3/

[Максим Тимофеев]

eellazy: вот решение https://jsfiddle.net/2s50jgf7/4/

[eellazy]

Максим Тимофеев: Да работает. А если не известно айди? Так как cms сама присваивает. можно ли классом?

[Максим Тимофеев]

eellazy: да конечно

Answer 2

[Павел]

$(this).removeClass('checked');