fix 3513: supprime la supperposition des aides markdown

[artragis]

Q	R
Type de modification	correction de bug
Ticket(s) (issue(s)) concerné(s)	(ex #3513 )

QA

• Buildez le front
• Dans la page de rédaction d'un message (i.e forum/mp/topic/tuto/article), vérifiez que si vous appuyez successivement sur les boutons "smiley"/"caractères spéciaux"/"blocs spéciaux" vous n'avez pas de superposition. Vérifiez que le menu "titres" ne gène pas non plus.

[sandhose]

jQuery est pas dispo dans ce scope via $. Le mieux, c'est de pas l'utiliser:

                if (this.currentElemPopup){
-                    $(this.currentElemPopup).remove();
+                    elemButton.removeChild(this.currentElemPopup);
+                    this.currentElemPopup = null;
                }

[artragis]

Corrigé, merci.

[coveralls]

Coverage remained the same at 87.337% when pulling 3087820 on artragis:patch-7 into cf876e5 on zestedesavoir:release-v18.

[coveralls]

Coverage remained the same at 87.337% when pulling 7ac7924 on artragis:patch-7 into cf876e5 on zestedesavoir:release-v18.

[sandhose]

Sorry, j'avais pas testé le code. Voici le patch qui corrige le bug:

diff --git a/assets/js/editor.js b/assets/js/editor.js
index 440e430..e7744c3 100644
--- a/assets/js/editor.js
+++ b/assets/js/editor.js
@@ -231,6 +231,7 @@
         setup: function(textareaId) {
             var elemTexta = document.getElementById(textareaId);
             var elemTools = document.createElement("ul");
+            var self = this;
             elemTools.className = "zform-toolbar hide-for-small";

             elemTexta.parentNode.insertBefore(elemTools, elemTexta);
@@ -248,6 +249,8 @@
                     while (menus[i]) {
                         if (menus[i].getAttribute("data-zform-info") !== "dontclose" || event.target.nodeName.toLowerCase() === "textarea") {
                             menus[i].style.display = "none";
+                            if (menus[i] == self.currentElemPopup)
+                                self.currentElemPopup = null;
                         }
                         i++;
                     }
@@ -289,16 +292,21 @@
             if (currentButton.hasOwnProperty("image")) elemButton.innerHTML = '<img src="' + currentButton.image +  '" alt="' + currentButton.title + '" />';

             if (currentButton.action) { // Button with a submenu
+                var self = this;
                 elemButton.style.position = "relative";
-                if (this.currentElemPopup){
-                    elemButton.removeChild(this.currentElemPopup);
-                    this.currentElemPopup = null;
-                }
                 this.addEvent(elemButton, "click", function(event, elemPopup) {
                     event.preventDefault();

                     if (elemPopup = this.getElementsByTagName("div")[0]) {
                         elemPopup.style.display = "block";
+
+                        if(self.currentElemPopup)
+                            self.currentElemPopup.style.display = "none";
+
+                        if(self.currentElemPopup == elemPopup)
+                            self.currentElemPopup = null;
+                        else
+                            self.currentElemPopup = elemPopup;
                     }
                 });

@@ -311,7 +319,6 @@
                 elemPopup.style.whiteSpace = "nowrap";
                 elemPopup.style.textAlign = "left";
                 elemPopup = this[currentButton.action](elemPopup, currentButton, opts.textarea);
-                this.currentElemPopup = elemPopup;
             } else {
                 this.addEvent(elemButton, "click", (function(_button, _textareaId, _this, _tagtype, _extraoption) {
                     return function(event) {

[artragis]

ok
merci.

[artragis]

Done.

[coveralls]

Coverage remained the same at 87.337% when pulling a8cad6c on artragis:patch-7 into cf876e5 on zestedesavoir:release-v18.

[Emeric54]

QA : On est bon ici 👍