From 6957ea6fe222a5193ce27a797488055f16a9fa88 Mon Sep 17 00:00:00 2001
From: Mael <mael.thomas@data.gouv.fr>
Date: Mon, 21 Jan 2019 18:33:23 +0100
Subject: [PATCH] :sparkles: Nettoyage de deux erreurs JS anodines

---
 source/components/utils/Scroll.js        |  3 +-
 source/sites/mycompanyinfrance.fr/App.js | 59 +++++++++++++-----------
 2 files changed, 34 insertions(+), 28 deletions(-)

diff --git a/source/components/utils/Scroll.js b/source/components/utils/Scroll.js
index 07dcd7d6b..2eb86fb8a 100644
--- a/source/components/utils/Scroll.js
+++ b/source/components/utils/Scroll.js
@@ -1,4 +1,5 @@
 import React, { Component } from 'react'
+import { dissoc } from 'ramda'
 
 const forEachParent = (node, fn) => {
 	if (!node) {
@@ -73,7 +74,7 @@ export class ScrollToElement extends Component {
 	render() {
 		return (
 			<div
-				{...this.props}
+				{...dissoc('onlyIfNotVisible', this.props)}
 				style={{
 					...this.props.style,
 					...(!this.props.children ? { position: 'absolute' } : {})
diff --git a/source/sites/mycompanyinfrance.fr/App.js b/source/sites/mycompanyinfrance.fr/App.js
index 494ab99f1..fca85bb5f 100644
--- a/source/sites/mycompanyinfrance.fr/App.js
+++ b/source/sites/mycompanyinfrance.fr/App.js
@@ -79,35 +79,40 @@ let RouterSwitch = compose(
 )(({ sitePaths }) => {
 	return (
 		<Switch>
-			<Route exact path="/" component={Landing} />
-			<div className="app-container">
-				{/* Passing location down to prevent update blocking */}
-				<Navigation location={location} />
-				<div className="app-content">
-					<ProgressHeader />
-					<div
-						className="ui__ container"
-						style={{ flexGrow: 1, flexShrink: 0 }}>
-						<Route path={sitePaths.entreprise.index} component={CompanyIndex} />
-						<Route
-							path={sitePaths.sécuritéSociale.index}
-							component={SocialSecurity}
-						/>
-						<Route
-							path={sitePaths.démarcheEmbauche.index}
-							component={HiringProcess}
-						/>
-						<Route
-							path={sitePaths.documentation.index + '/:name+'}
-							component={RulePage}
-						/>
-						{process.env.NODE_ENV !== 'production' && (
-							<Route exact path="/sitemap" component={Sitemap} />
-						)}
+			<>
+				<Route exact path="/" component={Landing} />
+				<div className="app-container">
+					{/* Passing location down to prevent update blocking */}
+					<Navigation location={location} />
+					<div className="app-content">
+						<ProgressHeader />
+						<div
+							className="ui__ container"
+							style={{ flexGrow: 1, flexShrink: 0 }}>
+							<Route
+								path={sitePaths.entreprise.index}
+								component={CompanyIndex}
+							/>
+							<Route
+								path={sitePaths.sécuritéSociale.index}
+								component={SocialSecurity}
+							/>
+							<Route
+								path={sitePaths.démarcheEmbauche.index}
+								component={HiringProcess}
+							/>
+							<Route
+								path={sitePaths.documentation.index + '/:name+'}
+								component={RulePage}
+							/>
+							{process.env.NODE_ENV !== 'production' && (
+								<Route exact path="/sitemap" component={Sitemap} />
+							)}
+						</div>
+						<Footer />
 					</div>
-					<Footer />
 				</div>
-			</div>
+			</>
 		</Switch>
 	)
 })