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] =?UTF-8?q?:bug:=20R=C3=A9pare=20les=20liens=20dans=20les?=
 =?UTF-8?q?=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"