From d354cd967f304bf2c593bbc6664f1036b4cb6199 Mon Sep 17 00:00:00 2001
From: Hugo <hello@hzilliox.fr>
Date: Thu, 19 Mar 2020 18:58:18 +0100
Subject: [PATCH] Fix closing edit box issue when using fa-pencil as listener
 (#59)

---
 assets/javascripts/issue_dynamic_edit.js | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/assets/javascripts/issue_dynamic_edit.js b/assets/javascripts/issue_dynamic_edit.js
index 486d7d3..0268381 100644
--- a/assets/javascripts/issue_dynamic_edit.js
+++ b/assets/javascripts/issue_dynamic_edit.js
@@ -33,7 +33,13 @@ if (!document.getElementById(cssId)) {
 
 $(document).on(_CONF_LISTENER_TYPE, function(e) {
 	$('.issue .attributes .attribute .value').removeClass('edited');
-
+	// bind click to show edit block if click inside an edit box or on trigger, except button inside edit box
+	if(!$(e.target).closest('a.btn.btn-primary').length &&
+		($(e.target).closest('.' + _CONF_LISTENER_TARGET).length ||
+		$(e.target).closest('span.dynamicEdit').length)
+		) {
+		$(e.target).closest('.value').addClass('edited');
+	}
 	if ($(e.target).closest('a').length) { return; }
 	if ($(e.target).closest('.' + _CONF_LISTENER_TARGET).length) {
 		// avoid text selection if dblclick
@@ -48,9 +54,6 @@ $(document).on(_CONF_LISTENER_TYPE, function(e) {
 				sel.empty();
 			}
 		}
-
-		// we show the edit box
-		$(e.target).closest('.value').addClass('edited');
 	}
 });