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.
This commit is contained in:
Lars Eggert 2022-04-08 20:55:41 +03:00 committed by GitHub
parent 19e639f779
commit 3245678e10
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 4 additions and 34 deletions

View file

@ -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");
}
}
});
});
});
});

View file

@ -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");
});
});
});