From d47c8a9e10171bfa66afcddde400c91454a540ce Mon Sep 17 00:00:00 2001
From: "D. Berge" <dberge@aaltronav.eu>
Date: Sun, 27 Feb 2022 19:42:57 +0100
Subject: [PATCH] Add (disabled) active row highlighter.

It implements the same functionality as in other tabs
such as sequences, lines, etc., but it is disabled here
because in my opinion it doesn't look too nice.

It will probably be a matter of enabling it at some point
and asking for feedback on user preference.
---
 lib/www/client/source/src/views/Log.vue | 19 ++++++++++++++++++-
 1 file changed, 18 insertions(+), 1 deletion(-)

diff --git a/lib/www/client/source/src/views/Log.vue b/lib/www/client/source/src/views/Log.vue
index df3ab9f..a6f799d 100644
--- a/lib/www/client/source/src/views/Log.vue
+++ b/lib/www/client/source/src/views/Log.vue
@@ -100,6 +100,7 @@
     :items="rows"
     :items-per-page.sync="itemsPerPage"
     item-key="key"
+    :item-class="(item) => (activeItem == item) ? 'align-top blue accent-1 elevation-3' : 'align-top'"
     sort-by="tstamp"
     :sort-desc="true"
     :search="filter"
@@ -107,6 +108,7 @@
     :loading="loading"
     fixed-header
     :footer-props='{itemsPerPageOptions: [ 10, 25, 50, 100, 500, -1 ]}'
+    @click:row="setActiveItem"
     >
 
       <template v-slot:item.tstamp="{value}">
@@ -253,7 +255,10 @@ export default {
       editedEvent: {},
       labelSearch: null,
       queuedReload: false,
-      itemsPerPage: 25
+      itemsPerPage: 25,
+
+      // Row highlighter
+      activeItem: null,
     }
   },
   
@@ -566,6 +571,18 @@ export default {
       }
     },
 
+    setActiveItem (item) {
+    // Disable setting the active item for now,
+    // it's kind of annoying.
+      return;
+
+      /*
+      this.activeItem = this.activeItem == item
+        ? null
+        : item;
+      */
+    },
+
     ...mapActions(["api", "showSnack"])
   },