feat(app): add indeterminate tooltip to test result row

hpi-dhc · Jan 23, 2025 · e60efb4 · e60efb4
1 parent 0e8d30c
commit e60efb4
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 7 deletions.
diff --git a/app/lib/common/widgets/annotation_table.dart b/app/lib/common/widgets/annotation_table.dart
@@ -108,16 +108,20 @@ bool testResultIsUnknown(BuildContext context, String phenotype) =>
 TableRowDefinition testResultTableRow(
   BuildContext context,
   {
+    required GenotypeResult genotypeResult, 
     required String key,
     required String value,
     String? keyTooltip,
-    String? valueTooltip,
   }
 ) => TableRowDefinition(
   key,
   value,
   keyTooltip: keyTooltip,
-  valueTooltip: valueTooltip,
+  valueTooltip: value == indeterminateResult
+    ? context.l10n.indeterminate_result_tooltip(
+        genotypeResult.geneDisplayString,
+      )
+    : null,
   italicValue: testResultIsUnknown(context, value),
 );
 
@@ -133,13 +137,9 @@ TableRowDefinition phenotypeTableRow(
   final value = possiblyAdaptedPhenotype(context, genotypeResult, drug: drug);
   return testResultTableRow(
     context,
+    genotypeResult: genotypeResult,
     key: key,
     value: value,
     keyTooltip: keyTooltip,
-    valueTooltip: value == indeterminateResult
-      ? context.l10n.indeterminate_result_tooltip(
-          genotypeResult.geneDisplayString,
-        )
-      : null,
   );
 }
diff --git a/app/lib/report/pages/gene.dart b/app/lib/report/pages/gene.dart
@@ -98,6 +98,7 @@ class GenePage extends HookWidget {
   Widget _buildGeneResults(BuildContext context) => buildTable([
     testResultTableRow(
       context,
+      genotypeResult: genotypeResult,
       key: context.l10n.gene_page_genotype,
       value: genotypeResult.variantDisplayString(context),
       keyTooltip: context.l10n.gene_page_genotype_tooltip,