Merge pull request #20317 from guerler/fix_display_window_manager

[25.0] Avoid displaying dataset tab view in window manager
This commit is contained in:
Dannon
2025-06-04 10:43:39 -04:00
committed by GitHub
2 changed files with 5 additions and 5 deletions
@@ -268,10 +268,10 @@ function onDisplay() {
// Only conditionally force to keep urls clean most of the time.
if (route.path === itemUrls.value.display) {
// @ts-ignore - monkeypatched router, drop with migration.
router.push(itemUrls.value.display, { title: props.name, force: true });
router.push(itemUrls.value.display, { force: true, preventWindowManager: true });
} else if (itemUrls.value.display) {
// @ts-ignore - monkeypatched router, drop with migration.
router.push(itemUrls.value.display, { title: props.name });
router.push(itemUrls.value.display, { preventWindowManager: true });
}
}
}
+3 -3
View File
@@ -7,13 +7,13 @@ import { addSearchParams } from "utils/url";
* refresh if needed.
*
* @param {String} Location as parsed to original router.push()
* @param {Object} Custom options, to provide a title and/or force reload
* @param {Object} Custom options, to provide a title, force reload, and/or prevent window manager
*/
export function patchRouterPush(VueRouter) {
const originalPush = VueRouter.prototype.push;
VueRouter.prototype.push = function push(location, options = {}) {
// add key to location to force component refresh
const { title, force } = options;
const { title, force, preventWindowManager } = options;
if (force) {
location = addSearchParams(location, { __vkey__: Date.now() });
}
@@ -27,7 +27,7 @@ export function patchRouterPush(VueRouter) {
}
// show location in window manager
const Galaxy = getGalaxyInstance();
if (title && Galaxy.frame && Galaxy.frame.active) {
if (title && !preventWindowManager && Galaxy.frame && Galaxy.frame.active) {
Galaxy.frame.add({ title: title, url: location });
return;
}