🐛 Répare les liens dans les modales (overlay)

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
pull/549/head
Johan Girod 2019-05-23 11:41:57 +02:00
parent 769f3967ad
commit 29b6536eb3
No known key found for this signature in database
GPG Key ID: 9E27B57DA2E8AE12
2 changed files with 6 additions and 11 deletions

View File

@ -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>

View File

@ -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"