From 57203727b69fa3764a4b0b0ab024f531a0e397ab Mon Sep 17 00:00:00 2001 From: evazion Date: Sun, 29 Apr 2018 00:17:56 -0500 Subject: [PATCH] post tooltips: hide on keypresses (#3689). --- app/assets/javascripts/post_tooltips.js.erb | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/app/assets/javascripts/post_tooltips.js.erb b/app/assets/javascripts/post_tooltips.js.erb index 22db6cf6c..f48f3434a 100644 --- a/app/assets/javascripts/post_tooltips.js.erb +++ b/app/assets/javascripts/post_tooltips.js.erb @@ -63,7 +63,8 @@ Danbooru.PostTooltip.initialize = function () { } }); - // Hide tooltips when clicking thumbnails. + // Hide tooltips when pressing keys or clicking thumbnails. + $(document).on("keydown", Danbooru.PostTooltip.hide); $(document).on("click", Danbooru.PostTooltip.POST_SELECTOR, Danbooru.PostTooltip.hide); // Disable tooltips on touch devices. https://developer.mozilla.org/en-US/docs/Web/API/Touch_events/Supporting_both_TouchEvent_and_MouseEvent @@ -73,6 +74,11 @@ Danbooru.PostTooltip.initialize = function () { }; Danbooru.PostTooltip.hide = function (event) { + // Hide on any key except control (user may be control-clicking link inside tooltip). + if (event.type === "keydown" && event.ctrlKey === true) { + return; + } + $(".post-tooltip:visible").qtip("hide"); };