nav should react to plot_prep

stjude · Jan 15, 2025 · c820370 · c820370
1 parent 8b3f218
commit c820370
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/client/mass/nav.js b/client/mass/nav.js
@@ -128,8 +128,12 @@ class TdbNav {
 		if (action.type.startsWith('filter')) return true
 		if (action.type.startsWith('cohort')) return true
 		if (action.type.startsWith('tab')) return true
+
+		// do not use startsWith('plot_') to exclude 'plot_edit' as no need for nav to react to that
+		if (action.type == 'plot_prep') return true
 		if (action.type == 'plot_create') return true
 		if (action.type == 'plot_delete') return true
+
 		if (action.type == 'app_refresh') return true
 		if (action.type.endsWith('_customTerm')) return true
 		if (action.type.endsWith('_group')) return true

diff --git a/client/mass/store.js b/client/mass/store.js
@@ -249,6 +249,8 @@ TdbStore.prototype.actions = {
 		})
 	},
 
+	// dispatch "plot_prep" action to produce a 'initiating' UI of this plot, for user to fill in additional details to launch the plot
+	// example: table, scatterplot which requires user to select two terms
 	async plot_prep(action) {
 		const plot = {
 			id: 'id' in action ? action.id : getId()