Skip to content

Commit

Permalink
Merge pull request galaxyproject#17111 from dannon/backport-forcerelo…
Browse files Browse the repository at this point in the history
…ad-display

[23.1] Allow re-click of display with force reloading if already on the route.
  • Loading branch information
martenson authored Nov 30, 2023
2 parents cb1bb4a + 43bcc91 commit 0e3a0ca
Showing 1 changed file with 8 additions and 1 deletion.
9 changes: 8 additions & 1 deletion client/src/components/History/Content/ContentItem.vue
Original file line number Diff line number Diff line change
Expand Up @@ -231,7 +231,14 @@ export default {
const url = entryPointsForHda[0].target;
window.open(url, "_blank");
} else {
this.$router.push(this.itemUrls.display, { title: this.name });
// vue-router 4 supports a native force push with clean URLs,
// but we're using a workaround with a __vkey__ bit as a workaround
// Only conditionally force to keep urls clean most of the time.
if (this.$router.currentRoute.path === this.itemUrls.display) {
this.$router.push(this.itemUrls.display, { title: this.name, force: true });
} else {
this.$router.push(this.itemUrls.display, { title: this.name });
}
}
},
onDelete(recursive = false) {
Expand Down

0 comments on commit 0e3a0ca

Please sign in to comment.