From 02170b958e226e5218473f2d863eab916d27c97f Mon Sep 17 00:00:00 2001
From: tamslo <tamaraslosarek@gmail.com>
Date: Fri, 21 Feb 2025 10:09:29 +0100
Subject: [PATCH] feat(app): clarify that guideline is based on genes

---
 .../widgets/annotation_cards/guideline.dart   | 22 ++++++++++++++-----
 app/lib/l10n/app_en.arb                       |  8 ++++---
 2 files changed, 22 insertions(+), 8 deletions(-)

diff --git a/app/lib/drug/widgets/annotation_cards/guideline.dart b/app/lib/drug/widgets/annotation_cards/guideline.dart
index 6d16c68a..6e974576 100644
--- a/app/lib/drug/widgets/annotation_cards/guideline.dart
+++ b/app/lib/drug/widgets/annotation_cards/guideline.dart
@@ -44,6 +44,7 @@ class GuidelineAnnotationCard extends StatelessWidget {
   Widget _buildResultSection(BuildContext context) {
     final implicationText = drug.userGuideline?.annotations.implication;
     final recommendationText = drug.userGuideline?.annotations.recommendation;
+    final descriptionStyle = TextStyle(fontStyle: FontStyle.italic);
     return Column(
       crossAxisAlignment: CrossAxisAlignment.start,
       children: [
@@ -80,10 +81,15 @@ class GuidelineAnnotationCard extends StatelessWidget {
               Text.rich(
                 TextSpan(
                   children: [
-                    if (implicationText != null) TextSpan(
+                    if (
+                      implicationText != null &&
+                      drug.warningLevel != WarningLevel.none
+                    ) TextSpan(
                       text: context.l10n.drugs_page_implication_description,
+                      style: descriptionStyle,
                     ),
-                    WidgetSpan(child: SizedBox(height: PharMeTheme.mediumSpace * 1.2)),
+                    TextSpan(text: ':\n', style: descriptionStyle),
+                    WidgetSpan(child: SizedBox(height: PharMeTheme.mediumSpace * 1.3)),
                     TextSpan(
                       text:
                         implicationText ?? context.l10n.drugs_page_no_guidelines_text,
@@ -95,13 +101,19 @@ class GuidelineAnnotationCard extends StatelessWidget {
                 ),
               ),
               if (recommendationText != null) ...[
-                SizedBox(height: PharMeTheme.smallToMediumSpace),
+                SizedBox(height: PharMeTheme.mediumSpace),
                 Text.rich(
                   TextSpan(children: [
                     TextSpan(
-                      text: context.l10n.drugs_page_recommendation_description,
+                      text: context.l10n.drugs_page_recommendation_description_part_1,
+                      style: descriptionStyle,
                     ),
-                    WidgetSpan(child: SizedBox(height: PharMeTheme.mediumSpace * 1.2)),
+                    TextSpan(
+                      text: context.l10n.drugs_page_recommendation_description_part_2,
+                      style: descriptionStyle.copyWith(fontWeight: FontWeight.bold),
+                    ),
+                    TextSpan(text: ':\n', style: descriptionStyle),
+                    WidgetSpan(child: SizedBox(height: PharMeTheme.mediumSpace * 1.3)),
                     TextSpan(
                       text: recommendationText,
                       style: TextStyle(fontWeight: FontWeight.bold),
diff --git a/app/lib/l10n/app_en.arb b/app/lib/l10n/app_en.arb
index 901f49d2..28141537 100644
--- a/app/lib/l10n/app_en.arb
+++ b/app/lib/l10n/app_en.arb
@@ -291,10 +291,12 @@
   },
   "drugs_page_tooltip_guideline_missing": "Guidelines provide recommendations on which medications to use based on your genes. However, no guideline is present in this case (yet).",
   "@drugs_page_tooltip_guideline_missing": {},
-  "drugs_page_implication_description": "What your result means: ",
+  "drugs_page_implication_description": "What your gene result means",
   "@drugs_page_implication_description": {},
-  "drugs_page_recommendation_description": "What to do: ",
-  "@drugs_page_recommendation_description": {},
+  "drugs_page_recommendation_description_part_1": "What to do",
+  "@drugs_page_recommendation_description_part_1": {},
+  "drugs_page_recommendation_description_part_2": " based on your genes",
+  "@drugs_page_recommendation_description_part_2": {},
 
   "report_content_explanation": "This is your PGx (pharmacogenomic) test report. Tap on a gene name for more details on your results and a list of associated medications.",
   "@report_content_explanation": {},