Merge pull request #647 from neuroscout/fix_review_tooltip

Have report tooltip follow same display rules as rest of anlysis buil…
neuroscout · Aug 28, 2019 · fa08c0f · fa08c0f
2 parents 3e5c33e + 174c47f
commit fa08c0f
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/neuroscout/frontend/src/analysis_builder/Builder.tsx b/neuroscout/frontend/src/analysis_builder/Builder.tsx
@@ -1201,6 +1201,7 @@ export default class AnalysisBuilder extends React.Component<BuilderProps & Rout
                         analysisId={analysis.analysisId}
                         runIds={analysis.runIds}
                         postReports={this.state.postReports}
+                        defaultVisible={this.state.doTooltip && this.state.activeTab === ('review' as TabName)}
                       />
                       <Review
                         model={this.state.model}

diff --git a/neuroscout/frontend/src/analysis_builder/Report.tsx b/neuroscout/frontend/src/analysis_builder/Report.tsx
@@ -78,6 +78,7 @@ interface ReportProps {
   analysisId?: string;
   runIds: string[];
   postReports: boolean;
+  defaultVisible: boolean;
 }
 
 interface ReportState {
@@ -208,7 +209,7 @@ export class Report extends React.Component<ReportProps, ReportState> {
            <Tooltip
             title={'Here you can preview the final design and correlation matrices. \
             \nClick on the design matrix columns to view the timecourse in detail.'}
-            defaultVisible={true}
+            defaultVisible={this.props.defaultVisible}
            >
              <Icon type="info-circle" style={{ fontSize: '15px'}}/>
            </Tooltip>