Make copy-to-clipboard functionality more generic

Bloke · Bloke · commit c1f9b26dc155 · 2026-03-03T13:51:32.000Z
See #2061.
diff --git a/textpattern/include/txp_diag.php b/textpattern/include/txp_diag.php
@@ -647,7 +647,7 @@ function doDiagnostics()
             inputLabel(
                 'diag_clear_private',
                 checkbox('diag_clear_private', 1, false, 0, 'diag_clear_private').'<span class="txp-textarea-options">
-                        <button class="action-copy txp-reduced-ui-button"><span class="ui-icon ui-icon-clipboard"></span> ' . gTxt('copy_to_clipboard') . '</button>
+                        <button class="action-copy-clip txp-reduced-ui-button" data-source="#diagnostics-detail"><span class="ui-icon ui-icon-clipboard"></span> ' . gTxt('copy_to_clipboard') . '</button>
                     </span>',
                 'diag_clear_private', 'diag_clear_private', array('class' => 'txp-form-field'),
                 ''
diff --git a/textpattern/textpattern.js b/textpattern/textpattern.js
@@ -2595,11 +2595,6 @@ textpattern.Route.add('diag', function () {
         $('#diagnostics-detail').val(diag_data);
     });
     $('#diag_clear_private').change();
-    $('.action-copy').click(function(ev) {
-        ev.preventDefault();
-        let diagData = $('#diagnostics-detail').val();
-        navigator.clipboard.writeText(diagData);
-    });
 });
 
 // Languages panel.
@@ -2848,6 +2843,17 @@ textpattern.Route.add('', function () {
         }
     });
 
+    // Copy to clipboard.
+    $('.action-copy-clip').click(function(ev) {
+        ev.preventDefault();
+        let source = $(this).data('source');
+        let $source = $(source);
+
+        if ($source.length) {
+            navigator.clipboard.writeText($source.val());
+        }
+    });
+
     // Pane states
     var hasTabs = $('.txp-layout:has(.switcher-list li a[data-txp-pane])');
 
