This is an automated email from the ASF dual-hosted git repository.
sbp pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/tooling-trusted-releases.git
The following commit(s) were added to refs/heads/main by this push:
new 1e1f7a9 Fix a lint error in the code to copy to clipboard
1e1f7a9 is described below
commit 1e1f7a921461e18b3968d9b8aea4ba056333f0f6
Author: Sean B. Palmer <[email protected]>
AuthorDate: Thu Dec 11 17:23:59 2025 +0000
Fix a lint error in the code to copy to clipboard
---
atr/static/js/src/clipboard-copy.js | 4 +++-
atr/static/js/src/copy-variable.js | 2 ++
2 files changed, 5 insertions(+), 1 deletion(-)
diff --git a/atr/static/js/src/clipboard-copy.js
b/atr/static/js/src/clipboard-copy.js
index 4e5c038..c0b8e12 100644
--- a/atr/static/js/src/clipboard-copy.js
+++ b/atr/static/js/src/clipboard-copy.js
@@ -1,9 +1,11 @@
+// We should consider merging this functionality with copy-variable.js
+
document.addEventListener("DOMContentLoaded", () => {
const copyButtons = document.querySelectorAll(".atr-copy-btn");
copyButtons.forEach((button) => {
button.addEventListener("click", function () {
- const targetId =
this.getAttribute("data-clipboard-target");
+ const targetId = this.dataset.clipboardTarget;
const targetElement = document.querySelector(targetId);
if (targetElement) {
diff --git a/atr/static/js/src/copy-variable.js
b/atr/static/js/src/copy-variable.js
index ed2ab68..c0c40ae 100644
--- a/atr/static/js/src/copy-variable.js
+++ b/atr/static/js/src/copy-variable.js
@@ -1,3 +1,5 @@
+// We should consider merging this functionality with clipboard-copy.js
+
document.addEventListener("DOMContentLoaded", () => {
document.querySelectorAll(".copy-var-btn").forEach((btn) => {
btn.addEventListener("click", () => {
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
