tdf#127566: don't change divcopyable border width in onclick handler

Instead, define border width in respective CSS, to make it consistent
in both clicked and non-clicked states, and avoid "jumping" on click.

Change-Id: Ic8ab8d0fd586059e21de6aa866fef95226774acf
Reviewed-on: https://gerrit.libreoffice.org/c/help/+/87718
Tested-by: Jenkins
Reviewed-by: Olivier Hallot <olivier.hallot@libreoffice.org>

2 files changed, 3 insertions, 4 deletions

diff --git a/help3xsl/default.css b/help3xsl/default.css
index eeaa160d12..fe70e4262d 100644
--- a/help3xsl/default.css
+++ b/help3xsl/default.css
@@ -296,6 +296,7 @@ h6 {
 
 .pycode,
 .bascode {
+    border: solid 5px transparent;
     margin-top: 15px;
     margin-bottom: 15px;
 }
diff --git a/help3xsl/help.js b/help3xsl/help.js
index 75541e6745..48767222e3 100644
--- a/help3xsl/help.js
+++ b/help3xsl/help.js
@@ -106,19 +106,17 @@ for (var i = 0, len = itemcopyable.length; i < len; i++) {
     (function() {
         var item = itemcopyable[i];
 
-        function changeBorder(item, color, colorToChangeBackTo) {
+        function changeBorder(item, color) {
             var saveBorder  = item.style.border;
-            item.style.border = "solid 5px";
             item.style.borderColor = color;
 
             setTimeout(function() {
                 item.style.border = saveBorder;
-                item.style.borderColor = colorToChangeBackTo;
             }, 150);
         }
         item.onclick = function() {
             document.execCommand("copy");
-            changeBorder(item, "#18A303", "transparent");
+            changeBorder(item, "#18A303");
         };
         item.addEventListener("copy", function(event) {
             event.preventDefault();