From 25c901e013a4528706493c00bb13e1605e0d4621 Mon Sep 17 00:00:00 2001
From: Coteh <3276350+Coteh@users.noreply.github.com>
Date: Sun, 16 Jan 2022 03:04:56 -0500
Subject: [PATCH] Add jump to context WIP

---
 assets/components.d.ts          |  1 +
 assets/components/LogViewer.vue | 59 ++++++++++++++++++++++++++++++---
 assets/components/Search.vue    |  7 +---
 assets/composables/search.ts    | 13 +++++++-
 4 files changed, 69 insertions(+), 11 deletions(-)

diff --git a/assets/components.d.ts b/assets/components.d.ts
index 436ac4a7..df6d5256 100644
--- a/assets/components.d.ts
+++ b/assets/components.d.ts
@@ -6,6 +6,7 @@ declare module 'vue' {
   export interface GlobalComponents {
     CarbonCaretDown: typeof import('~icons/carbon/caret-down')['default']
     CilColumns: typeof import('~icons/cil/columns')['default']
+    CilFindInPage: typeof import('~icons/cil/find-in-page')['default']
     ContainerStat: typeof import('./components/ContainerStat.vue')['default']
     ContainerTitle: typeof import('./components/ContainerTitle.vue')['default']
     FuzzySearchModal: typeof import('./components/FuzzySearchModal.vue')['default']
diff --git a/assets/components/LogViewer.vue b/assets/components/LogViewer.vue
index d02fb4b3..d6c344bb 100644
--- a/assets/components/LogViewer.vue
+++ b/assets/components/LogViewer.vue
@@ -1,8 +1,23 @@
 <template>
   <ul class="events" :class="size">
-    <li v-for="item in filtered" :key="item.key" :data-event="item.event">
-      <span class="date" v-if="showTimestamp"> <relative-time :date="item.date"></relative-time></span>
-      <span class="text" v-html="colorize(item.message)"></span>
+    <li
+      v-for="item in filtered"
+      :key="item.key"
+      :data-event="item.event"
+      :data-key="item.key"
+      :class="item.selected ? 'selected' : ''"
+    >
+      <div>
+        <span class="date" v-if="showTimestamp"> <relative-time :date="item.date"></relative-time></span>
+        <span class="text" v-html="colorize(item.message)"></span>
+      </div>
+      <div class="line-options" v-if="isSearching()">
+        <o-button variant="primary" inverted size="small" type="button" class="jump" @click="jumpToLine">
+          <span class="icon">
+            <cil-find-in-page />
+          </span>
+        </o-button>
+      </div>
     </li>
   </ul>
 </template>
@@ -27,7 +42,27 @@ const ansiConvertor = new AnsiConvertor({ escapeXML: true });
 const colorize = (value: string) =>
   ansiConvertor.toHtml(value).replace("&lt;mark&gt;", "<mark>").replace("&lt;/mark&gt;", "</mark>");
 const { messages } = toRefs(props);
-const filtered = useSearchFilter().filteredMessages(messages);
+const { filteredMessages, searchFilter, resetSearch, isSearching } = useSearchFilter();
+const filtered = filteredMessages(messages);
+const jumpToLine = async (e) => {
+  const line = e.target.closest("li");
+  if (line.tagName !== "LI") {
+    return;
+  }
+  resetSearch();
+  for (const item of messages.value) {
+    item.selected = false;
+    if (item.key === line.dataset.key) {
+      item.selected = true;
+    }
+  }
+  // TODO prevent ScrollableView from automatically sticking to the bottom when closing search and jumping to context (ScrollableView.vue:47)
+  // if that could be achieved, then these timing hacks can be removed
+  await new Promise((resolve) => setTimeout(resolve, 10));
+  window.scrollTo(0, 0);
+  await new Promise((resolve) => setTimeout(resolve, 1000));
+  window.scrollTo(0, line.offsetTop - 200);
+};
 </script>
 <style scoped lang="scss">
 .events {
@@ -35,6 +70,7 @@ const filtered = useSearchFilter().filteredMessages(messages);
   font-family: SFMono-Regular, Consolas, Liberation Mono, monaco, Menlo, monospace;
 
   & > li {
+    display: flex;
     word-wrap: break-word;
     line-height: 130%;
     &:last-child {
@@ -47,6 +83,21 @@ const filtered = useSearchFilter().filteredMessages(messages);
     &[data-event="container-started"] {
       color: hsl(141, 53%, 53%);
     }
+    &.selected {
+      background-color: white;
+      color: black;
+    }
+    &.selected > .date {
+      background-color: white;
+    }
+    & > .line-options {
+      flex: 1;
+      display: flex;
+      flex-direction: row-reverse;
+      & > .jump {
+        cursor: pointer;
+      }
+    }
   }
 
   &.small {
diff --git a/assets/components/Search.vue b/assets/components/Search.vue
index 7f29ebd7..29edb860 100644
--- a/assets/components/Search.vue
+++ b/assets/components/Search.vue
@@ -29,7 +29,7 @@ import { useSearchFilter } from "@/composables/search";
 import { ref, nextTick, onMounted, onUnmounted } from "vue";
 
 const input = ref<HTMLInputElement>();
-const { searchFilter, showSearch } = useSearchFilter();
+const { searchFilter, showSearch, resetSearch } = useSearchFilter();
 
 onMounted(() => {
   hotkeys("command+f, ctrl+f", (event, handler) => {
@@ -46,11 +46,6 @@ onUnmounted(() => {
   hotkeys.unbind("command+f, ctrl+f");
   hotkeys.unbind("esc");
 });
-
-function resetSearch() {
-  searchFilter.value = "";
-  showSearch.value = false;
-}
 </script>
 
 <style lang="scss" scoped>
diff --git a/assets/composables/search.ts b/assets/composables/search.ts
index 4f514356..4f4b5a15 100644
--- a/assets/composables/search.ts
+++ b/assets/composables/search.ts
@@ -31,9 +31,20 @@ export function useSearchFilter() {
     });
   }
 
+  function resetSearch() {
+    searchFilter.value = "";
+    showSearch.value = false;
+  }
+
+  function isSearching() {
+    return showSearch.value && searchFilter.value;
+  }
+
   return {
     filteredMessages,
     searchFilter,
-    showSearch
+    showSearch,
+    resetSearch,
+    isSearching,
   };
 }