[Inspector Tabs] Updates

e2e/helper/planningUtils.js

@@ -224,7 +224,7 @@ export async function createTimelistWithPlanAndSetActivityInProgress(page, planJ
   await page.getByRole('button', { name: 'Edit Object' }).click();
 
   // Find the display properties section in the inspector
-  await page.getByRole('tab', { name: 'View Properties' }).click();
+  await page.getByRole('tab', { name: 'Config' }).click();
   // Switch to expanded view and save the setting
   await page.getByLabel('Display Style').selectOption({ label: 'Expanded' });
 

e2e/tests/functional/planning/timelist.e2e.spec.js

@@ -132,7 +132,7 @@ test("View a timelist in expanded view, verify all the activities are displayed
     await page.getByRole('button', { name: 'Edit Object' }).click();
 
     // Find the display properties section in the inspector
-    await page.getByRole('tab', { name: 'View Properties' }).click();
+    await page.getByRole('tab', { name: 'Config' }).click();
     // Switch to expanded view and save the setting
     await page.getByLabel('Display Style').selectOption({ label: 'Expanded' });
 

e2e/tests/functional/plugins/lad/lad.e2e.spec.js

@@ -53,7 +53,7 @@ test.describe('Testing LAD table configuration', () => {
   test('in edit mode, LAD Tables provide ability to hide columns', async ({ page }) => {
     // Edit LAD table
     await page.getByLabel('Edit Object').click();
-    await page.getByRole('tab', { name: 'LAD Table Configuration' }).click();
+    await page.getByRole('tab', { name: 'Config' }).click();
 
     // make sure headers are visible initially
     await expect(page.getByRole('columnheader', { name: 'Timestamp', exact: true })).toBeVisible();
@@ -114,7 +114,7 @@ test.describe('Testing LAD table configuration', () => {
 
     // Edit LAD table
     await page.getByLabel('Edit Object').click();
-    await page.getByRole('tab', { name: 'LAD Table Configuration' }).click();
+    await page.getByRole('tab', { key: 'lad-table-configuration' }).click();
 
     // show timestamp column
     await page.getByLabel('Timestamp', { exact: true }).check();
@@ -142,7 +142,7 @@ test.describe('Testing LAD table configuration', () => {
 
     // Edit LAD table
     await page.getByLabel('Edit Object').click();
-    await page.getByRole('tab', { name: 'LAD Table Configuration' }).click();
+    await page.getByRole('tab', { key: 'lad-table-configuration' }).click();
 
     // show units, type, and WATCH columns
     await page.getByLabel('Units').check();
@@ -182,7 +182,7 @@ test.describe('Testing LAD table configuration', () => {
 
     // Edit LAD table
     await page.getByLabel('Edit Object').click();
-    await page.getByRole('tab', { name: 'LAD Table Configuration' }).click();
+    await page.getByRole('tab', { key: 'lad-table-configuration' }).click();
 
     // make sure Sine Wave headers are visible initially too
     await expect(page.getByRole('columnheader', { name: 'Timestamp', exact: true })).toBeVisible();

src/api/annotation/AnnotationAPI.js

@@ -95,6 +95,7 @@ export default class AnnotationAPI extends EventEmitter {
     this.availableTags = {};
     this.namespaceToSaveAnnotations = '';
     this.#targetComparatorMap = new Map();
+    this.annotatableTypes = [];
 
     this.ANNOTATION_TYPES = ANNOTATION_TYPES;
     this.ANNOTATION_TYPE = ANNOTATION_TYPE;
@@ -115,6 +116,17 @@ export default class AnnotationAPI extends EventEmitter {
         domainObject.annotationType = domainObject.annotationType || 'plotspatial';
       }
     });
+
+    this.openmct.on('start', () => {
+      const types = this.openmct.types.getTypes();
+      const typeKeys = Object.keys(types);
+
+      typeKeys.forEach((key) => {
+        if (types[key].definition.annotatable) {
+          this.annotatableTypes.push(key);
+        }
+      });
+    });
   }
   /**
    * Creates an annotation on a given domain object (e.g., a plot) and a set of targets (e.g., telemetry objects)
@@ -582,4 +594,13 @@ export default class AnnotationAPI extends EventEmitter {
       _.isEqual(targets, otherTargets)
     );
   }
+
+  /**
+   * Checks if the given type is annotatable
+   * @param {string} type The type to check
+   * @returns {boolean} Returns true if the type is annotatable
+   */
+  isAnnotatableType(type) {
+    return this.annotatableTypes.some((annotatableType) => annotatableType === type);
+  }
 }

src/api/telemetry/TelemetryAPI.js

@@ -284,6 +284,33 @@ export default class TelemetryAPI {
     return value;
   }
 
+  /**
+   * Determines whether a domain object has numeric telemetry data.
+   * A domain object has numeric telemetry if it:
+   * 1. Has a telemetry property
+   * 2. Has telemetry metadata with domain values (like timestamps)
+   * 3. Has range values (measurements) where at least one is numeric
+   *
+   * @method hasNumericTelemetry
+   * @param {import('openmct').DomainObject} domainObject The domain object to check
+   * @returns {boolean} True if the object has numeric telemetry, false otherwise
+   */
+  hasNumericTelemetry(domainObject) {
+    if (!Object.prototype.hasOwnProperty.call(domainObject, 'telemetry')) {
+      return false;
+    }
+
+    const metadata = this.openmct.telemetry.getMetadata(domainObject);
+    const rangeValues = metadata.valuesForHints(['range']);
+    const domains = metadata.valuesForHints(['domain']);
+
+    return (
+      domains.length > 0 &&
+      rangeValues.length > 0 &&
+      !rangeValues.every((value) => value.format === 'string')
+    );
+  }
+
   /**
    * Generates a numeric hash value for an options object. The hash is consistent
    * for equivalent option objects regardless of property order.

src/api/types/TypeRegistry.js

@@ -89,6 +89,17 @@ export default class TypeRegistry {
   get(typeKey) {
     return this.types[typeKey] || UNKNOWN_TYPE;
   }
+  /**
+   * List all registered types.
+   * @returns {Type[]} all registered types
+   */
+  getTypes() {
+    return this.types;
+  }
+  /**
+   * Import legacy types.
+   * @param {TypeDefinition[]} types the types to import
+   */
   importLegacyTypes(types) {
     types
       .filter((t) => this.get(t.key) === UNKNOWN_TYPE)

src/plugins/LADTable/LADTableConfiguration.js

@@ -41,9 +41,10 @@ export default class LADTableConfiguration extends EventEmitter {
   }
 
   getConfiguration() {
-    const configuration = this.domainObject.configuration || {};
-    configuration.hiddenColumns = configuration.hiddenColumns || {};
+    const configuration = this.domainObject.configuration ?? {};
+    configuration.hiddenColumns = configuration.hiddenColumns ?? {};
     configuration.isFixedLayout = configuration.isFixedLayout ?? true;
+    configuration.objectStyles = configuration.objectStyles ?? {};
 
     return configuration;
   }

src/plugins/LADTable/LADTableConfigurationViewProvider.js

@@ -27,7 +27,7 @@
 export default function LADTableConfigurationViewProvider(openmct) {
   return {
     key: 'lad-table-configuration',
-    name: 'LAD Table Configuration',
+    name: 'Config',
     canView(selection) {
       if (selection.length !== 1 || selection[0].length === 0) {
         return false;
@@ -61,7 +61,7 @@
           _destroy = destroy;
         },
         priority() {
-          return 1;
+          return openmct.editor.isEditing() ? openmct.priority.HIGH + 1 : openmct.priority.DEFAULT;
         },
         destroy() {
           if (_destroy) {

src/plugins/LADTable/components/LadTableConfiguration.vue

@@ -22,28 +22,24 @@
 
 <template>
   <div class="c-inspect-properties">
-    <template v-if="isEditing">
-      <div class="c-inspect-properties__header">Table Column Visibility</div>
-      <ul class="c-inspect-properties__section">
-        <li v-for="(title, key) in headers" :key="key" class="c-inspect-properties__row">
-          <div class="c-inspect-properties__label" title="Show or hide column">
-            <label :for="key + 'ColumnControl'">{{ title }}</label>
-          </div>
-          <div class="c-inspect-properties__value">
-            <input
-              :id="key + 'ColumnControl'"
-              type="checkbox"
-              :checked="configuration.hiddenColumns[key] !== true"
-              @change="toggleColumn(key)"
-            />
-          </div>
-        </li>
-      </ul>
-    </template>
-    <template v-else>
-      <div class="c-inspect-properties__header">LAD Table Configuration</div>
-      <div class="c-inspect-properties__row--span-all">Only available in edit mode.</div>
-    </template>
+    <div class="c-inspect-properties__header">Table Column Visibility</div>
+    <ul class="c-inspect-properties__section">
+      <li v-for="(title, key) in headers" :key="key" class="c-inspect-properties__row">
+        <div class="c-inspect-properties__label" title="Show or hide column">
+          <label :for="key + 'ColumnControl'">{{ title }}</label>
+        </div>
+        <div class="c-inspect-properties__value">
+          <input
+            :id="key + 'ColumnControl'"
+            type="checkbox"
+            v-if="isEditing"
+            :checked="configuration.hiddenColumns[key] !== true"
+            @change="toggleColumn(key)"
+          />
+          <span v-if="!isEditing && configuration.hiddenColumns[key] !== true">Visible</span>
+        </div>
+      </li>
+    </ul>
   </div>
 </template>
 
@@ -62,7 +58,8 @@
       isEditing: this.openmct.editor.isEditing(),
       configuration: ladTableConfiguration.getConfiguration(),
       items: [],
-      ladTableObjects: []
+      ladTableObjects: [],
+      ladTelemetryObjects: {}
     };
   },
   computed: {
@@ -150,11 +147,14 @@
       this.ladTableObjects.push(ladTable);
 
       const composition = this.openmct.composition.get(ladTable.domainObject);
-
+      composition.on('add', this.addItem);
+      composition.on('remove', this.removeItem);
       composition.load();
 
       this.compositions.push({
-        composition
+        composition,
+        addCallback: this.addItem,
+        removeCallback: this.removeItem
       });
     },
     removeLadTable(identifier) {

src/plugins/LADTable/plugin.js

@@ -39,6 +39,9 @@
       cssClass: 'icon-tabular-lad',
       initialize(domainObject) {
         domainObject.composition = [];
+        domainObject.configuration = {
+          objectStyles: {}
+        };
       }
     });
 

src/plugins/charts/bar/inspector/BarGraphInspectorViewProvider.js

@@ -41,7 +41,7 @@ export default function BarGraphInspectorViewProvider(openmct) {
           _destroy = destroy;
         },
         priority: function () {
-          return openmct.priority.HIGH + 1;
+          return openmct.editor.isEditing() ? openmct.priority.HIGH + 1 : openmct.priority.DEFAULT;
         },
         destroy: function () {
           if (_destroy) {

src/plugins/charts/scatter/inspector/ScatterPlotInspectorViewProvider.js

@@ -40,7 +40,7 @@ export default function ScatterPlotInspectorViewProvider(openmct) {
           _destroy = destroy;
         },
         priority: function () {
-          return openmct.priority.HIGH + 1;
+          return openmct.editor.isEditing() ? openmct.priority.HIGH + 1 : openmct.priority.DEFAULT;
         },
         destroy: function () {
           if (_destroy) {

src/plugins/conditionWidget/components/ConditionWidget.vue

@@ -79,6 +79,13 @@ export default {
   },
   mounted() {
     if (this.domainObject) {
+      if (!this.domainObject.configuration) {
+        // older versions didn't initialize configuration
+        this.domainObject.configuration = {
+          objectStyles: {}
+        };
+      }
+
       this.listenToConditionSetChanges();
     }
   },

src/plugins/conditionWidget/plugin.js

@@ -34,7 +34,9 @@ export default function plugin() {
       creatable: true,
       cssClass: 'icon-condition-widget',
       initialize(domainObject) {
-        domainObject.configuration = {};
+        domainObject.configuration = {
+          objectStyles: {}
+        };
         domainObject.label = 'Condition Widget';
         domainObject.conditionalLabel = '';
         domainObject.url = '';

src/plugins/displayLayout/AlphanumericFormatViewProvider.js

@@ -65,7 +65,9 @@ class AlphanumericFormatView {
   }
 
   priority() {
-    return 1;
+    return this.openmct.editor.isEditing()
+      ? this.openmct.priority.DEFAULT
+      : this.openmct.priority.LOW;
   }
 
   destroy() {

src/plugins/displayLayout/DisplayLayoutType.js

@@ -31,7 +31,8 @@ export default function DisplayLayoutType() {
       domainObject.composition = [];
       domainObject.configuration = {
         items: [],
-        layoutGrid: [10, 10]
+        layoutGrid: [10, 10],
+        objectStyles: {}
       };
     },
     form: [

src/plugins/displayLayout/components/AlphanumericFormat.vue

@@ -32,13 +32,16 @@
         </div>
         <div class="c-inspect-properties__value">
           <input
+            v-if="isEditing"
             id="telemetryPrintfFormat"
             type="text"
-            :disabled="!isEditing"
             :value="telemetryFormat"
             :placeholder="nonMixedFormat ? '' : 'Mixed'"
             @change="formatTelemetry"
           />
+          <template v-if="!isEditing && telemetryFormat?.length">
+            {{ telemetryFormat }}
+          </template>
         </div>
       </li>
     </ul>

src/plugins/displayLayout/components/DisplayLayout.vue

@@ -226,6 +226,11 @@ export default {
     this.composition.load();
     this.gridDimensions = [this.$el.offsetWidth, this.$el.scrollHeight];
 
+    // Initialize objectStyles if it doesn't exist for older versions
+    if (!this.domainObject.configuration.objectStyles) {
+      this.domainObject.configuration.objectStyles = {};
+    }
+
     this.unObserveItems = this.openmct.objects.observe(
       this.domainObject,
       'configuration.items',

src/plugins/filters/FiltersInspectorViewProvider.js

@@ -43,8 +43,6 @@ export default class FiltersInspectorViewProvider {
     let openmct = this.openmct;
     let _destroy = null;
 
-    const domainObject = selection?.[0]?.[0]?.context?.item;
-
     return {
       show: function (element) {
         const { destroy } = mount(
@@ -69,13 +67,6 @@ export default class FiltersInspectorViewProvider {
         if (isEditing) {
           return true;
         }
-
-        const metadata = openmct.telemetry.getMetadata(domainObject);
-        const metadataWithFilters = metadata
-          ? metadata.valueMetadatas.filter((value) => value.filters)
-          : [];
-
-        return metadataWithFilters.length;
       },
       priority: function () {
         return openmct.priority.DEFAULT;

src/plugins/filters/components/FiltersView.vue

@@ -38,6 +38,9 @@
       @update-filters="persistFilters"
     />
   </ul>
+  <span v-else>
+    This view doesn't include any parameters that have configured filter criteria.
+  </span>
 </template>
 
 <script>