.getSelection новый код дублирования плюс один щелчок

Question

Да, у меня была старая почта, почти такой же вопрос, но когда я потратил время и написал это сам, вместо того, чтобы использовать код @Tim Downs, я стал ближе, чем я думал. Только проблема в настоящее время является то, что каждый раз, когда я нажимаю кнопку добавляет еще один кусок так лучший способ показать вам это, перейдя в мусорную себя и пытается его

http://jsbin.com/ogiyed/1/edit

код здесь:

function getSelected() { 
    if(window.getSelection) { return window.getSelection(); } 
    else if(document.getSelection) { return document.getSelection(); } 
       else { 
         var selection = document.selection && document.selection.createRange(); 
         if(selection.text) { return selection.text; } 
      return false; 
     } 
     return false; 
    } 

$('.selections').not('.username').mouseup(function(e) { 
var uid = $('.selections').find('.username').text(); 
var selection = getSelected(); 
if(selection) { 
$('button').click(function() { 
var text = $('#textarea_content'); 
text.val(text.val()+'[quote="'+ uid +'"]'+ selection +'[/quote]'); 
}); 
} 
}); 

это почти работает как шарм;)

Answer 1

Вы назначаете обработчик щелчка внутри обработчик MouseUp, поэтому на каждом MouseUp он добавит дополнительный обработчик кликов, все из которых будут выполняться при каждом нажатии кнопки.

Если вам необходимо переопределить существующий обработчик щелчка вы сначала удалите предыдущую с .off('click'), а затем назначить новый:

$('button').off('click').click(function() { 
    var text = $('#textarea_content'); 
    text.val(text.val()+'[quote="'+ uid +'"]'+ selection +'[/quote]'); 
}); 

Изменено демо: http://jsbin.com/ogiyed/4/

(как в сторону: отступ ваш код!)