From b92099858b1caa5ba04992a42c6897a58edc2d71 Mon Sep 17 00:00:00 2001
From: Ryan Pratt <ryan.pratt@openc3.com>
Date: Wed, 20 Nov 2024 15:08:14 -0700
Subject: [PATCH 1/7] Fix cmdtlmserver raw-dialog z-index

---
 .../src/tools/CmdTlmServer/CmdTlmServer.vue                     | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/openc3-cosmos-init/plugins/packages/openc3-cosmos-tool-cmdtlmserver/src/tools/CmdTlmServer/CmdTlmServer.vue b/openc3-cosmos-init/plugins/packages/openc3-cosmos-tool-cmdtlmserver/src/tools/CmdTlmServer/CmdTlmServer.vue
index d9fce864dc..80b949246e 100644
--- a/openc3-cosmos-init/plugins/packages/openc3-cosmos-tool-cmdtlmserver/src/tools/CmdTlmServer/CmdTlmServer.vue
+++ b/openc3-cosmos-init/plugins/packages/openc3-cosmos-tool-cmdtlmserver/src/tools/CmdTlmServer/CmdTlmServer.vue
@@ -22,7 +22,7 @@
 
 <template>
   <top-bar :menus="menus" :title="title" />
-  <v-card>
+  <v-card style="z-index: 2 !important">
     <v-expansion-panels v-model="panel">
       <v-expansion-panel>
         <v-expansion-panel-title>

From 7893832ea60cae7084e03961e7b71d88391c8dc1 Mon Sep 17 00:00:00 2001
From: Ryan Pratt <ryan.pratt@openc3.com>
Date: Wed, 20 Nov 2024 15:29:57 -0700
Subject: [PATCH 2/7] Fix raw-dialog resize and scroll behavior

---
 .../src/tools/CmdTlmServer/RawDialog.vue      | 93 ++++++++++---------
 1 file changed, 47 insertions(+), 46 deletions(-)

diff --git a/openc3-cosmos-init/plugins/packages/openc3-cosmos-tool-cmdtlmserver/src/tools/CmdTlmServer/RawDialog.vue b/openc3-cosmos-init/plugins/packages/openc3-cosmos-tool-cmdtlmserver/src/tools/CmdTlmServer/RawDialog.vue
index c107da3815..d9e56b03fa 100644
--- a/openc3-cosmos-init/plugins/packages/openc3-cosmos-tool-cmdtlmserver/src/tools/CmdTlmServer/RawDialog.vue
+++ b/openc3-cosmos-init/plugins/packages/openc3-cosmos-tool-cmdtlmserver/src/tools/CmdTlmServer/RawDialog.vue
@@ -27,44 +27,44 @@
     class="raw-dialog"
     ref="rawDialog"
   >
+    <div ref="bar" class="toolbar-wrapper">
+      <v-toolbar height="24">
+        <v-tooltip location="top">
+          <template v-slot:activator="{ props }">
+            <div v-bind="props">
+              <v-icon data-test="copy-icon" @click="copyRawData">
+                mdi-content-copy
+              </v-icon>
+            </div>
+          </template>
+          <span> Copy </span>
+        </v-tooltip>
+        <v-tooltip location="top">
+          <template v-slot:activator="{ props }">
+            <div v-bind="props">
+              <v-icon data-test="download" @click="downloadRawData">
+                mdi-download
+              </v-icon>
+            </div>
+          </template>
+          <span> Download </span>
+        </v-tooltip>
+        <v-spacer />
+        <span> {{ type }} </span>
+        <v-spacer />
+        <v-tooltip location="top">
+          <template v-slot:activator="{ props }">
+            <div v-bind="props">
+              <v-icon data-test="close" @click="$emit('close')">
+                mdi-close-box
+              </v-icon>
+            </div>
+          </template>
+          <span> Close </span>
+        </v-tooltip>
+      </v-toolbar>
+    </div>
     <v-card>
-      <div ref="bar">
-        <v-toolbar height="24">
-          <v-tooltip location="top">
-            <template v-slot:activator="{ props }">
-              <div v-bind="props">
-                <v-icon data-test="copy-icon" @click="copyRawData">
-                  mdi-content-copy
-                </v-icon>
-              </div>
-            </template>
-            <span> Copy </span>
-          </v-tooltip>
-          <v-tooltip location="top">
-            <template v-slot:activator="{ props }">
-              <div v-bind="props">
-                <v-icon data-test="download" @click="downloadRawData">
-                  mdi-download
-                </v-icon>
-              </div>
-            </template>
-            <span> Download </span>
-          </v-tooltip>
-          <v-spacer />
-          <span> {{ type }} </span>
-          <v-spacer />
-          <v-tooltip location="top">
-            <template v-slot:activator="{ props }">
-              <div v-bind="props">
-                <v-icon data-test="close" @click="$emit('close')">
-                  mdi-close-box
-                </v-icon>
-              </div>
-            </template>
-            <span> Close </span>
-          </v-tooltip>
-        </v-toolbar>
-      </div>
       <v-card-title class="d-flex align-center justify-content-space-between">
         <span> {{ header }} </span>
         <v-spacer />
@@ -200,9 +200,8 @@ export default {
       this.dragX = e.clientX
       this.dragY = e.clientY
       // set the element's new position:
-      this.top = this.$refs.bar.parentElement.parentElement.offsetTop - yOffset
-      this.left =
-        this.$refs.bar.parentElement.parentElement.offsetLeft - xOffset
+      this.top = this.$refs.bar.parentElement.offsetTop - yOffset
+      this.left = this.$refs.bar.parentElement.offsetLeft - xOffset
     },
     closeDragElement: function () {
       // stop moving when mouse button is released
@@ -338,18 +337,20 @@ export default {
   border-color: white;
   resize: both;
   overflow: auto;
+  min-height: 28px;
   max-height: 85vh;
+  min-width: 800px;
   background-color: var(--color-background-base-selected);
 }
-.raw-dialog :deep(.v-card) {
-  height: 100%;
-  min-width: 800px;
+.raw-dialog .toolbar-wrapper {
+  position: sticky;
+  top: 0;
+  z-index: 1;
 }
-.raw-dialog :deep(.v-card__text) {
-  height: 100%;
+.raw-dialog .v-card-text {
   background-color: var(--color-background-base-selected);
 }
-.v-textarea :deep(textarea) {
+.v-textarea textarea {
   margin-top: 10px;
   font-family: 'Courier New', Courier, monospace;
   overflow-y: scroll;

From 5ff319162b1b3065106de9095af319fba2156f98 Mon Sep 17 00:00:00 2001
From: Ryan Pratt <ryan.pratt@openc3.com>
Date: Wed, 20 Nov 2024 15:35:15 -0700
Subject: [PATCH 3/7] fix deep selectors I just broke

---
 .../src/tools/CmdTlmServer/RawDialog.vue                      | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/openc3-cosmos-init/plugins/packages/openc3-cosmos-tool-cmdtlmserver/src/tools/CmdTlmServer/RawDialog.vue b/openc3-cosmos-init/plugins/packages/openc3-cosmos-tool-cmdtlmserver/src/tools/CmdTlmServer/RawDialog.vue
index d9e56b03fa..138fe7602e 100644
--- a/openc3-cosmos-init/plugins/packages/openc3-cosmos-tool-cmdtlmserver/src/tools/CmdTlmServer/RawDialog.vue
+++ b/openc3-cosmos-init/plugins/packages/openc3-cosmos-tool-cmdtlmserver/src/tools/CmdTlmServer/RawDialog.vue
@@ -347,10 +347,10 @@ export default {
   top: 0;
   z-index: 1;
 }
-.raw-dialog .v-card-text {
+.raw-dialog :deep(.v-card-text) {
   background-color: var(--color-background-base-selected);
 }
-.v-textarea textarea {
+.v-textarea :deep(textarea) {
   margin-top: 10px;
   font-family: 'Courier New', Courier, monospace;
   overflow-y: scroll;

From eedd8ecd2e0a2e724dbc15cf4a8350e5492ec1ca Mon Sep 17 00:00:00 2001
From: Ryan Pratt <ryan.pratt@openc3.com>
Date: Thu, 21 Nov 2024 14:32:57 -0700
Subject: [PATCH 4/7] Fix raw-dialog textarea scroll and wrapping

---
 .../src/tools/CmdTlmServer/RawDialog.vue                       | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/openc3-cosmos-init/plugins/packages/openc3-cosmos-tool-cmdtlmserver/src/tools/CmdTlmServer/RawDialog.vue b/openc3-cosmos-init/plugins/packages/openc3-cosmos-tool-cmdtlmserver/src/tools/CmdTlmServer/RawDialog.vue
index 138fe7602e..bbb41c2f89 100644
--- a/openc3-cosmos-init/plugins/packages/openc3-cosmos-tool-cmdtlmserver/src/tools/CmdTlmServer/RawDialog.vue
+++ b/openc3-cosmos-init/plugins/packages/openc3-cosmos-tool-cmdtlmserver/src/tools/CmdTlmServer/RawDialog.vue
@@ -339,7 +339,7 @@ export default {
   overflow: auto;
   min-height: 28px;
   max-height: 85vh;
-  min-width: 800px;
+  min-width: 830px;
   background-color: var(--color-background-base-selected);
 }
 .raw-dialog .toolbar-wrapper {
@@ -353,6 +353,5 @@ export default {
 .v-textarea :deep(textarea) {
   margin-top: 10px;
   font-family: 'Courier New', Courier, monospace;
-  overflow-y: scroll;
 }
 </style>

From 52845f78031c10eedfc79a624f6130a0451d82de Mon Sep 17 00:00:00 2001
From: Ryan Pratt <ryan.pratt@openc3.com>
Date: Thu, 21 Nov 2024 16:28:59 -0700
Subject: [PATCH 5/7] Fix v-textarea sizing

---
 .../src/tools/CmdTlmServer/RawDialog.vue         | 16 +++++++++++++---
 .../src/tools/CmdTlmServer/Updater.js            |  2 +-
 2 files changed, 14 insertions(+), 4 deletions(-)

diff --git a/openc3-cosmos-init/plugins/packages/openc3-cosmos-tool-cmdtlmserver/src/tools/CmdTlmServer/RawDialog.vue b/openc3-cosmos-init/plugins/packages/openc3-cosmos-tool-cmdtlmserver/src/tools/CmdTlmServer/RawDialog.vue
index bbb41c2f89..e5debcda3e 100644
--- a/openc3-cosmos-init/plugins/packages/openc3-cosmos-tool-cmdtlmserver/src/tools/CmdTlmServer/RawDialog.vue
+++ b/openc3-cosmos-init/plugins/packages/openc3-cosmos-tool-cmdtlmserver/src/tools/CmdTlmServer/RawDialog.vue
@@ -99,7 +99,13 @@
             <span> {{ receivedCount }} </span>
           </v-col>
         </v-row>
-        <v-textarea v-model="rawData" class="pa-0 ma-0" auto-grow readonly />
+        <v-textarea
+          class="pa-0 ma-0"
+          v-model="rawData"
+          :rows="numRows"
+          no-resize
+          readonly
+        />
       </v-card-text>
     </v-card>
   </div>
@@ -172,6 +178,10 @@ export default {
       style['z-index'] = this.zIndex
       return style
     },
+    numRows() {
+      // This is because v-textarea doesn't behave correctly with really long monospace text
+      return this.rawData.split('\n').length
+    },
   },
   mounted() {
     this.$refs.bar.onmousedown = this.dragMouseDown
@@ -335,11 +345,11 @@ export default {
   border: solid;
   border-width: 1px;
   border-color: white;
-  resize: both;
+  resize: vertical;
   overflow: auto;
   min-height: 28px;
   max-height: 85vh;
-  min-width: 830px;
+  width: 815px;
   background-color: var(--color-background-base-selected);
 }
 .raw-dialog .toolbar-wrapper {
diff --git a/openc3-cosmos-init/plugins/packages/openc3-cosmos-tool-cmdtlmserver/src/tools/CmdTlmServer/Updater.js b/openc3-cosmos-init/plugins/packages/openc3-cosmos-tool-cmdtlmserver/src/tools/CmdTlmServer/Updater.js
index ac17cac298..b47fc551c8 100644
--- a/openc3-cosmos-init/plugins/packages/openc3-cosmos-tool-cmdtlmserver/src/tools/CmdTlmServer/Updater.js
+++ b/openc3-cosmos-init/plugins/packages/openc3-cosmos-tool-cmdtlmserver/src/tools/CmdTlmServer/Updater.js
@@ -40,7 +40,7 @@ export default {
   mounted() {
     this.changeUpdater()
   },
-  beforeDestroy() {
+  beforeUnmount() {
     if (this.updater != null) {
       clearInterval(this.updater)
       this.updater = null

From 55545073c32499a4747e52e405c22fda10b2bbd7 Mon Sep 17 00:00:00 2001
From: Ryan Pratt <ryan.pratt@openc3.com>
Date: Thu, 21 Nov 2024 17:13:34 -0700
Subject: [PATCH 6/7] Remove remaining beforeDestroys

---
 .../openc3-tool-common/src/components/CriticalCmdDialog.vue     | 2 +-
 .../openc3-tool-common/src/components/EditScreenDialog.vue      | 2 +-
 .../packages/openc3-tool-common/src/tools/admin/EditDialog.vue  | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/openc3-cosmos-init/plugins/packages/openc3-tool-common/src/components/CriticalCmdDialog.vue b/openc3-cosmos-init/plugins/packages/openc3-tool-common/src/components/CriticalCmdDialog.vue
index eca4d2add2..774962b760 100644
--- a/openc3-cosmos-init/plugins/packages/openc3-tool-common/src/components/CriticalCmdDialog.vue
+++ b/openc3-cosmos-init/plugins/packages/openc3-tool-common/src/components/CriticalCmdDialog.vue
@@ -122,7 +122,7 @@ export default {
       return !this.formValid && !this.canApprove
     },
   },
-  beforeDestroy() {
+  beforeUnmount() {
     clearInterval(this.updater)
     this.updater = null
     this.canApprove = false
diff --git a/openc3-cosmos-init/plugins/packages/openc3-tool-common/src/components/EditScreenDialog.vue b/openc3-cosmos-init/plugins/packages/openc3-tool-common/src/components/EditScreenDialog.vue
index 26cf5f1605..f48aa042ad 100644
--- a/openc3-cosmos-init/plugins/packages/openc3-tool-common/src/components/EditScreenDialog.vue
+++ b/openc3-cosmos-init/plugins/packages/openc3-tool-common/src/components/EditScreenDialog.vue
@@ -208,7 +208,7 @@ export default {
     this.editor.clearSelection()
     this.editor.focus()
   },
-  beforeDestroy() {
+  beforeUnmount() {
     this.editor.destroy()
     this.editor.container.remove()
   },
diff --git a/openc3-cosmos-init/plugins/packages/openc3-tool-common/src/tools/admin/EditDialog.vue b/openc3-cosmos-init/plugins/packages/openc3-tool-common/src/tools/admin/EditDialog.vue
index f272c55f22..808c8e3b3c 100644
--- a/openc3-cosmos-init/plugins/packages/openc3-tool-common/src/tools/admin/EditDialog.vue
+++ b/openc3-cosmos-init/plugins/packages/openc3-tool-common/src/tools/admin/EditDialog.vue
@@ -146,7 +146,7 @@ export default {
       this.editor.setReadOnly(true)
     }
   },
-  beforeDestroy() {
+  beforeUnmount() {
     if (this.editor) {
       this.editor.destroy()
     }

From 60973a35ddef7183de92bd555dc8e5caf92ecbfd Mon Sep 17 00:00:00 2001
From: Jason Thomas <jason@openc3.com>
Date: Fri, 22 Nov 2024 11:48:48 -0700
Subject: [PATCH 7/7] Always hide scrollbar on textarea, add rows fudge factor

---
 .../src/tools/CmdTlmServer/RawDialog.vue                   | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/openc3-cosmos-init/plugins/packages/openc3-cosmos-tool-cmdtlmserver/src/tools/CmdTlmServer/RawDialog.vue b/openc3-cosmos-init/plugins/packages/openc3-cosmos-tool-cmdtlmserver/src/tools/CmdTlmServer/RawDialog.vue
index e5debcda3e..b1671e7766 100644
--- a/openc3-cosmos-init/plugins/packages/openc3-cosmos-tool-cmdtlmserver/src/tools/CmdTlmServer/RawDialog.vue
+++ b/openc3-cosmos-init/plugins/packages/openc3-cosmos-tool-cmdtlmserver/src/tools/CmdTlmServer/RawDialog.vue
@@ -13,7 +13,7 @@
 # GNU Affero General Public License for more details.
 
 # Modified by OpenC3, Inc.
-# All changes Copyright 2023, OpenC3, Inc.
+# All changes Copyright 2024, OpenC3, Inc.
 # All Rights Reserved
 #
 # This file may also be used under the terms of a commercial license
@@ -180,7 +180,9 @@ export default {
     },
     numRows() {
       // This is because v-textarea doesn't behave correctly with really long monospace text
-      return this.rawData.split('\n').length
+      let lines = this.rawData.split('\n').length
+      // Add a small fudge factor every 2000 lines to prevent clipping at the bottom
+      return lines + Math.floor(lines / 2000)
     },
   },
   mounted() {
@@ -363,5 +365,6 @@ export default {
 .v-textarea :deep(textarea) {
   margin-top: 10px;
   font-family: 'Courier New', Courier, monospace;
+  overflow-y: hidden;
 }
 </style>