From 38e960eb3f8155a907b95d528ba685c531bf21cb Mon Sep 17 00:00:00 2001
From: Pablo Vazquez <contact@pablovazquez.art>
Date: Mon, 25 Sep 2017 00:35:35 +0200
Subject: [PATCH] Comments: Fix multiple posting when using a hotkey

On every new item loaded with comments, we would bind the click without
unbinding first, leading to multiple posting when triggering the comment
submission.
---
 .../nodes/custom/comment/list_embed.pug       | 36 +++++++++++++------
 1 file changed, 26 insertions(+), 10 deletions(-)

diff --git a/src/templates/nodes/custom/comment/list_embed.pug b/src/templates/nodes/custom/comment/list_embed.pug
index 29f1b466..3b7eb630 100644
--- a/src/templates/nodes/custom/comment/list_embed.pug
+++ b/src/templates/nodes/custom/comment/list_embed.pug
@@ -76,6 +76,8 @@
 
 | {% block comment_scripts %}
 script.
+
+	// If there's a comment link in the URL, scroll there
 	function scrollToLinkedComment() {
 		var scrollToId = location.hash;
 		if (scrollToId.length <= 1) return;
@@ -86,13 +88,16 @@ script.
 	}
 	$(scrollToLinkedComment);
 
-	// Markdown initialization
+	// Initialize Markdown to later convert comment as we type
 	var convert = new Markdown.getSanitizingConverter();
 	Markdown.Extra.init(convert);
 	convert = convert.makeHtml;
 
-	/* Submit new comment */
-	$(document).on('click', 'body .comment-action-submit', function(e){
+
+	// Submit new comment
+	$(document)
+		.off('click','body .comment-action-submit')
+		.on( 'click','body .comment-action-submit', function(e){
 		post_comment($(this));
 	});
 
@@ -106,10 +111,12 @@ script.
 	});
 	{% endif %}
 
-	/* Edit comment */
 
+	// Writing comment
 	// Markdown convert as we type in the textarea
-	$(document).on('keyup','body .comment-reply-field textarea',function(e){
+	$(document)
+		.off('keyup','body .comment-reply-field textarea')
+		.on( 'keyup','body .comment-reply-field textarea',function(e){
 		var $textarea = $(this);
 		var $container = $(this).parent();
 		var $preview = $textarea.next().next();
@@ -134,8 +141,12 @@ script.
 	});
 
 
-	/* Enter edit mode */
-	$(document).on('click','body .comment-action-edit span.edit_mode',function(){
+	// Edit comment
+	// Enter edit mode
+	$(document)
+		.off('click','body .comment-action-edit span.edit_mode')
+		.on( 'click','body .comment-action-edit span.edit_mode',function(){
+
 		comment_mode(this, 'edit');
 
 		var parent_div = $(this).closest('.comment-container');
@@ -162,12 +173,17 @@ script.
 		});
 	});
 
-	$(document).on('click','body .comment-action-edit span.edit_cancel',function(e){
+	// Abort, abort
+	$(document)
+		.off('click','body .comment-action-edit span.edit_cancel')
+		.on( 'click','body .comment-action-edit span.edit_cancel',function(e){
 		commentEditCancel(this, true);
 	});
 
-	/* Save edited comment */
-	$(document).on('click','body .comment-action-edit span.edit_save',function(e){
+	// Save edited comment
+	$(document)
+		.off('click','body .comment-action-edit span.edit_save')
+		.on( 'click','body .comment-action-edit span.edit_save',function(e){
 		var $button = $(this);
 		var $container = $button.closest('.comment-container');