Показать сообщение отдельно
  #6 (permalink)  
Старый 16.09.2017, 11:25
Профессор
Отправить личное сообщение для Nexus Посмотреть профиль Найти все сообщения от Nexus
 
Регистрация: 04.12.2012
Сообщений: 3,733

SERblY, на элемент можно повесить более одного обработчика событий.
$('.notepad').click(function(event){
	event.preventDefault();
	var id=$(this).attr("data-id"),
		el = $(this),
		n_count = $('.n_count').html();
		
	$.ajax({
		type: "GET",
		url: TPL_PATH + "fav/",
		cache: false,
		data: "id="+id,
		success: function(html){
			n_count = parseInt(n_count);
			if(html == 1){
				el.removeClass('btn-success',500).addClass('btn-default',500);
				$('.n_count').html(n_count - 1);
			}else if(html == 2){
				el.removeClass('btn-default',500).addClass('btn-success',500);
				$('.n_count').html(n_count + 1).css('display', 'inline');
			}
		}
	});
}).click(function(){//По клику на элемент с классом .notepad
	$('.note2').fadeOut(500,function(){//Элемент с классом .note2 плавно сменит display на none
		$(this).remove();//После того, как он исчезнет, он будет удален
	});
});


PS. Какого стиля кода придерживаться - личное решение каждого, но, по-моему, лучше не делать после каждой строки отступ. Маленькие скриптики еще безобидно смотрятся, однако когда открываешь файл на 400+ строк, работать с ним дико сложно. Это все имхо.
Ответить с цитированием