From 0b654fa8dc132cce191f43fb9bdf4d5864e11b7c Mon Sep 17 00:00:00 2001
From: silverwind <me@silverwind.io>
Date: Wed, 18 Oct 2023 17:16:06 +0200
Subject: [PATCH] Clipboard copy enhancements (#27669)

1. Do not show temporary tooltips that are triggered from within
dropdowns. Previously this resulted in the tooltip being stuck to
top-left of the page like seen on issue comment URL copy. I could not
figure out any tippy options that prevent this, so I think it's better
to just not show it.
1. Refactor `initGlobalCopyToClipboardListener` so that it does not run
a often useless `document.querySelector` on every click, make
`data-clipboard-text-type` work with `data-clipboard-target`. No use in
current code base but still good to have. Finally some minor code
cleanup in the function.

Point 1 is for this copy button:

<img width="229" alt="image"
src="https://github.com/go-gitea/gitea/assets/115237/81f34746-8ea5-43d9-8c6f-f6f417a9e4ad">

---------

Co-authored-by: Giteabot <teabot@gitea.io>
---
 web_src/js/features/clipboard.js | 21 ++++++++++++++-------
 web_src/js/modules/tippy.js      |  5 +++++
 2 files changed, 19 insertions(+), 7 deletions(-)

diff --git a/web_src/js/features/clipboard.js b/web_src/js/features/clipboard.js
index bcbdae2704..224628658e 100644
--- a/web_src/js/features/clipboard.js
+++ b/web_src/js/features/clipboard.js
@@ -4,19 +4,25 @@ import {clippie} from 'clippie';
 
 const {copy_success, copy_error} = window.config.i18n;
 
-// For all DOM elements with [data-clipboard-target] or [data-clipboard-text],
-// this copy-to-clipboard will work for them
+// Enable clipboard copy from HTML attributes. These properties are supported:
+// - data-clipboard-text: Direct text to copy, has highest precedence
+// - data-clipboard-target: Holds a selector for a <input> or <textarea> whose content is copied
+// - data-clipboard-text-type: When set to 'url' will convert relative to absolute urls
 export function initGlobalCopyToClipboardListener() {
   document.addEventListener('click', (e) => {
     let target = e.target;
-    // in case <button data-clipboard-text><svg></button>, so we just search
+    // In case <button data-clipboard-text><svg></button>, so we just search
     // up to 3 levels for performance
     for (let i = 0; i < 3 && target; i++) {
-      let txt = target.getAttribute('data-clipboard-text');
-      if (txt && target.getAttribute('data-clipboard-text-type') === 'url') {
-        txt = toAbsoluteUrl(txt);
+      let text = target.getAttribute('data-clipboard-text');
+
+      if (!text && target.getAttribute('data-clipboard-target')) {
+        text = document.querySelector(target.getAttribute('data-clipboard-target'))?.value;
+      }
+
+      if (text && target.getAttribute('data-clipboard-text-type') === 'url') {
+        text = toAbsoluteUrl(text);
       }
-      const text = txt || document.querySelector(target.getAttribute('data-clipboard-target'))?.value;
 
       if (text) {
         e.preventDefault();
@@ -28,6 +34,7 @@ export function initGlobalCopyToClipboardListener() {
 
         break;
       }
+
       target = target.parentElement;
     }
   });
diff --git a/web_src/js/modules/tippy.js b/web_src/js/modules/tippy.js
index 372f7bc8f3..4eed0aa4bb 100644
--- a/web_src/js/modules/tippy.js
+++ b/web_src/js/modules/tippy.js
@@ -169,6 +169,11 @@ export function initGlobalTooltips() {
 }
 
 export function showTemporaryTooltip(target, content) {
+  // if the target is inside a dropdown, don't show the tooltip because when the dropdown
+  // closes, the tippy would be pushed unsightly to the top-left of the screen like seen
+  // on the issue comment menu.
+  if (target.closest('.ui.dropdown > .menu')) return;
+
   const tippy = target._tippy ?? attachTooltip(target, content);
   tippy.setContent(content);
   if (!tippy.state.isShown) tippy.show();