From 769f3967ad6f101ead50460f492ad13def135b07 Mon Sep 17 00:00:00 2001
From: Johan Girod <johan.girod@beta.gouv.fr>
Date: Thu, 23 May 2019 11:30:33 +0200
Subject: [PATCH 1/3] =?UTF-8?q?:bug:=20R=C3=A9pare=20l'affichage=20des=20v?=
 =?UTF-8?q?aleurs=20dans=20les=20pages=20r=C3=A8gles?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 source/components/TargetSelection.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/source/components/TargetSelection.js b/source/components/TargetSelection.js
index fb34f2960..384f21b9e 100644
--- a/source/components/TargetSelection.js
+++ b/source/components/TargetSelection.js
@@ -28,7 +28,7 @@ export default compose(
 	withColours,
 	reduxForm({
 		form: 'conversation',
-		destroyOnUnmount: true
+		destroyOnUnmount: false
 	}),
 	connect(
 		state => ({

From 29b6536eb32a2b9cf9d8419e2563d83379c4c8c6 Mon Sep 17 00:00:00 2001
From: Johan Girod <johan.girod@beta.gouv.fr>
Date: Thu, 23 May 2019 11:41:57 +0200
Subject: [PATCH 2/3] =?UTF-8?q?:bug:=20R=C3=A9pare=20les=20liens=20dans=20?=
 =?UTF-8?q?les=20modales=20(overlay)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Les événements de clique à l'intérieur de la modale étaient stoppés et annulé (via preventDefault et stopPropagation). Cela, afin d'éviter de déclencher le onClose de la div parente.

Avec l'ajout du composant FocusTrap, on a plus besoin d'un tel mécanisme (qui a de grosse limites). J'ai donc supprimé les onClick sur les divs concernées

Fix #548
---
 source/components/LegalNotice.js |  6 ++++--
 source/components/Overlay.js     | 11 ++---------
 2 files changed, 6 insertions(+), 11 deletions(-)

diff --git a/source/components/LegalNotice.js b/source/components/LegalNotice.js
index d84ae003b..ec99b80c1 100644
--- a/source/components/LegalNotice.js
+++ b/source/components/LegalNotice.js
@@ -44,7 +44,9 @@ export const LegalNoticeContent = withTranslation()(() => (
 				<br />
 				San Francisco, CA 94107 <br />
 				Site web :&nbsp;
-				<a href="https://www.netlify.com">https://www.netlify.com</a>
+				<a href="https://www.netlify.com" target="_blank">
+					https://www.netlify.com
+				</a>
 			</Trans>
 		</p>
 		<h2>
@@ -52,7 +54,7 @@ export const LegalNoticeContent = withTranslation()(() => (
 		</h2>
 		<p>
 			<Trans i18nKey="legalNotice.contact.content">
-				<a href="mailto:contact@mon-entreprise.beta.gouv.fr">
+				<a target="_blank" href="mailto:contact@mon-entreprise.beta.gouv.fr">
 					contact@mon-entreprise.beta.gouv.fr
 				</a>
 			</Trans>
diff --git a/source/components/Overlay.js b/source/components/Overlay.js
index 2ec061ab3..a74782150 100644
--- a/source/components/Overlay.js
+++ b/source/components/Overlay.js
@@ -7,21 +7,14 @@ export default class Overlay extends Component {
 	render() {
 		const { onClose, children, ...otherProps } = this.props
 		return (
-			<div id="overlayWrapper" onClick={onClose}>
+			<div id="overlayWrapper">
 				<animate.fromBottom>
 					<FocusTrap
 						focusTrapOptions={{
 							onDeactivate: onClose,
 							clickOutsideDeactivates: true
 						}}>
-						<div
-							aria-modal="true"
-							id="overlayContent"
-							{...otherProps}
-							onClick={e => {
-								e.preventDefault()
-								e.stopPropagation()
-							}}>
+						<div aria-modal="true" id="overlayContent" {...otherProps}>
 							{children}
 							<LinkButton
 								aria-label="close"

From 5331415e6b16c0435d603181a26d27b21c459d49 Mon Sep 17 00:00:00 2001
From: Johan Girod <johan.girod@beta.gouv.fr>
Date: Thu, 23 May 2019 10:10:16 +0200
Subject: [PATCH 3/3] =?UTF-8?q?Am=C3=A9liore=20le=20bouton=20faire=20une?=
 =?UTF-8?q?=20simulation=20sur=20les=20pages=20r=C3=A8gles?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Lorsque la règle concerne contrat salarié, redirige vers le simulateur salarié
Lorsqu'elle concerne les auto-entrepreneurs ou les indépendant, vers les simulateur de revenu associés

Permet d'améliorer la rétention des utilisateurs arrivés sur les pages docs comme landing (ex : mot clé salaire net après impôts)
---
 source/components/rule/Rule.js | 13 +++++++++----
 1 file changed, 9 insertions(+), 4 deletions(-)

diff --git a/source/components/rule/Rule.js b/source/components/rule/Rule.js
index e31d6c248..1aad01428 100644
--- a/source/components/rule/Rule.js
+++ b/source/components/rule/Rule.js
@@ -129,12 +129,17 @@ export default compose(
 									{!valuesToShow && (
 										<div style={{ textAlign: 'center', marginTop: '1em' }}>
 											<Link
-												className="ui__ plain button"
+												className="ui__ cta plain button"
 												target="_parent"
 												to={
-													sitePaths.sécuritéSociale
-														? sitePaths.sécuritéSociale.index
-														: sitePaths.index
+													dottedName.includes('contrat salarié')
+														? sitePaths.sécuritéSociale.salarié
+														: dottedName.includes('auto entrepreneur')
+														? sitePaths.sécuritéSociale['auto-entrepreneur']
+														: dottedName.includes('indépendant')
+														? sitePaths.sécuritéSociale.indépendant
+														: // otherwise
+														  sitePaths.sécuritéSociale.index
 												}>
 												<T>Faire une simulation</T>
 											</Link>