From d1225dce60388a63c85cce014401d82a16112957 Mon Sep 17 00:00:00 2001
From: Friedjoff Trautwein <friedjoff.trautwein@geops.de>
Date: Tue, 28 Apr 2020 15:17:16 +0200
Subject: [PATCH] feat: add help buttons to location components

---
 src/components/HelpModal.js              |  6 ++-
 src/components/LocationForm.js           |  9 ++++-
 src/components/LocationForm.module.css   |  6 +++
 src/components/LocationResult.js         | 51 ++++++++++++++++--------
 src/components/LocationResult.module.css | 10 +++++
 src/i18n/resources/de/translation.json   |  6 ++-
 6 files changed, 67 insertions(+), 21 deletions(-)

diff --git a/src/components/HelpModal.js b/src/components/HelpModal.js
index 42c2c418..559e204c 100644
--- a/src/components/HelpModal.js
+++ b/src/components/HelpModal.js
@@ -6,7 +6,7 @@ import { ReactComponent as InfoIcon } from '../icons/info.svg';
 
 import styles from './HelpModal.module.css';
 
-function HelpModal({ children, header }) {
+function HelpModal({ children, color, header }) {
   return (
     <Modal
       actions={[{ key: 'done', content: 'Ok' }]}
@@ -14,17 +14,19 @@ function HelpModal({ children, header }) {
         <Modal.Content className={styles.text}>{children}</Modal.Content>
       }
       header={header || undefined}
-      trigger={<InfoIcon fill="white" className={styles.icon} />}
+      trigger={<InfoIcon fill={color} className={styles.icon} />}
     />
   );
 }
 
 HelpModal.propTypes = {
   children: PropTypes.node.isRequired,
+  color: PropTypes.string,
   header: PropTypes.string,
 };
 
 HelpModal.defaultProps = {
+  color: 'white',
   header: '',
 };
 
diff --git a/src/components/LocationForm.js b/src/components/LocationForm.js
index 81a6e091..ba212763 100644
--- a/src/components/LocationForm.js
+++ b/src/components/LocationForm.js
@@ -2,13 +2,14 @@ import intersection from 'lodash.intersection';
 import React from 'react';
 import { useTranslation } from 'react-i18next';
 import { useDispatch, useSelector } from 'react-redux';
-import { Accordion, Form, Label, Segment } from 'semantic-ui-react';
+import { Accordion, Form, Header, Label, Segment } from 'semantic-ui-react';
 // eslint-disable-next-line import/no-unresolved
 import { info } from 'lib/src';
 
 import Button from './Button';
 import Checkbox from './Checkbox';
 import Dropdown from './Dropdown';
+import HelpModal from './HelpModal';
 import Input from './Input';
 import styles from './LocationForm.module.css';
 import { setFormLocation } from '../store/actions';
@@ -422,6 +423,12 @@ function LocationForm() {
           value={location.silverFirArea}
         />
       )}
+      <div className={styles.help}>
+        <HelpModal color="#006268" header={t('location.header')}>
+          {t('location.help')}
+        </HelpModal>
+      </div>
+      <Header>{t('location.header')}</Header>
       <Accordion fluid panels={panels} styled />
       {intersection(Object.keys(formLocation), filterFields).length > 0 && (
         <Button
diff --git a/src/components/LocationForm.module.css b/src/components/LocationForm.module.css
index 573898d4..5dd91a78 100644
--- a/src/components/LocationForm.module.css
+++ b/src/components/LocationForm.module.css
@@ -2,6 +2,12 @@
   margin: 0 20px;
 }
 
+.help {
+  position: absolute;
+  right: 10px;
+  margin-top: 15px;
+}
+
 .resetButton:global(.ui.button) {
   margin-top: 1em;
 }
diff --git a/src/components/LocationResult.js b/src/components/LocationResult.js
index fc7b92f0..05a8eacb 100644
--- a/src/components/LocationResult.js
+++ b/src/components/LocationResult.js
@@ -9,6 +9,7 @@ import { info } from 'lib/src';
 import Dropdown from './Dropdown';
 import Ecogram from './Ecogram';
 import ForestTypeButton from './ForestTypeButton';
+import HelpModal from './HelpModal';
 import { setFormLocation } from '../store/actions';
 import styles from './LocationResult.module.css';
 
@@ -37,28 +38,44 @@ function LocationResult() {
 
   return forestTypes ? (
     <Form className={styles.form}>
-      {hasMainGroup && <Header>{t('forestType.group.main')}</Header>}
+      {hasMainGroup && (
+        <>
+          <Header>{t('forestType.group.main')}</Header>
+          <div className={styles.mainHelp}>
+            <HelpModal color="#006268" header={t('forestType.group.main')}>
+              {t('location.mainResultHelp')}
+            </HelpModal>
+          </div>
+        </>
+      )}
       {hasMainGroup && ecogram && (
         <Ecogram data={ecogram} selectForestType={selectForestType} />
       )}
       {hasMainGroup && !ecogram && <Message>{t('location.noEcogram')}</Message>}
       {hasOtherGroup && (
-        <Dropdown
-          search
-          label={t('forestType.group.other')}
-          options={forestTypes.other.map((key) => ({
-            key,
-            content: (
-              <>
-                <ForestTypeButton code={key} compact />
-                {key} - {info('forestType', key)[i18n.language]}
-              </>
-            ),
-            text: `${key} - ${info('forestType', key)[i18n.language]}`,
-            value: key,
-          }))}
-          value={formLocation.forestType}
-        />
+        <>
+          <div className={styles.otherHelp}>
+            <HelpModal color="#006268" header={t('forestType.group.other')}>
+              {t('location.otherResultHelp')}
+            </HelpModal>
+          </div>
+          <Dropdown
+            search
+            label={t('forestType.group.other')}
+            options={forestTypes.other.map((key) => ({
+              key,
+              content: (
+                <>
+                  <ForestTypeButton code={key} compact />
+                  {key} - {info('forestType', key)[i18n.language]}
+                </>
+              ),
+              text: `${key} - ${info('forestType', key)[i18n.language]}`,
+              value: key,
+            }))}
+            value={formLocation.forestType}
+          />
+        </>
       )}
     </Form>
   ) : null;
diff --git a/src/components/LocationResult.module.css b/src/components/LocationResult.module.css
index 76c45da0..9046b976 100644
--- a/src/components/LocationResult.module.css
+++ b/src/components/LocationResult.module.css
@@ -3,3 +3,13 @@
   padding: 1em;
   box-shadow: 0 -20px 20px -20px #777;
 }
+
+.mainHelp {
+  position: absolute;
+  right: 15px;
+  margin-top: -35px;
+}
+.otherHelp {
+  position: absolute;
+  right: 15px;
+}
diff --git a/src/i18n/resources/de/translation.json b/src/i18n/resources/de/translation.json
index 27fc528a..4d767e92 100644
--- a/src/i18n/resources/de/translation.json
+++ b/src/i18n/resources/de/translation.json
@@ -38,7 +38,11 @@
     "impressum": "Die Tree App ist ein Umsetzungsprodukt des Forschungsprogramms „Wald und Klimawandel“ von BAFU und WSL."
   },
   "location": {
-    "reset": "Filter zurücksetzen",
+    "header": "Filterkriterien",
+    "help": "TODO",
+    "mainResultHelp": "TODO",
+    "otherResultHelp": "TODO",
+    "reset": "Filterkriterien zurücksetzen",
     "noEcogram": "Kein Ökogram gefunden."
   },
   "map": {