From b8bf8fbc183095dfe839b495b9eca0d96b011254 Mon Sep 17 00:00:00 2001
From: jaknapper <jaknapper@hotmail.com>
Date: Mon, 28 Jul 2025 12:21:47 +0100
Subject: [PATCH 1/2] Log display pauses on hover

---
 .../labThingsComponents/actionLogDisplay.vue  | 57 ++++++++++++++++++-
 1 file changed, 54 insertions(+), 3 deletions(-)

diff --git a/webapp/src/components/labThingsComponents/actionLogDisplay.vue b/webapp/src/components/labThingsComponents/actionLogDisplay.vue
index db8d4535..de02fcaa 100644
--- a/webapp/src/components/labThingsComponents/actionLogDisplay.vue
+++ b/webapp/src/components/labThingsComponents/actionLogDisplay.vue
@@ -1,4 +1,9 @@
 <template>
+  <div
+    class="log-wrapper"
+    @mouseenter="onMouseEnter"
+    @mouseleave="onMouseLeave"
+  >
   <div
     ref="logContainer"
     class="log-container uk-margin-left uk-margin-right uk-margin"
@@ -16,6 +21,12 @@
     </div>
     <div v-if="taskStatus == 'completed'" class="uk-alert uk-alert-success">
       The task completed successfully.
+      </div>
+      
+      <!-- Paused banner outside scroll container, positioned relative to wrapper -->
+    <div v-if="userIsHovering" class="paused-banner">
+      Auto-scroll paused
+      </div>
     </div>
   </div>
 </template>
@@ -35,6 +46,12 @@ export default {
     }
   },
 
+  data() {
+    return {
+      userIsHovering: false,
+    };
+  },
+
   watch: {
     log: function() {
       this.scrollToBottom();
@@ -45,9 +62,18 @@ export default {
   },
 
   methods: {
+    onMouseEnter() {
+      this.userIsHovering = true;
+    },
+    onMouseLeave() {
+      this.userIsHovering = false;
+    },
+
     scrollToBottom() {
-      this.$nextTick(function() {
-        let viewer = this.$refs.logContainer;
+      this.$nextTick(() => {
+        if (this.userIsHovering) return;
+
+        const viewer = this.$refs.logContainer;
         viewer.scrollTop = viewer.scrollHeight;
       });
     }
@@ -56,13 +82,38 @@ export default {
 </script>
 
 <style scoped>
-.log-container {
+.log-wrapper {
   position: relative;
+  height: 200px;
+  overflow: hidden;
+  margin-bottom: 1em;
+  margin-top: 1em;
+}
+
+.log-container {
+  height: 100%;
   overflow-y: auto;
   overflow-x: auto;
   background-color: white;
   color: black;
   padding: 0.5em;
   border: 1px solid black;
+  box-sizing: border-box;
+}
+
+/* Floating banner fixed at top-right of wrapper, not affected by scroll */
+.paused-banner {
+  position: absolute;
+  top: 8px;
+  right: 40px;
+  color: black;
+  font-weight: bold;
+  padding: 4px;
+  border-radius: 4px;
+  z-index: 10;
+  font-size: 0.85em;
+  pointer-events: none;
+  box-shadow: 0 0 4px rgba(0, 0, 0, 0.2);
+  background-color: white;
 }
 </style>

From a8137fc6a06545de2d2b28d067fa47f5180d326e Mon Sep 17 00:00:00 2001
From: Joe Knapper <joe.knapper@glasgow.ac.uk>
Date: Mon, 28 Jul 2025 12:03:21 +0000
Subject: [PATCH 2/2] Apply suggestions from code review of branch pause-logger

Co-authored-by: Julian Stirling <julian@julianstirling.co.uk>
---
 webapp/src/components/labThingsComponents/actionLogDisplay.vue | 1 +
 1 file changed, 1 insertion(+)

diff --git a/webapp/src/components/labThingsComponents/actionLogDisplay.vue b/webapp/src/components/labThingsComponents/actionLogDisplay.vue
index de02fcaa..1d111287 100644
--- a/webapp/src/components/labThingsComponents/actionLogDisplay.vue
+++ b/webapp/src/components/labThingsComponents/actionLogDisplay.vue
@@ -70,6 +70,7 @@ export default {
     },
 
     scrollToBottom() {
+    /*Scroll to bottom of log unless the user is hovering over the log.*/
       this.$nextTick(() => {
         if (this.userIsHovering) return;