feat(#706): move consequence to tooltip

hpi-dhc · Sep 4, 2024 · b71e503 · b71e503
1 parent ed15bb7
commit b71e503
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 14 deletions.
diff --git a/app/lib/common/models/drug/drug_inhibitors.dart b/app/lib/common/models/drug/drug_inhibitors.dart
@@ -117,28 +117,24 @@ String inhibitionTooltipText(
   final activeInhibitorsWithBrandNames =
     activeInhibitors.map(getDrugNames).toList();
   final inhibitorsString = enumerationWithAnd(
-      activeInhibitorsWithBrandNames,
-      context,
-    );
-  return context.l10n.inhibitors_tooltip(inhibitorsString);
+    activeInhibitorsWithBrandNames,
+    context,
+  );
+  final consequence = activeInhibitors.all(isModerateInhibitor)
+    ? context.l10n.inhibitors_consequence_not_adapted
+    : context.l10n.inhibitors_consequence_adapted(genotypeResult.phenotype);
+  return '$consequence\n\n${context.l10n.inhibitors_tooltip(inhibitorsString)}';
 }
 
 Table buildDrugInteractionInfo(
   BuildContext context,
   GenotypeResult genotypeResult,
   { String? drug }
 ) {
-  final activeInhibitors = activeInhibitorsFor(
-    genotypeResult.gene,
-    drug: drug,
-  );
-  final consequence = activeInhibitors.all(isModerateInhibitor)
-    ? context.l10n.inhibitors_consequence_not_adapted
-    : context.l10n.inhibitors_consequence_adapted(genotypeResult.phenotype);
   return buildTable([
     TableRowDefinition(
       drugInteractionIndicator,
-      '${context.l10n.inhibitor_message} ($consequence)',
+      context.l10n.inhibitor_message,
       tooltip: inhibitionTooltipText(context, genotypeResult, drug: drug),
     )],
     boldHeader: false,

diff --git a/app/lib/l10n/app_en.arb b/app/lib/l10n/app_en.arb
@@ -104,8 +104,8 @@
   },
 
   "inhibitor_message": "One or more of the medications you are currently taking may interact with your genetic result",
-  "inhibitors_consequence_adapted": "phenotype adapted from {originalPhenotype}",
-  "inhibitors_consequence_not_adapted": "phenotype not adapted",
+  "inhibitors_consequence_adapted": "Your phenotype was adapted from {originalPhenotype}.",
+  "inhibitors_consequence_not_adapted": "Your phenotype was not adapted.",
   "inhibitors_tooltip": "Current interacting medications: {inhibitors}",
 
   "drugs_page_inhibitor_direct_salutation": "you are",