From 3245678e109f8722ac451d2b4b1ca73ec764233c Mon Sep 17 00:00:00 2001 From: Lars Eggert Date: Fri, 8 Apr 2022 20:55:41 +0300 Subject: [PATCH] fix: Hide tooltips after review wish icon is clicked (#3799) Similar how it's done with the track/untrack icon. Also remove duplicate javascript for this functionality. --- ietf/static/js/doc-search.js | 33 +-------------------------------- ietf/static/js/ietf.js | 5 +++-- 2 files changed, 4 insertions(+), 34 deletions(-) diff --git a/ietf/static/js/doc-search.js b/ietf/static/js/doc-search.js index 40dee547e..58347ed50 100644 --- a/ietf/static/js/doc-search.js +++ b/ietf/static/js/doc-search.js @@ -76,35 +76,4 @@ $(document) updateAdvanced(); } - - $(".review-wish-add-remove-doc.ajax, .track-untrack-doc") - .on("click", function (e) { - e.preventDefault(); - var trigger = $(this); - $.ajax({ - url: trigger.attr("href"), - type: "POST", - cache: false, - dataType: "json", - success: function (response) { - if (response.success) { - trigger.parent() - .find(".tooltip") - .remove(); - trigger.attr("hidden", true); - - var target_unhide = null; - if (trigger.hasClass("review-wish-add-remove-doc")) { - target_unhide = ".review-wish-add-remove-doc"; - } else if (trigger.hasClass("track-untrack-doc")) { - target_unhide = ".track-untrack-doc"; - } - trigger.parent() - .find(target_unhide) - .not(trigger) - .removeAttr("hidden"); - } - } - }); - }); - }); \ No newline at end of file + }); diff --git a/ietf/static/js/ietf.js b/ietf/static/js/ietf.js index fe4a68229..6e9d33f67 100644 --- a/ietf/static/js/ietf.js +++ b/ietf/static/js/ietf.js @@ -274,8 +274,9 @@ $(document) dataType: 'json', success: function (response) { if (response.success) { + // hide tooltip after clicking icon trigger.parent() - .find(".track-untrack-doc") + .find(".review-wish-add-remove-doc.ajax, .track-untrack-doc") .tooltip("hide"); trigger.addClass("d-none"); @@ -331,4 +332,4 @@ $(document) .siblings(".full") .removeClass("d-none"); }); - }); \ No newline at end of file + });