diff --git a/platform/viewer/src/connectedComponents/ToolbarRow.js b/platform/viewer/src/connectedComponents/ToolbarRow.js index b012e6eb0..2dca5835f 100644 --- a/platform/viewer/src/connectedComponents/ToolbarRow.js +++ b/platform/viewer/src/connectedComponents/ToolbarRow.js @@ -71,9 +71,9 @@ class ToolbarRow extends Component { const hasActiveContext = this.props.activeContexts.some(actx => contexts.includes(actx) ); - // TODO: what should `isDisabled` receive? - // studies? activeViewport? + // It's a bit beefy to pass studies; probably only need to be reactive on `studyInstanceUIDs` and activeViewport? + // Note: This does not cleanly handle `studies` prop updating with panel open const isDisabled = menuOption.isDisabled(this.props.studies); if (hasActiveContext && !isDisabled) {