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+ строк, работать с ним дико сложно. Это все имхо.