fix(#7524): Open in New Tab action from a sub-object in a layout

src/plugins/inspectorDataVisualization/TelemetryFrame.vue

@@ -120,7 +120,9 @@ export default {
         'tc.mode': 'fixed'
       };
       const newTabAction = this.openmct.actions.getAction('newTab');
-      newTabAction.invoke([sourceTelemObject], urlParams);
+      // No view context needed, so pass undefined.
+      // The urlParams arg will override the global time bounds with the dataviz plot bounds.
+      newTabAction.invoke([sourceTelemObject], undefined, urlParams);
       this.showMenu = false;
     },
     previewTelemetry() {

src/plugins/openInNewTabAction/openInNewTabAction.js

@@ -31,8 +31,26 @@ export default class OpenInNewTab {
 
     this._openmct = openmct;
   }
-  invoke(objectPath, urlParams = undefined) {
-    let url = objectPathToUrl(this._openmct, objectPath, urlParams);
+
+  /**
+   * Invokes the "Open in New Tab" action. This will open the object in a new
+   * browser tab. The URL for the new tab is determined by the current object
+   * path and any custom time bounds.
+   *
+   * @param {import('@/api/objects/ObjectAPI').DomainObject[]} objectPath The current object path
+   * @param {ViewContext} _view The view context for the object being opened (unused)
+   * @param {Object<string, string | number>} customUrlParams Provides the ability to override
+   * the global time conductor bounds. It is an object with the following key/value pairs:
+   * ```
+   * {
+   *  'tc.start': <number>,
+   *  'tc.end': <number>,
+   *  'tc.mode': 'fixed' | 'local' | <string>
+   * }
+   * ```
+   */
+  invoke(objectPath, _view, customUrlParams) {
+    const url = objectPathToUrl(this._openmct, objectPath, customUrlParams);
     window.open(url);
   }
 }

src/tools/url.js

@@ -29,9 +29,15 @@
  * @param {import('../../openmct').OpenMCT} openmct
  * @returns {Array<string>} newTabParams
  */
-export function paramsToArray(openmct) {
+export function paramsToArray(openmct, customUrlParams = {}) {
   let urlParams = openmct.router.getParams();
 
+  // Merge the custom URL parameters with the current URL parameters.
+  Object.entries(customUrlParams).forEach((param) => {
+    const [key, value] = param;
+    urlParams[key] = value;
+  });
+
   if (urlParams['tc.mode'] === 'fixed') {
     delete urlParams['tc.startDelta'];
     delete urlParams['tc.endDelta'];
@@ -43,21 +49,6 @@ export function paramsToArray(openmct) {
   return Object.entries(urlParams).map(([key, value]) => `${key}=${value}`);
 }
 
-// export function paramsToArray(openmct) {
-//   const urlParams = getUrlParams(openmct);
-//   // Filter out any parameters whose values are objects, then map the rest to the desired string format.
-//   const newTabParams = Object.entries(urlParams).reduce((acc, [key, value]) => {
-//     // Only process the parameter if its value is not an object (or is null, which is fine to include).
-//     if (typeof value !== 'object' || value === null) {
-//       const param = `${key}=${encodeURIComponent(value)}`;
-//       acc.push(param);
-//     }
-//     return acc;
-//   }, []);
-
-//   return newTabParams;
-// }
-
 export function identifierToString(openmct, objectPath) {
   return '#/browse/' + openmct.objects.getRelativePath(objectPath);
 }