Fix the edit screen dialog error list

OpenC3 · Dec 20, 2024 · 1484870 · 1484870
1 parent 27a72df
commit 1484870
Show file tree

Hide file tree

Showing 8 changed files with 52 additions and 15 deletions.
diff --git a/openc3-cosmos-init/plugins/packages/openc3-vue-common/src/components/EditScreenDialog.vue b/openc3-cosmos-init/plugins/packages/openc3-vue-common/src/components/EditScreenDialog.vue
@@ -52,7 +52,7 @@
           </v-tooltip>
         </div>
       </v-toolbar>
-      <v-card-text>
+      <v-card-text style="max-height: 90vh">
         <v-row class="mt-3"> Upload a screen file. </v-row>
         <v-row no-gutters align="center">
           <v-btn
@@ -88,10 +88,13 @@
             </v-list>
           </v-menu>
         </v-row>
-        <v-row v-for="(error, index) in editErrors" :key="index" class="my-3">
-          <span class="text-red" v-text="error"></span>
+        <!-- Make the error messages a max height and scrollable -->
+        <v-row style="max-height: 120px; overflow-y: auto">
+          <div v-for="(error, index) in editErrors" :key="index">
+            <span class="text-red" v-text="error"></span>
+          </div>
         </v-row>
-        <v-row>
+        <v-row class="mt-5">
           <span
             >Ctrl-space brings up autocomplete. Right click keywords for
             documentation.</span
@@ -168,10 +171,8 @@ export default {
       if (this.errors.length !== 0) {
         let messages = new Set()
         let result = []
+        this.errors.sort((a, b) => a.lineNumber - b.lineNumber)
         for (const error of this.errors) {
-          if (messages.has(error.message)) {
-            continue
-          }
           let msg = `At ${error.lineNumber}: (${error.line}) ${error.message}.`
           if (error.usage) {
             msg += ` Usage: ${error.usage}`
@@ -322,7 +323,7 @@ export default {
 </style>
 <style scoped>
 .editor {
-  height: 50vh;
+  height: 45vh;
   width: 75vw;
   position: relative;
   font-size: 16px;

diff --git a/openc3-cosmos-init/plugins/packages/openc3-vue-common/src/components/Openc3Screen.vue b/openc3-cosmos-init/plugins/packages/openc3-vue-common/src/components/Openc3Screen.vue
@@ -163,7 +163,7 @@
     />
 
     <!-- Error dialog -->
-    <v-dialog v-model="errorDialog" max-width="600">
+    <v-dialog v-model="errorDialog" width="60vw">
       <v-toolbar height="24">
         <v-spacer />
         <span> Screen: {{ target }} {{ screen }} Errors </span>

diff --git a/...-cosmos-init/plugins/packages/openc3-vue-common/src/widgets/LabelvaluelimitsbarWidget.vue b/...-cosmos-init/plugins/packages/openc3-vue-common/src/widgets/LabelvaluelimitsbarWidget.vue
@@ -35,6 +35,8 @@
       :settings="[...appliedSettings]"
       :screen-values="screenValues"
       :widget-index="2"
+      :line="line"
+      :lineNumber="lineNumber"
     />
   </div>
 </template>

diff --git a/...smos-init/plugins/packages/openc3-vue-common/src/widgets/LabelvaluelimitscolumnWidget.vue b/...smos-init/plugins/packages/openc3-vue-common/src/widgets/LabelvaluelimitscolumnWidget.vue
@@ -34,6 +34,8 @@
       :settings="[...appliedSettings]"
       :screen-values="screenValues"
       :widget-index="2"
+      :line="line"
+      :lineNumber="lineNumber"
     />
     <value-widget
       v-bind="$attrs"

diff --git a/openc3-cosmos-init/plugins/packages/openc3-vue-common/src/widgets/LimitsbarWidget.vue b/openc3-cosmos-init/plugins/packages/openc3-vue-common/src/widgets/LimitsbarWidget.vue
@@ -60,12 +60,26 @@ export default {
           return `RL/${values[0]} YL/${values[1]} YH/${values[2]} RH/${values[3]} GL/${values[4]} GH/${values[5]}`
         }
       } else {
-        throw new Error(
-          `Item ${this.parameters.slice(0, 3).join(' ')} has no limits settings`,
-        )
+        // See errorCaptured in Openc3Screen.vue for how this is parsed
+        throw {
+          line: this.line,
+          lineNumber: this.lineNumber,
+          keyword: 'LIMITSBAR',
+          parameters: this.parameters,
+          message: 'Item has no limits settings',
+          usage: 'Only items with limits',
+        }
       }
     },
   },
+  created() {
+    this.verifyNumParams(
+      'LIMITSBAR',
+      3,
+      6,
+      'LIMITSBAR <TARGET> <PACKET> <ITEM> <TYPE> <WIDTH> <HEIGHT>',
+    )
+  },
 }
 </script>
 

diff --git a/openc3-cosmos-init/plugins/packages/openc3-vue-common/src/widgets/LimitscolumnWidget.vue b/openc3-cosmos-init/plugins/packages/openc3-vue-common/src/widgets/LimitscolumnWidget.vue
@@ -68,12 +68,26 @@ export default {
           '--blue-height': this.blue + '%',
         }
       } else {
-        throw new Error(
-          `Item ${this.parameters.slice(0, 3).join(' ')} has no limits settings`,
-        )
+        // See errorCaptured in Openc3Screen.vue for how this is parsed
+        throw {
+          line: this.line,
+          lineNumber: this.lineNumber,
+          keyword: 'LIMITSCOLUMN',
+          parameters: this.parameters,
+          message: 'Item has no limits settings',
+          usage: 'Only items with limits',
+        }
       }
     },
   },
+  created() {
+    this.verifyNumParams(
+      'LIMITSCOLUMN',
+      3,
+      6,
+      'LIMITSCOLUMN <TARGET> <PACKET> <ITEM> <TYPE> <WIDTH> <HEIGHT>',
+    )
+  },
 }
 </script>
 

diff --git a/openc3-cosmos-init/plugins/packages/openc3-vue-common/src/widgets/ValuelimitsbarWidget.vue b/openc3-cosmos-init/plugins/packages/openc3-vue-common/src/widgets/ValuelimitsbarWidget.vue
@@ -36,6 +36,8 @@
       :settings="[...appliedSettings]"
       :screen-values="screenValues"
       :widget-index="1"
+      :line="line"
+      :lineNumber="lineNumber"
     />
   </div>
 </template>

diff --git a/...c3-cosmos-init/plugins/packages/openc3-vue-common/src/widgets/ValuelimitscolumnWidget.vue b/...c3-cosmos-init/plugins/packages/openc3-vue-common/src/widgets/ValuelimitscolumnWidget.vue
@@ -29,6 +29,8 @@
       :settings="[...appliedSettings]"
       :screen-values="screenValues"
       :widget-index="1"
+      :line="line"
+      :lineNumber="lineNumber"
     />
     <value-widget
       v-bind="$attrs"