0

Довольно распространенная на JQ ошибка - код срабатывает только со второго нажатия на элемент.
Функцию запускал при помощи CLICK и ON, но код срабатывает только со второго раза. Причина судя по всему в функции .data() но не знаю как исправить код чтобы он работал как надо. из атрибута DATA-price мы берем текст и вставляем в нужное место.
Помогите исправить

$("ul li").on( "click",function(){
	var a=$(this).data( "price" );
	$(".row.active .price").html(a);
	$("ul li").removeClass("active");
	$(this).addClass("active");
});

3
  • Конкретно в приведенном коде никакой ошибки нет, он отлично срабатывает с первого раза, где-то в другом месте что-то у вас 25 дек 2016 в 9:30
  • 1
    добавьте html иначе не понятно
    – SVE
    25 дек 2016 в 9:53
  • спасибо Вадиму за подсказку, оказалось косяк мой. Не был внимателен в написании селекторов
    – Alex Latro
    25 дек 2016 в 11:34

2 ответа 2

1

все работает и с первого раза:

$("ul li").on( "click",function(){
	var a=$(this).data( "price" );
	$(".row.active .price").html(a);
	$("ul li").removeClass("active");
	$(this).addClass("active");
});
.active {
  color: red;  
}
<script src="https://code.jquery.com/jquery-2.0.3.js"></script>
<ul>
  <li>
    lorem
  </li>
  <li>
    ipsum
  </li>
</ul>

4
  • а вас не смущает что в коде на JS, какой-то data-price целых 2 строчки занимает? может выпилить эти строчки, всё равно в вашем примере они не используются
    – MasterAlex
    25 дек 2016 в 10:24
  • @MasterAlex, а Вас не смущает что они в условии вопроса стоят (эти 2 строчки) ? Структура html выбрана мной произвольна, потому что автор не привел свой html код (возможно в нем причина проблемы).
    – SVE
    25 дек 2016 в 11:23
  • я спросил не про вопрос, а про ответ, ваш ответ содержит код из вопроса, но вы его не используете по назначению, а именно вытаскивание дата-атрибута и вставка в нужный блок. Зачем тогда вообще нужен ваш ответ с простейшей покраской текста по клику?
    – MasterAlex
    25 дек 2016 в 11:29
  • 1
    Что считаю нужным то и добавляю в код! И все работает!
    – SVE
    25 дек 2016 в 11:42
0

Оказалось код и правда работает. по началу мешал скрипт который использовал такойже селектор.
После переделки селектора оказалось что внутренний селектор (строка скрипта - 3) выбирает самый первый элемент на странице и изменяет значение в нем, и только со второго раза в нужном элементе.
После добавления правильного селектора код заработал как надо

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими условиями использования и подтверждаете, что прочитали и поняли наши политику конфиденциальности и нормы поведения.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.