From 9dc7593fd2f8162662d2b978837fd07ce6adf30c Mon Sep 17 00:00:00 2001
From: Mael <mael.thomas@data.gouv.fr>
Date: Fri, 19 Apr 2019 15:14:18 +0200
Subject: [PATCH 1/5] :art: Redesign de la zone de retours utilisateur

---
 source/components/Feedback/Feedback.css       | 16 +++++++++++--
 source/components/Feedback/PageFeedback.js    | 24 ++++++++++---------
 .../layout/Footer/Footer.js                   |  1 +
 3 files changed, 28 insertions(+), 13 deletions(-)

diff --git a/source/components/Feedback/Feedback.css b/source/components/Feedback/Feedback.css
index eb9bf2fad..f2726d4fd 100644
--- a/source/components/Feedback/Feedback.css
+++ b/source/components/Feedback/Feedback.css
@@ -1,8 +1,20 @@
 .feedback-page {
 	align-items: flex-start;
 	display: flex;
-	align-items: flex-end;
-	justify-content: space-between;
+	justify-content: center;
 	padding-top: 0.6rem;
 	padding-bottom: 0.6rem;
+	background: var(--lighterColour);
+	border-radius: 0.9rem;
+	max-width: 36em;
+	padding: 0.6em;
+	margin: 1em 0 1em auto;
+}
+.feedback-page.stickToFooter {
+	margin-bottom: 0;
+	border-bottom-left-radius: 0;
+	border-bottom-right-radius: 0;
+}
+.feedback-page button.link-button {
+	margin: 0 0.6em;
 }
diff --git a/source/components/Feedback/PageFeedback.js b/source/components/Feedback/PageFeedback.js
index be32ec981..fa906b422 100644
--- a/source/components/Feedback/PageFeedback.js
+++ b/source/components/Feedback/PageFeedback.js
@@ -11,6 +11,7 @@ import Form from './FeedbackForm'
 import type { Tracker } from 'Components/utils/withTracker'
 import type { Location } from 'react-router-dom'
 import type { Node } from 'react'
+import classNames from 'classnames'
 
 type OwnProps = {
 	blacklist: Array<string>,
@@ -89,6 +90,7 @@ class PageFeedback extends Component<Props, State> {
 		this.setState({ showForm: true })
 	}
 	render() {
+		let { stickToFooter = false } = this.props
 		if (this.feedbackAlreadyGiven) {
 			return null
 		}
@@ -96,7 +98,10 @@ class PageFeedback extends Component<Props, State> {
 			this.props.location.pathname === '/' ? '' : this.props.location.pathname
 		return (
 			!this.props.blacklist.includes(pathname) && (
-				<div className="feedback-page ui__ container notice">
+				<div
+					className={classNames('feedback-page', 'ui__', 'notice', {
+						stickToFooter
+					})}>
 					{!this.state.showForm && !this.state.showThanks && (
 						<>
 							<div>
@@ -107,27 +112,24 @@ class PageFeedback extends Component<Props, State> {
 								)}{' '}
 								<div style={{ display: 'inline-block' }}>
 									<button
-										style={{ marginLeft: '0.4rem' }}
 										className="ui__ link-button"
 										onClick={() => this.handleFeedback({ useful: true })}>
 										<Trans>Oui</Trans>
 									</button>{' '}
 									<button
-										style={{ marginLeft: '0.4rem' }}
 										className="ui__ link-button"
 										onClick={() => this.handleFeedback({ useful: false })}>
 										<Trans>Non</Trans>
 									</button>
+									<button
+										className="ui__ link-button"
+										onClick={this.handleErrorReporting}>
+										<Trans i18nKey="feedback.reportError">
+											Faire une suggestion
+										</Trans>
+									</button>
 								</div>
 							</div>
-							<button
-								style={{ textAlign: 'right' }}
-								className="ui__ link-button"
-								onClick={this.handleErrorReporting}>
-								<Trans i18nKey="feedback.reportError">
-									Faire une suggestion
-								</Trans>
-							</button>{' '}
 						</>
 					)}
 					{this.state.showThanks && (
diff --git a/source/sites/mycompanyinfrance.fr/layout/Footer/Footer.js b/source/sites/mycompanyinfrance.fr/layout/Footer/Footer.js
index c1ba89b85..f6fd7548b 100644
--- a/source/sites/mycompanyinfrance.fr/layout/Footer/Footer.js
+++ b/source/sites/mycompanyinfrance.fr/layout/Footer/Footer.js
@@ -60,6 +60,7 @@ const Footer = ({ colours: { colour }, tracker, t, sitePaths }) => {
 				))}
 			</Helmet>
 			<PageFeedback
+				stickToFooter={true}
 				blacklist={feedbackBlacklist.map(lens => view(lens, sitePaths))}
 			/>
 			<footer className="footer" style={{ backgroundColor: `${colour}22` }}>

From 6655217208137b76bde6e105f301f98730eaad86 Mon Sep 17 00:00:00 2001
From: Mael <mael.thomas@data.gouv.fr>
Date: Fri, 19 Apr 2019 15:23:40 +0200
Subject: [PATCH 2/5] =?UTF-8?q?D=C3=A9placement=20du=20retour=20utilisateu?=
 =?UTF-8?q?r=20des=20simulateurs?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Toujours présent pour avoir moins de choses qui apparaîssent après une
saisie. L'utilisateur sait alors qu'il peut faire un retour quand il en
aura envie par la suite.
---
 source/components/SalaryExplanation.js               | 12 ++++++++++--
 source/components/Simulation.js                      |  4 ++--
 .../pages/SocialSecurity/Salarié.js                  |  8 --------
 3 files changed, 12 insertions(+), 12 deletions(-)

diff --git a/source/components/SalaryExplanation.js b/source/components/SalaryExplanation.js
index 7380761c6..58a6acad6 100644
--- a/source/components/SalaryExplanation.js
+++ b/source/components/SalaryExplanation.js
@@ -1,7 +1,7 @@
 import { startConversation } from 'Actions/actions'
 import withTracker from 'Components/utils/withTracker'
 import { compose } from 'ramda'
-import React, { Component } from 'react'
+import { React, T } from 'Components'
 import { connect } from 'react-redux'
 import { formValueSelector } from 'redux-form'
 import ficheDePaieSelectors from 'Selectors/ficheDePaieSelectors'
@@ -24,10 +24,18 @@ export default compose(
 		}
 	)
 )(
-	class SalaryExplanation extends Component {
+	class SalaryExplanation extends React.Component {
 		render() {
 			return (
 				<Animate.fromBottom delay={2000}>
+					<p>
+						<T k="simulateurs.salarié.description">
+							Dès que le salarié est déclaré et payé, il est couvert par le
+							régime général de la Sécurité sociale (santé, maternité,
+							invalidité, vieillesse, maladie professionnelle et accidents) et
+							chômage.
+						</T>
+					</p>
 					{!this.props.conversationStarted ? (
 						<SalaryFirstExplanation {...this.props} />
 					) : (
diff --git a/source/components/Simulation.js b/source/components/Simulation.js
index 1d7a78846..e874fc8d8 100644
--- a/source/components/Simulation.js
+++ b/source/components/Simulation.js
@@ -101,8 +101,7 @@ export default compose(
 					)}
 
 					{showTargets && this.props.targets}
-					{!noUserInput && this.props.explanation}
-					{!noUserInput && !noFeedback && (
+					{!noFeedback && (
 						<div style={{ margin: '-0.6rem' }}>
 							<PageFeedback
 								customMessage={
@@ -114,6 +113,7 @@ export default compose(
 							/>
 						</div>
 					)}
+					{!noUserInput && this.props.explanation}
 				</>
 			)
 		}
diff --git a/source/sites/mycompanyinfrance.fr/pages/SocialSecurity/Salarié.js b/source/sites/mycompanyinfrance.fr/pages/SocialSecurity/Salarié.js
index 1b2d10f31..0f1685063 100644
--- a/source/sites/mycompanyinfrance.fr/pages/SocialSecurity/Salarié.js
+++ b/source/sites/mycompanyinfrance.fr/pages/SocialSecurity/Salarié.js
@@ -55,14 +55,6 @@ const Salarié = ({ t }) => (
 			<T k="simulateurs.salarié.titre">Simulateur de salaire</T>
 		</h1>
 		<SalarySimulation />
-		<p>
-			<T k="simulateurs.salarié.description">
-				Dès que l'embauche d'un salarié est déclarée et qu'il est payé, il est
-				couvert par le régime général de la Sécurité sociale (santé, maternité,
-				invalidité, vieillesse, maladie professionnelle et accidents) et
-				chômage.
-			</T>
-		</p>
 	</>
 )
 export default compose(

From 837d595458ee0891e17f84e14e93ec6bb26fffa2 Mon Sep 17 00:00:00 2001
From: Mael <mael.thomas@data.gouv.fr>
Date: Fri, 19 Apr 2019 18:07:29 +0200
Subject: [PATCH 3/5] :bug: Annulation de la tentative de suggestion

---
 source/components/Feedback/FeedbackForm.js | 7 ++-----
 source/components/Feedback/PageFeedback.js | 7 ++++++-
 2 files changed, 8 insertions(+), 6 deletions(-)

diff --git a/source/components/Feedback/FeedbackForm.js b/source/components/Feedback/FeedbackForm.js
index 71062185d..cfdc076b8 100644
--- a/source/components/Feedback/FeedbackForm.js
+++ b/source/components/Feedback/FeedbackForm.js
@@ -6,7 +6,7 @@ import React, { Component } from 'react'
 import { Trans } from 'react-i18next'
 import type { Tracker } from 'Components/utils/withTracker'
 
-type Props = { onEnd: () => void, tracker: Tracker }
+type Props = { onEnd: () => void, tracker: Tracker, onCancel: () => void }
 
 class FeedbackForm extends Component<Props> {
 	formRef: ?HTMLFormElement
@@ -23,9 +23,6 @@ class FeedbackForm extends Component<Props> {
 			// $FlowFixMe
 			body: new FormData(this.formRef)
 		})
-		this.handleClose()
-	}
-	handleClose = () => {
 		this.props.onEnd()
 	}
 
@@ -34,7 +31,7 @@ class FeedbackForm extends Component<Props> {
 			<ScrollToElement onlyIfNotVisible>
 				<div style={{ textAlign: 'end' }}>
 					<button
-						onClick={this.handleClose}
+						onClick={() => this.props.onCancel()}
 						className="ui__ link-button"
 						style={{ textDecoration: 'none', marginLeft: '0.3rem' }}
 						aria-label="close">
diff --git a/source/components/Feedback/PageFeedback.js b/source/components/Feedback/PageFeedback.js
index fa906b422..01987ff66 100644
--- a/source/components/Feedback/PageFeedback.js
+++ b/source/components/Feedback/PageFeedback.js
@@ -145,7 +145,12 @@ class PageFeedback extends Component<Props, State> {
 					)}
 					{this.state.showForm && (
 						<Form
-							onEnd={() => this.setState({ showThanks: true, showForm: false })}
+							onEnd={() =>
+								this.setState({ showThanks: false, showForm: false })
+							}
+							onCancel={() =>
+								this.setState({ showThanks: false, showForm: false })
+							}
 						/>
 					)}
 				</div>

From 7b87b8009b6263161675cdeadbf920424a2419c1 Mon Sep 17 00:00:00 2001
From: Mael <mael.thomas@data.gouv.fr>
Date: Fri, 19 Apr 2019 19:04:56 +0200
Subject: [PATCH 4/5] :art: design final des retours utilsiateur

---
 source/components/Feedback/Feedback.css    |  13 ++-
 source/components/Feedback/PageFeedback.js | 109 +++++++++++----------
 2 files changed, 64 insertions(+), 58 deletions(-)

diff --git a/source/components/Feedback/Feedback.css b/source/components/Feedback/Feedback.css
index f2726d4fd..e379e7d8c 100644
--- a/source/components/Feedback/Feedback.css
+++ b/source/components/Feedback/Feedback.css
@@ -1,14 +1,11 @@
 .feedback-page {
-	align-items: flex-start;
 	display: flex;
-	justify-content: center;
+	justify-content: flex-end;
 	padding-top: 0.6rem;
 	padding-bottom: 0.6rem;
 	background: var(--lighterColour);
 	border-radius: 0.9rem;
-	max-width: 36em;
-	padding: 0.6em;
-	margin: 1em 0 1em auto;
+	padding: 0.6em 1em;
 }
 .feedback-page.stickToFooter {
 	margin-bottom: 0;
@@ -18,3 +15,9 @@
 .feedback-page button.link-button {
 	margin: 0 0.6em;
 }
+
+@media (min-width: 1200px) {
+	.feedback-page .feedbackButtons {
+		display: inline;
+	}
+}
diff --git a/source/components/Feedback/PageFeedback.js b/source/components/Feedback/PageFeedback.js
index 01987ff66..ed26919bc 100644
--- a/source/components/Feedback/PageFeedback.js
+++ b/source/components/Feedback/PageFeedback.js
@@ -16,7 +16,8 @@ import classNames from 'classnames'
 type OwnProps = {
 	blacklist: Array<string>,
 	customMessage?: Node,
-	customEventName?: string
+	customEventName?: string,
+	stickToFooter: boolean
 }
 type Props = OwnProps & {
 	location: Location,
@@ -98,61 +99,63 @@ class PageFeedback extends Component<Props, State> {
 			this.props.location.pathname === '/' ? '' : this.props.location.pathname
 		return (
 			!this.props.blacklist.includes(pathname) && (
-				<div
-					className={classNames('feedback-page', 'ui__', 'notice', {
-						stickToFooter
-					})}>
-					{!this.state.showForm && !this.state.showThanks && (
-						<>
-							<div>
-								{this.props.customMessage || (
-									<Trans i18nKey="feedback.question">
-										Cette page vous est utile ?
-									</Trans>
-								)}{' '}
-								<div style={{ display: 'inline-block' }}>
-									<button
-										className="ui__ link-button"
-										onClick={() => this.handleFeedback({ useful: true })}>
-										<Trans>Oui</Trans>
-									</button>{' '}
-									<button
-										className="ui__ link-button"
-										onClick={() => this.handleFeedback({ useful: false })}>
-										<Trans>Non</Trans>
-									</button>
-									<button
-										className="ui__ link-button"
-										onClick={this.handleErrorReporting}>
-										<Trans i18nKey="feedback.reportError">
-											Faire une suggestion
+				<div style={{ display: 'flex', justifyContent: 'center' }}>
+					<div
+						className={classNames('feedback-page', 'ui__', 'notice', {
+							stickToFooter
+						})}>
+						{!this.state.showForm && !this.state.showThanks && (
+							<>
+								<div>
+									{this.props.customMessage || (
+										<Trans i18nKey="feedback.question">
+											Cette page vous est utile ?
 										</Trans>
-									</button>
+									)}{' '}
+									<div className="feedbackButtons">
+										<button
+											className="ui__ link-button"
+											onClick={() => this.handleFeedback({ useful: true })}>
+											<Trans>Oui</Trans>
+										</button>{' '}
+										<button
+											className="ui__ link-button"
+											onClick={() => this.handleFeedback({ useful: false })}>
+											<Trans>Non</Trans>
+										</button>
+										<button
+											className="ui__ link-button"
+											onClick={this.handleErrorReporting}>
+											<Trans i18nKey="feedback.reportError">
+												Faire une suggestion
+											</Trans>
+										</button>
+									</div>
 								</div>
+							</>
+						)}
+						{this.state.showThanks && (
+							<div>
+								<Trans i18nKey="feedback.thanks">
+									Merci pour votre retour ! Vous pouvez nous contacter
+									directement à{' '}
+									<a href="mailto:contact@embauche.beta.gouv.fr">
+										contact@embauche.beta.gouv.fr
+									</a>
+								</Trans>
 							</div>
-						</>
-					)}
-					{this.state.showThanks && (
-						<div>
-							<Trans i18nKey="feedback.thanks">
-								Merci pour votre retour ! Vous pouvez nous contacter directement
-								à{' '}
-								<a href="mailto:contact@embauche.beta.gouv.fr">
-									contact@embauche.beta.gouv.fr
-								</a>
-							</Trans>
-						</div>
-					)}
-					{this.state.showForm && (
-						<Form
-							onEnd={() =>
-								this.setState({ showThanks: false, showForm: false })
-							}
-							onCancel={() =>
-								this.setState({ showThanks: false, showForm: false })
-							}
-						/>
-					)}
+						)}
+						{this.state.showForm && (
+							<Form
+								onEnd={() =>
+									this.setState({ showThanks: false, showForm: false })
+								}
+								onCancel={() =>
+									this.setState({ showThanks: false, showForm: false })
+								}
+							/>
+						)}
+					</div>
 				</div>
 			)
 		)

From b2c0b70fd6827d3b4f044a1f290c41f41249a93a Mon Sep 17 00:00:00 2001
From: Mael <mael.thomas@data.gouv.fr>
Date: Fri, 19 Apr 2019 19:05:01 +0200
Subject: [PATCH 5/5] =?UTF-8?q?:sparkles:=20S=C3=A9paration=20simulation?=
 =?UTF-8?q?=20g=C3=A9n=C3=A9rique=20et=20simulation=20cotisations?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Le sélecteur noUserInput fait ce qu'il dit faire. Un autre spécifique à
TargetSelection est créé. En conséquence, l'explication y est intégrée.
---
 source/components/QuickLinks.js               | 16 ++--------
 source/components/Simulation.js               |  6 ++--
 source/components/TargetSelection.js          | 23 ++++++++-----
 source/components/targetSelectionSelectors.js | 32 +++++++++++++++++++
 source/selectors/analyseSelectors.js          | 28 +++-------------
 .../pages/SocialSecurity/AssimiléSalarié.js   | 31 ++++++++++--------
 .../pages/SocialSecurity/AutoEntrepreneur.js  |  7 ++--
 .../pages/SocialSecurity/Indépendant.js       | 15 +++++----
 .../pages/SocialSecurity/Salarié.js           |  5 ++-
 9 files changed, 90 insertions(+), 73 deletions(-)
 create mode 100644 source/components/targetSelectionSelectors.js

diff --git a/source/components/QuickLinks.js b/source/components/QuickLinks.js
index 72d2fcbad..021fd2986 100644
--- a/source/components/QuickLinks.js
+++ b/source/components/QuickLinks.js
@@ -7,7 +7,6 @@ import { Trans } from 'react-i18next'
 import { connect } from 'react-redux'
 import { withRouter } from 'react-router'
 import { animated, Spring } from 'react-spring'
-import { noUserInputSelector } from 'Selectors/analyseSelectors'
 import type { Location } from 'react-router'
 
 type OwnProps = {
@@ -16,17 +15,10 @@ type OwnProps = {
 type Props = OwnProps & {
 	startConversation: (?string) => void,
 	location: Location,
-	userInput: boolean,
-	conversationStarted: boolean
+	show: boolean
 }
 
-const QuickLinks = ({
-	startConversation,
-	userInput,
-	quickLinks,
-	conversationStarted
-}: Props) => {
-	const show = userInput && !conversationStarted
+const QuickLinks = ({ startConversation, show, quickLinks }: Props) => {
 	return (
 		<Spring
 			to={{
@@ -45,7 +37,7 @@ const QuickLinks = ({
 						flexWrap: 'wrap-reverse',
 						fontSize: '110%',
 						justifyContent: 'space-evenly',
-						marginBottom: '0.6rem'
+						marginBottom: '1rem'
 					}}>
 					{toPairs(quickLinks).map(([label, dottedName]) => (
 						<button
@@ -67,8 +59,6 @@ export default (compose(
 	connect(
 		(state, props) => ({
 			key: props.language,
-			userInput: !noUserInputSelector(state),
-			conversationStarted: state.conversationStarted,
 			quickLinks: state.simulation?.config["questions à l'affiche"]
 		}),
 		{
diff --git a/source/components/Simulation.js b/source/components/Simulation.js
index e874fc8d8..05d78714d 100644
--- a/source/components/Simulation.js
+++ b/source/components/Simulation.js
@@ -45,8 +45,7 @@ export default compose(
 			let arePreviousAnswers = previousAnswers.length > 0,
 				displayConversation =
 					!targetsTriggerConversation || conversationStarted,
-				showTargets =
-					targetsTriggerConversation || !noUserInput || showTargetsAnyway
+				showTargets = targetsTriggerConversation || showTargetsAnyway
 			return (
 				<>
 					{this.state.displayAnswers && (
@@ -102,7 +101,7 @@ export default compose(
 
 					{showTargets && this.props.targets}
 					{!noFeedback && (
-						<div style={{ margin: '-0.6rem' }}>
+						<div>
 							<PageFeedback
 								customMessage={
 									<T k="feedback.simulator">
@@ -113,7 +112,6 @@ export default compose(
 							/>
 						</div>
 					)}
-					{!noUserInput && this.props.explanation}
 				</>
 			)
 		}
diff --git a/source/components/TargetSelection.js b/source/components/TargetSelection.js
index ee4f100a0..6aaef3b01 100644
--- a/source/components/TargetSelection.js
+++ b/source/components/TargetSelection.js
@@ -14,14 +14,14 @@ import { Link } from 'react-router-dom'
 import { change, Field, formValueSelector, reduxForm } from 'redux-form'
 import {
 	analysisWithDefaultsSelector,
-	flatRulesSelector,
-	noUserInputSelector
+	flatRulesSelector
 } from 'Selectors/analyseSelectors'
 import Animate from 'Ui/animate'
 import AnimatedTargetValue from 'Ui/AnimatedTargetValue'
 import CurrencyInput from './CurrencyInput/CurrencyInput'
 import QuickLinks from './QuickLinks'
 import './TargetSelection.css'
+import { firstStepCompletedSelector } from './targetSelectionSelectors'
 
 export default compose(
 	withTranslation(),
@@ -36,7 +36,7 @@ export default compose(
 				formValueSelector('conversation')(state, dottedName),
 			analysis: analysisWithDefaultsSelector(state),
 			flatRules: flatRulesSelector(state),
-			noUserInput: noUserInputSelector(state),
+			firstStepCompleted: firstStepCompletedSelector(state),
 			conversationStarted: state.conversationStarted,
 			activeInput: state.activeTargetInput,
 			objectifs: state.simulation?.config.objectifs || []
@@ -80,12 +80,18 @@ export default compose(
 			}
 		}
 		render() {
-			let { colours, noUserInput, analysis } = this.props,
+			let {
+					colours,
+					firstStepCompleted,
+					conversationStarted,
+					analysis,
+					explanation
+				} = this.props,
 				inversionFail = analysis.cache.inversionFail
 
 			return (
 				<div id="targetSelection">
-					{!noUserInput && (
+					{firstStepCompleted && (
 						<Controls
 							inversionFail={inversionFail}
 							controls={analysis.controls}
@@ -105,7 +111,8 @@ export default compose(
 						}}>
 						{this.renderOutputList()}
 					</section>
-					<QuickLinks />
+					<QuickLinks show={firstStepCompleted && !conversationStarted} />
+					{firstStepCompleted && explanation}
 				</div>
 			)
 		}
@@ -258,7 +265,7 @@ let TargetInputOrValue = withLanguage(
 		activeInput,
 		setActiveInput,
 		language,
-		noUserInput,
+		firstStepCompleted,
 		inversionFail
 	}) => (
 		<span className="targetInputOrValue">
@@ -278,7 +285,7 @@ let TargetInputOrValue = withLanguage(
 						target,
 						activeInput,
 						setActiveInput,
-						noUserInput,
+						firstStepCompleted,
 						inversionFail
 					}}
 				/>
diff --git a/source/components/targetSelectionSelectors.js b/source/components/targetSelectionSelectors.js
new file mode 100644
index 000000000..d91523a89
--- /dev/null
+++ b/source/components/targetSelectionSelectors.js
@@ -0,0 +1,32 @@
+import { createSelector } from 'reselect'
+import {
+	formattedSituationSelector,
+	targetNamesSelector,
+	parsedRulesSelector
+} from 'Selectors/analyseSelectors'
+import { findRuleByDottedName } from 'Engine/rules'
+
+export let firstStepCompletedSelector = createSelector(
+	[
+		formattedSituationSelector,
+		targetNamesSelector,
+		parsedRulesSelector,
+		state => state.simulation?.config?.bloquant
+	],
+	(situation, targetNames, parsedRules, bloquant) => {
+		if (!situation) {
+			return true
+		}
+		const situations = Object.keys(situation)
+		const allBlockingAreAnswered =
+			bloquant && bloquant.every(rule => situations.includes(rule))
+		const targetIsAnswered =
+			targetNames &&
+			targetNames.some(
+				targetName =>
+					findRuleByDottedName(parsedRules, targetName)?.formule &&
+					targetName in situation
+			)
+		return allBlockingAreAnswered || targetIsAnswered
+	}
+)
diff --git a/source/selectors/analyseSelectors.js b/source/selectors/analyseSelectors.js
index 5b7fd63e5..e63472139 100644
--- a/source/selectors/analyseSelectors.js
+++ b/source/selectors/analyseSelectors.js
@@ -21,7 +21,9 @@ import {
 	intersection,
 	isNil,
 	mergeDeepWith,
-	pick
+	pick,
+	isEmpty,
+	dissoc
 } from 'ramda'
 import { getFormValues } from 'redux-form'
 import { createSelector, createSelectorCreator, defaultMemoize } from 'reselect'
@@ -65,28 +67,8 @@ export let formattedSituationSelector = createSelector(
 )
 
 export let noUserInputSelector = createSelector(
-	[
-		formattedSituationSelector,
-		targetNamesSelector,
-		parsedRulesSelector,
-		state => state.simulation?.config?.bloquant
-	],
-	(situation, targetNames, parsedRules, bloquant) => {
-		if (!situation) {
-			return true
-		}
-		const situations = Object.keys(situation)
-		const allBlockingAreAnswered =
-			bloquant && bloquant.every(rule => situations.includes(rule))
-		const targetIsAnswered =
-			targetNames &&
-			targetNames.some(
-				targetName =>
-					findRuleByDottedName(parsedRules, targetName)?.formule &&
-					targetName in situation
-			)
-		return !(allBlockingAreAnswered || targetIsAnswered)
-	}
+	[formattedSituationSelector],
+	situation => !situation || isEmpty(dissoc('période', situation))
 )
 
 let validatedStepsSelector = createSelector(
diff --git a/source/sites/mycompanyinfrance.fr/pages/SocialSecurity/AssimiléSalarié.js b/source/sites/mycompanyinfrance.fr/pages/SocialSecurity/AssimiléSalarié.js
index b34e2990b..0c6fc31b3 100644
--- a/source/sites/mycompanyinfrance.fr/pages/SocialSecurity/AssimiléSalarié.js
+++ b/source/sites/mycompanyinfrance.fr/pages/SocialSecurity/AssimiléSalarié.js
@@ -36,20 +36,23 @@ const AssimiléSalarié = ({ t }) => (
 		<Warning />
 		<Simulation
 			targetsTriggerConversation={true}
-			targets={<TargetSelection />}
-			explanation={
-				<>
-					<p>
-						{emoji('☂️ ')}{' '}
-						<T k="simulateurs.assimilé-salarié.explications">
-							Les gérants égalitaires ou minoritaires de SARL ou les dirigeants
-							de SA et SAS sont assimilés salariés et relèvent du régime
-							général. Par conséquent, le dirigeant a la même protection sociale
-							qu'un salarié, mis à part le chômage.
-						</T>
-					</p>
-					<SalaryExplanation />
-				</>
+			targets={
+				<TargetSelection
+					explanation={
+						<>
+							<p>
+								{emoji('☂️ ')}{' '}
+								<T k="simulateurs.assimilé-salarié.explications">
+									Les gérants égalitaires ou minoritaires de SARL ou les
+									dirigeants de SA et SAS sont assimilés salariés et relèvent du
+									régime général. Par conséquent, le dirigeant a la même
+									protection sociale qu'un salarié, mis à part le chômage.
+								</T>
+							</p>
+							<SalaryExplanation />
+						</>
+					}
+				/>
 			}
 		/>
 	</>
diff --git a/source/sites/mycompanyinfrance.fr/pages/SocialSecurity/AutoEntrepreneur.js b/source/sites/mycompanyinfrance.fr/pages/SocialSecurity/AutoEntrepreneur.js
index 6a54fa5af..3b6faa9fb 100644
--- a/source/sites/mycompanyinfrance.fr/pages/SocialSecurity/AutoEntrepreneur.js
+++ b/source/sites/mycompanyinfrance.fr/pages/SocialSecurity/AutoEntrepreneur.js
@@ -35,8 +35,11 @@ const AutoEntrepreneur = ({ t }) => (
 		<Warning simulateur="auto-entreprise" />
 		<Simulation
 			targetsTriggerConversation={true}
-			targets={<TargetSelection />}
-			explanation={<AvertissementProtectionSocialeIndépendants />}
+			targets={
+				<TargetSelection
+					explanation={<AvertissementProtectionSocialeIndépendants />}
+				/>
+			}
 		/>
 	</>
 )
diff --git a/source/sites/mycompanyinfrance.fr/pages/SocialSecurity/Indépendant.js b/source/sites/mycompanyinfrance.fr/pages/SocialSecurity/Indépendant.js
index fc0650438..72efb2079 100644
--- a/source/sites/mycompanyinfrance.fr/pages/SocialSecurity/Indépendant.js
+++ b/source/sites/mycompanyinfrance.fr/pages/SocialSecurity/Indépendant.js
@@ -35,12 +35,15 @@ const Indépendant = ({ t }) => (
 		<Warning />
 		<Simulation
 			targetsTriggerConversation={true}
-			targets={<TargetSelection />}
-			explanation={
-				<>
-					<AvertissementForfaitIndépendants />
-					<AvertissementProtectionSocialeIndépendants />
-				</>
+			targets={
+				<TargetSelection
+					explanation={
+						<>
+							<AvertissementForfaitIndépendants />
+							<AvertissementProtectionSocialeIndépendants />
+						</>
+					}
+				/>
 			}
 		/>
 	</>
diff --git a/source/sites/mycompanyinfrance.fr/pages/SocialSecurity/Salarié.js b/source/sites/mycompanyinfrance.fr/pages/SocialSecurity/Salarié.js
index 0f1685063..e121eab55 100644
--- a/source/sites/mycompanyinfrance.fr/pages/SocialSecurity/Salarié.js
+++ b/source/sites/mycompanyinfrance.fr/pages/SocialSecurity/Salarié.js
@@ -1,5 +1,4 @@
 import { React, T } from 'Components'
-import SalaryExplanation from 'Components/SalaryExplanation'
 import Simulation from 'Components/Simulation'
 import salariéConfig from 'Components/simulationConfigs/salarié.yaml'
 import withSimulationConfig from 'Components/simulationConfigs/withSimulationConfig'
@@ -9,6 +8,7 @@ import { compose } from 'ramda'
 import { Helmet } from 'react-helmet'
 import { withTranslation } from 'react-i18next'
 import { Link } from 'react-router-dom'
+import SalaryExplanation from 'Components/SalaryExplanation'
 
 export let SalarySimulation = withSitePaths(({ sitePaths }) => (
 	<Simulation
@@ -29,8 +29,7 @@ export let SalarySimulation = withSitePaths(({ sitePaths }) => (
 				)}
 			</>
 		}
-		targets={<TargetSelection />}
-		explanation={<SalaryExplanation />}
+		targets={<TargetSelection explanation={<SalaryExplanation />} />}
 	/>
 ))