From 8676889db183d2216bc4d29f6f29ca7c93bdb22e Mon Sep 17 00:00:00 2001
From: AlexisG <merkkur39@gmail.com>
Date: Wed, 6 Nov 2024 12:12:41 +0100
Subject: [PATCH 1/3] feat: Edit only Scan step in Modify action

This action is used in the three-dot menu of each file.
---
 src/components/Actions/Items/modify.jsx       |  7 +++-
 .../Contexts/StepperDialogProvider.jsx        | 17 +++++++-
 .../ModelSteps/Scan/ScanWrapper.jsx           |  9 ++++-
 .../ScanResult/ScanResultDialog.jsx           | 40 ++++++++++++++++---
 .../ScanResult/ScanResultDialog.spec.jsx      | 15 ++++++-
 src/components/Views/Edit.jsx                 | 26 ++++++++++--
 src/helpers/filterSteps.js                    | 11 ++++-
 7 files changed, 106 insertions(+), 19 deletions(-)

diff --git a/src/components/Actions/Items/modify.jsx b/src/components/Actions/Items/modify.jsx
index 13c0b40b..7a4b18da 100644
--- a/src/components/Actions/Items/modify.jsx
+++ b/src/components/Actions/Items/modify.jsx
@@ -22,9 +22,14 @@ export const modify = ({ t, navigate }) => {
       getCreatedByApp(docs[0]) === 'mespapiers',
     action: docs => {
       const country = docs[0].metadata.country?.toLowerCase()
+      const searchParams = new URLSearchParams({
+        model: 'scan',
+        ...(country && { country })
+      }).toString()
+
       navigate({
         pathname: `edit/${docs[0]._id}`,
-        ...(country && { search: `?country=${country}` })
+        search: searchParams
       })
     },
     // eslint-disable-next-line react/display-name
diff --git a/src/components/Contexts/StepperDialogProvider.jsx b/src/components/Contexts/StepperDialogProvider.jsx
index fc316bd6..6c341c47 100644
--- a/src/components/Contexts/StepperDialogProvider.jsx
+++ b/src/components/Contexts/StepperDialogProvider.jsx
@@ -18,7 +18,13 @@ import { useWebviewIntent } from 'cozy-intent'
 
 const StepperDialogContext = createContext()
 
-export const StepperDialogProvider = ({ children, isEdit }) => {
+/**
+ * @param {object} props
+ * @param {React.ReactNode} props.children
+ * @param {boolean} props.isEdit - True if the dialog is in edit mode
+ * @param {function} props.stepFilterFn - The filter function to apply on the steps
+ */
+export const StepperDialogProvider = ({ children, isEdit, stepFilterFn }) => {
   const [stepperDialogTitle, setStepperDialogTitle] = useState('')
   const [allCurrentSteps, setAllCurrentSteps] = useState([])
   const [currentStepIndex, setCurrentStepIndex] = useState(-1)
@@ -68,6 +74,7 @@ export const StepperDialogProvider = ({ children, isEdit }) => {
             steps: allCurrentStepsDefinitions,
             webviewIntent,
             isEdit,
+            stepFilterFn,
             fromFlagshipUpload
           })
           setAllCurrentSteps(filteredSteps)
@@ -75,7 +82,13 @@ export const StepperDialogProvider = ({ children, isEdit }) => {
       }
       buildAllCurrentSteps()
     }
-  }, [webviewIntent, currentDefinition, fromFlagshipUpload, isEdit])
+  }, [
+    webviewIntent,
+    currentDefinition,
+    fromFlagshipUpload,
+    isEdit,
+    stepFilterFn
+  ])
 
   useEffect(() => {
     const loadCreatePaperDataBackup = async () => {
diff --git a/src/components/ModelSteps/Scan/ScanWrapper.jsx b/src/components/ModelSteps/Scan/ScanWrapper.jsx
index 45e11ac2..c8219811 100644
--- a/src/components/ModelSteps/Scan/ScanWrapper.jsx
+++ b/src/components/ModelSteps/Scan/ScanWrapper.jsx
@@ -1,4 +1,5 @@
 import { PDFDocument } from 'pdf-lib'
+import propTypes from 'prop-types'
 import React, { useEffect, useState } from 'react'
 import { useParams, useSearchParams } from 'react-router-dom'
 import { PaperDefinitionsStepPropTypes } from 'src/PaperDefinitionsPropTypes'
@@ -37,7 +38,7 @@ const isFileEncryptedPDF = async file => {
   return pdf.isEncrypted
 }
 
-const ScanWrapper = ({ currentStep, onClose, onBack }) => {
+const ScanWrapper = ({ currentStep, onClose, onBack, onSubmit }) => {
   const client = useClient()
   const [searchParams] = useSearchParams()
   const { qualificationLabel } = useParams()
@@ -153,6 +154,7 @@ const ScanWrapper = ({ currentStep, onClose, onBack }) => {
         onChangeFile={onChangeFile}
         onClose={onClose}
         onBack={onBack}
+        onSubmit={onSubmit}
       />
     )
   }
@@ -180,7 +182,10 @@ const ScanWrapper = ({ currentStep, onClose, onBack }) => {
 }
 
 ScanWrapper.propTypes = {
-  currentStep: PaperDefinitionsStepPropTypes
+  currentStep: PaperDefinitionsStepPropTypes,
+  onClose: propTypes.func.isRequired,
+  onBack: propTypes.func.isRequired,
+  onSubmit: propTypes.func.isRequired
 }
 
 export default ScanWrapper
diff --git a/src/components/ModelSteps/ScanResult/ScanResultDialog.jsx b/src/components/ModelSteps/ScanResult/ScanResultDialog.jsx
index 1ae4f02d..2e12ebb8 100644
--- a/src/components/ModelSteps/ScanResult/ScanResultDialog.jsx
+++ b/src/components/ModelSteps/ScanResult/ScanResultDialog.jsx
@@ -1,4 +1,5 @@
 import cx from 'classnames'
+import propTypes from 'prop-types'
 import React, { useRef, useState } from 'react'
 import { useSearchParams } from 'react-router-dom'
 import CompositeHeaderImage from 'src/components/CompositeHeader/CompositeHeaderImage'
@@ -8,6 +9,7 @@ import OcrProcessingDialog from 'src/components/ModelSteps/ScanResult/OcrProcess
 import ScanResultCard from 'src/components/ModelSteps/ScanResult/ScanResultCard/ScanResultCard'
 import ScanResultTitle from 'src/components/ModelSteps/ScanResult/ScanResultTitle'
 import { makeFileFromBase64 } from 'src/components/ModelSteps/helpers'
+import SubmitButton from 'src/components/ModelSteps/widgets/SubmitButton'
 import StepperDialogTitle from 'src/components/StepperDialog/StepperDialogTitle'
 import { FLAGSHIP_SCAN_TEMP_FILENAME, KEYS } from 'src/constants'
 import { isFlagshipOCRAvailable } from 'src/helpers/isFlagshipOCRAvailable'
@@ -22,11 +24,22 @@ import useEventListener from 'cozy-ui/transpiled/react/hooks/useEventListener'
 import useBreakpoints from 'cozy-ui/transpiled/react/providers/Breakpoints'
 import { useI18n } from 'cozy-ui/transpiled/react/providers/I18n'
 
+/**
+ * @param {object} props
+ * @param {object} props.currentStep
+ * @param {Function} props.onClose
+ * @param {Function} props.onBack
+ * @param {Function} props.onChangeFile
+ * @param {Function} props.onSubmit
+ * @param {object} props.currentFile
+ * @param {Function} props.setCurrentFile
+ */
 const ScanResultDialog = ({
   currentStep,
   onClose,
   onBack,
   onChangeFile,
+  onSubmit,
   currentFile,
   setCurrentFile
 }) => {
@@ -96,6 +109,7 @@ const ScanResultDialog = ({
       <OcrProcessingDialog rotatedFile={currentFileRotated} onBack={onBack} />
     )
   }
+  const SubmitButtonComponent = isLastStep() ? SubmitButton : null
 
   return (
     <FixedDialog
@@ -140,12 +154,16 @@ const ScanResultDialog = ({
       }
       actions={
         <>
-          <Button
-            data-testid="next-button"
-            fullWidth
-            label={t('common.next')}
-            onClick={handleNextStep}
-          />
+          {SubmitButtonComponent ? (
+            <SubmitButtonComponent formData={formData} onSubmit={onSubmit} />
+          ) : (
+            <Button
+              data-testid="next-button"
+              fullWidth
+              label={t('common.next')}
+              onClick={handleNextStep}
+            />
+          )}
           {multipage && (
             <ButtonLink
               className="u-ml-0 u-mb-half"
@@ -164,4 +182,14 @@ const ScanResultDialog = ({
   )
 }
 
+ScanResultDialog.propTypes = {
+  currentStep: propTypes.object.isRequired,
+  onClose: propTypes.func.isRequired,
+  onBack: propTypes.func.isRequired,
+  onChangeFile: propTypes.func.isRequired,
+  currentFile: propTypes.object.isRequired,
+  setCurrentFile: propTypes.func.isRequired,
+  onSubmit: propTypes.func.isRequired
+}
+
 export default ScanResultDialog
diff --git a/src/components/ModelSteps/ScanResult/ScanResultDialog.spec.jsx b/src/components/ModelSteps/ScanResult/ScanResultDialog.spec.jsx
index 524d722d..eae30cdf 100644
--- a/src/components/ModelSteps/ScanResult/ScanResultDialog.spec.jsx
+++ b/src/components/ModelSteps/ScanResult/ScanResultDialog.spec.jsx
@@ -126,6 +126,17 @@ describe('AcquisitionResult component:', () => {
     expect(btn).toBeDefined()
   })
 
+  describe('Submit', () => {
+    it('should submit button must exist if this is the last step.', async () => {
+      const { findByTestId } = setup({
+        currentFile: mockFile({ name: 'test.pdf' }),
+        isLastStep: jest.fn(() => true)
+      })
+      const btn = await findByTestId('ButtonSave')
+      expect(btn).toBeDefined()
+    })
+  })
+
   describe('setCurrentFile', () => {
     it('should setCurrentFile must be called once with null when restarting the file selection', async () => {
       const mockSetCurrentFile = jest.fn()
@@ -245,7 +256,7 @@ describe('AcquisitionResult component:', () => {
         const { getByTestId } = setup({
           currentFile: mockFile({ name: 'test.pdf' }),
           mockIsFlagshipOCRAvailable: true,
-          isLastStep: jest.fn(() => true),
+          isLastStep: jest.fn(() => false),
           currentDefinition: { ocrAttributes: [] },
           allCurrentSteps: [{ isDisplayed: 'ocr' }]
         })
@@ -262,7 +273,7 @@ describe('AcquisitionResult component:', () => {
         const { findByTestId } = setup({
           currentFile: mockFile({ name: FLAGSHIP_SCAN_TEMP_FILENAME }),
           mockIsFlagshipOCRAvailable: true,
-          isLastStep: jest.fn(() => true),
+          isLastStep: jest.fn(() => false),
           mockGetAttributesFromOcr,
           currentDefinition: { ocrAttributes: [] },
           allCurrentSteps: [{ isDisplayed: 'ocr' }]
diff --git a/src/components/Views/Edit.jsx b/src/components/Views/Edit.jsx
index c357a741..392ec2f4 100644
--- a/src/components/Views/Edit.jsx
+++ b/src/components/Views/Edit.jsx
@@ -1,5 +1,5 @@
 import React, { useEffect, useState } from 'react'
-import { useNavigate, useParams } from 'react-router-dom'
+import { useNavigate, useParams, useSearchParams } from 'react-router-dom'
 import { FormDataProvider } from 'src/components/Contexts/FormDataProvider'
 import { useFormData } from 'src/components/Contexts/FormDataProvider'
 import { StepperDialogProvider } from 'src/components/Contexts/StepperDialogProvider'
@@ -51,7 +51,14 @@ const EditPaperModal = ({ ioCozyFile, contacts, onClose, onSubmit }) => {
   return <StepperDialogWrapper onClose={onClose} onSubmit={onSubmit} />
 }
 
-export const EditLoader = ({ onClose, onSubmit }) => {
+/**
+ * Also used with the intent
+ * @param {object} props
+ * @param {Function} props.onClose
+ * @param {Function} props.onSubmit
+ * @param {Function} props.stepFilterFn
+ */
+export const EditLoader = ({ onClose, onSubmit, stepFilterFn }) => {
   const { fileId } = useParams()
 
   const buildedFilesQuery = buildFileQueryById(fileId)
@@ -61,11 +68,12 @@ export const EditLoader = ({ onClose, onSubmit }) => {
   )
   const { contacts, isLoadingContacts } = useReferencedContact(files || [])
 
-  if (isQueryLoading(filesQueryResult) || isLoadingContacts)
+  if (isQueryLoading(filesQueryResult) || isLoadingContacts) {
     return <Spinner className="u-m-0" size="xxlarge" middle color="white" />
+  }
 
   return (
-    <StepperDialogProvider isEdit={true}>
+    <StepperDialogProvider isEdit={true} stepFilterFn={stepFilterFn}>
       <FormDataProvider>
         <EditPaperModal
           ioCozyFile={files[0]}
@@ -80,11 +88,21 @@ export const EditLoader = ({ onClose, onSubmit }) => {
 
 const EditWrapper = () => {
   const navigate = useNavigate()
+  const [searchParams] = useSearchParams()
+  const stepModel = searchParams.get('model')
+
+  const stepFilterFn = step => {
+    if (!stepModel) {
+      return true
+    }
+    return step.model === stepModel
+  }
 
   return (
     <EditLoader
       onClose={() => navigate('..')}
       onSubmit={() => navigate('..')}
+      stepFilterFn={stepFilterFn}
     />
   )
 }
diff --git a/src/helpers/filterSteps.js b/src/helpers/filterSteps.js
index 5acd3a0b..66cd16aa 100644
--- a/src/helpers/filterSteps.js
+++ b/src/helpers/filterSteps.js
@@ -5,13 +5,16 @@ import { isSomePaperStepsCompliantWithOCR } from 'src/helpers/isSomePaperStepsCo
  * Filter a list of step with OCR contraint
  * @param {object[]} steps list of step
  * @param {Object} [webviewIntent] - The webview intent service to interact with the native environment.
- * @param {function} webviewIntent.call - A function to call native methods, expecting the method name and its arguments.
+ * @param {boolean} [isEdit] - True if the current view is an edit view
+ * @param {Function} [stepFilterFn] - The filter function to apply on the steps
+ * @param {boolean} [fromFlagshipUpload] - True if the current view is from the flagship upload
  * @returns {Promise<object[]>} list of step filtered
  */
 export const filterSteps = async ({
   steps,
   webviewIntent,
   isEdit,
+  stepFilterFn,
   fromFlagshipUpload
 }) => {
   const isOCR =
@@ -19,11 +22,15 @@ export const filterSteps = async ({
     !fromFlagshipUpload &&
     isSomePaperStepsCompliantWithOCR(steps) &&
     (await isFlagshipOCRAvailable(webviewIntent))
-  return steps.filter(step => {
+  const stepsDisplayed = steps.filter(step => {
     if (isOCR) {
       return step.isDisplayed === 'all' || step.isDisplayed === 'ocr'
     } else {
       return step?.isDisplayed !== 'ocr'
     }
   })
+
+  return typeof stepFilterFn === 'function'
+    ? stepsDisplayed.filter(stepFilterFn)
+    : stepsDisplayed
 }

From d62d0216cc6ec5503823a59bc9837469da541a20 Mon Sep 17 00:00:00 2001
From: AlexisG <merkkur39@gmail.com>
Date: Wed, 6 Nov 2024 12:12:57 +0100
Subject: [PATCH 2/3] feat: Remove unused file

---
 .../ScanResult/ScanResultActions.jsx          | 52 -------------------
 1 file changed, 52 deletions(-)
 delete mode 100644 src/components/ModelSteps/ScanResult/ScanResultActions.jsx

diff --git a/src/components/ModelSteps/ScanResult/ScanResultActions.jsx b/src/components/ModelSteps/ScanResult/ScanResultActions.jsx
deleted file mode 100644
index 83397d53..00000000
--- a/src/components/ModelSteps/ScanResult/ScanResultActions.jsx
+++ /dev/null
@@ -1,52 +0,0 @@
-import cx from 'classnames'
-import PropTypes from 'prop-types'
-import React from 'react'
-import { PaperDefinitionsStepPropTypes } from 'src/PaperDefinitionsPropTypes'
-
-import DialogActions from 'cozy-ui/transpiled/react/DialogActions'
-import Button, { ButtonLink } from 'cozy-ui/transpiled/react/deprecated/Button'
-import useBreakpoints from 'cozy-ui/transpiled/react/providers/Breakpoints'
-import { useI18n } from 'cozy-ui/transpiled/react/providers/I18n'
-
-const ScanResultActions = ({ currentStep, onNextStep, onRepeatStep }) => {
-  const { t } = useI18n()
-  const { isMobile } = useBreakpoints()
-  const { multipage } = currentStep
-
-  return (
-    <DialogActions
-      disableSpacing
-      className={cx('columnLayout u-mb-1-half u-mt-0 cozyDialogActions', {
-        'u-mh-1': !isMobile,
-        'u-mh-0': isMobile
-      })}
-    >
-      <Button
-        className="u-db"
-        data-testid="next-button"
-        extension="full"
-        label={t('common.next')}
-        onClick={onNextStep}
-      />
-      {multipage && (
-        <ButtonLink
-          className="u-ml-0 u-mb-half"
-          data-testid="repeat-button"
-          extension="full"
-          theme="secondary"
-          icon="camera"
-          label={t('Acquisition.repeat')}
-          onClick={onRepeatStep}
-        />
-      )}
-    </DialogActions>
-  )
-}
-
-ScanResultActions.propTypes = {
-  onNextStep: PropTypes.func,
-  onRepeatStep: PropTypes.func,
-  currentStep: PaperDefinitionsStepPropTypes
-}
-
-export default ScanResultActions

From d1e42f84c9df5bc98520dd67f0f25e2367655c85 Mon Sep 17 00:00:00 2001
From: AlexisG <merkkur39@gmail.com>
Date: Fri, 8 Nov 2024 09:14:32 +0100
Subject: [PATCH 3/3] feat: Update ScanResultDialog informations

The information is no longer related to the device,
but to the file type.
---
 .../ScanResultCardImageActions.jsx            | 10 +++----
 .../ScanResultCardPDFActions.jsx              |  6 +----
 .../ScanResult/ScanResultDialog.jsx           | 21 ++++++++++-----
 .../ModelSteps/ScanResult/ScanResultTitle.jsx | 11 ++++----
 src/components/ModelSteps/helpers.js          |  7 +++++
 src/locales/en.json                           | 26 +++++++++----------
 src/locales/fr.json                           | 24 ++++++++---------
 7 files changed, 57 insertions(+), 48 deletions(-)

diff --git a/src/components/ModelSteps/ScanResult/ScanResultCard/ScanResultCardImageActions.jsx b/src/components/ModelSteps/ScanResult/ScanResultCard/ScanResultCardImageActions.jsx
index fca2c794..38879fc1 100644
--- a/src/components/ModelSteps/ScanResult/ScanResultCard/ScanResultCardImageActions.jsx
+++ b/src/components/ModelSteps/ScanResult/ScanResultCard/ScanResultCardImageActions.jsx
@@ -4,7 +4,6 @@ import React from 'react'
 import Button from 'cozy-ui/transpiled/react/Buttons'
 import Icon from 'cozy-ui/transpiled/react/Icon'
 import IconButton from 'cozy-ui/transpiled/react/IconButton'
-import useBreakpoints from 'cozy-ui/transpiled/react/providers/Breakpoints'
 import { useI18n } from 'cozy-ui/transpiled/react/providers/I18n'
 import { makeStyles } from 'cozy-ui/transpiled/react/styles'
 
@@ -25,15 +24,12 @@ const ScanResultCardImageActions = ({
 }) => {
   const classes = useStyles()
   const { t } = useI18n()
-  const { isDesktop } = useBreakpoints()
-
-  const device = isDesktop ? 'desktop' : 'mobile'
 
   return (
     <>
       <Button
         data-testid="retry-button"
-        label={t(`Acquisition.${device}.retry`)}
+        label={t('Acquisition.image.edit')}
         fullWidth
         variant="secondary"
         onClick={onCancel}
@@ -43,8 +39,8 @@ const ScanResultCardImageActions = ({
         classes={classes}
         size="small"
         onClick={onRotate}
-        aria-label={t('Acquisition.rotate')}
-        title={t('Acquisition.rotate')}
+        aria-label={t('Acquisition.image.rotate')}
+        title={t('Acquisition.image.rotate')}
         disabled={isImageRotating}
       >
         <Icon icon="rotate-left" />
diff --git a/src/components/ModelSteps/ScanResult/ScanResultCard/ScanResultCardPDFActions.jsx b/src/components/ModelSteps/ScanResult/ScanResultCard/ScanResultCardPDFActions.jsx
index 7d49e6fe..bf17acd2 100644
--- a/src/components/ModelSteps/ScanResult/ScanResultCard/ScanResultCardPDFActions.jsx
+++ b/src/components/ModelSteps/ScanResult/ScanResultCard/ScanResultCardPDFActions.jsx
@@ -2,19 +2,15 @@ import PropTypes from 'prop-types'
 import React from 'react'
 
 import Button from 'cozy-ui/transpiled/react/Buttons'
-import useBreakpoints from 'cozy-ui/transpiled/react/providers/Breakpoints'
 import { useI18n } from 'cozy-ui/transpiled/react/providers/I18n'
 
 const ScanResultCardPDFActions = ({ onCancel }) => {
   const { t } = useI18n()
-  const { isDesktop } = useBreakpoints()
-
-  const device = isDesktop ? 'desktop' : 'mobile'
 
   return (
     <Button
       data-testid="retry-button"
-      label={t(`Acquisition.${device}.retry`)}
+      label={t('Acquisition.pdf.edit')}
       fullWidth
       variant="secondary"
       onClick={onCancel}
diff --git a/src/components/ModelSteps/ScanResult/ScanResultDialog.jsx b/src/components/ModelSteps/ScanResult/ScanResultDialog.jsx
index 2e12ebb8..9e857de1 100644
--- a/src/components/ModelSteps/ScanResult/ScanResultDialog.jsx
+++ b/src/components/ModelSteps/ScanResult/ScanResultDialog.jsx
@@ -8,7 +8,10 @@ import { useStepperDialog } from 'src/components/Contexts/StepperDialogProvider'
 import OcrProcessingDialog from 'src/components/ModelSteps/ScanResult/OcrProcessingDialog'
 import ScanResultCard from 'src/components/ModelSteps/ScanResult/ScanResultCard/ScanResultCard'
 import ScanResultTitle from 'src/components/ModelSteps/ScanResult/ScanResultTitle'
-import { makeFileFromBase64 } from 'src/components/ModelSteps/helpers'
+import {
+  isPDFFile,
+  makeFileFromBase64
+} from 'src/components/ModelSteps/helpers'
 import SubmitButton from 'src/components/ModelSteps/widgets/SubmitButton'
 import StepperDialogTitle from 'src/components/StepperDialog/StepperDialogTitle'
 import { FLAGSHIP_SCAN_TEMP_FILENAME, KEYS } from 'src/constants'
@@ -21,7 +24,6 @@ import { FixedDialog } from 'cozy-ui/transpiled/react/CozyDialogs'
 import PointerAlert from 'cozy-ui/transpiled/react/PointerAlert'
 import { ButtonLink } from 'cozy-ui/transpiled/react/deprecated/Button'
 import useEventListener from 'cozy-ui/transpiled/react/hooks/useEventListener'
-import useBreakpoints from 'cozy-ui/transpiled/react/providers/Breakpoints'
 import { useI18n } from 'cozy-ui/transpiled/react/providers/I18n'
 
 /**
@@ -47,15 +49,14 @@ const ScanResultDialog = ({
   const { t } = useI18n()
   const webviewIntent = useWebviewIntent()
   const [searchParams] = useSearchParams()
-  const { isDesktop } = useBreakpoints()
 
-  const device = isDesktop ? 'desktop' : 'mobile'
   const imageRef = useRef(null)
   const [rotationImage, setRotationImage] = useState(0)
   const [ocrProcessing, setOcrProcessing] = useState(false)
   const {
     currentStepIndex,
     nextStep,
+    isEdit,
     isLastStep,
     allCurrentSteps,
     currentDefinition
@@ -110,6 +111,12 @@ const ScanResultDialog = ({
     )
   }
   const SubmitButtonComponent = isLastStep() ? SubmitButton : null
+  const isEditStatusChanged =
+    !isEdit || (isEdit && currentFile.name !== formData.file.name)
+
+  const tooltipKey = t(
+    `Acquisition.${isPDFFile(currentFile) ? 'pdf' : 'image'}.tooltip.${page}`
+  )
 
   return (
     <FixedDialog
@@ -127,7 +134,9 @@ const ScanResultDialog = ({
         <div className={cx('u-flex u-flex-column u-flex-justify-center')}>
           {!formData.data?.[0]?.file?.isBlank && (
             <>
-              <ScanResultTitle />
+              {isEditStatusChanged && (
+                <ScanResultTitle currentFile={currentFile} />
+              )}
               {tooltip && (
                 <PointerAlert
                   className="u-mb-1"
@@ -138,7 +147,7 @@ const ScanResultDialog = ({
                     />
                   }
                 >
-                  {t(`Acquisition.${device}.tooltip.${page}`)}
+                  {tooltipKey}
                 </PointerAlert>
               )}
             </>
diff --git a/src/components/ModelSteps/ScanResult/ScanResultTitle.jsx b/src/components/ModelSteps/ScanResult/ScanResultTitle.jsx
index 0f369541..d6a24338 100644
--- a/src/components/ModelSteps/ScanResult/ScanResultTitle.jsx
+++ b/src/components/ModelSteps/ScanResult/ScanResultTitle.jsx
@@ -1,8 +1,8 @@
 import React from 'react'
+import { isPDFFile } from 'src/components/ModelSteps/helpers'
 
 import Icon from 'cozy-ui/transpiled/react/Icon'
 import Typography from 'cozy-ui/transpiled/react/Typography'
-import useBreakpoints from 'cozy-ui/transpiled/react/providers/Breakpoints'
 import { useI18n } from 'cozy-ui/transpiled/react/providers/I18n'
 import { makeStyles } from 'cozy-ui/transpiled/react/styles'
 
@@ -13,12 +13,13 @@ const useStyles = makeStyles(() => ({
   }
 }))
 
-const ScanResultTitle = () => {
+const ScanResultTitle = ({ currentFile }) => {
   const styles = useStyles()
   const { t } = useI18n()
-  const { isDesktop } = useBreakpoints()
 
-  const device = isDesktop ? 'desktop' : 'mobile'
+  const title = t(
+    `Acquisition.${isPDFFile(currentFile) ? 'pdf' : 'image'}.success`
+  )
 
   return (
     <div className="u-flex u-flex-items-center u-flex-justify-center u-mb-1-half">
@@ -28,7 +29,7 @@ const ScanResultTitle = () => {
         aria-hidden="true"
       />
       <Typography variant="h4" className={styles.typography} role="status">
-        {t(`Acquisition.${device}.success`)}
+        {title}
       </Typography>
     </div>
   )
diff --git a/src/components/ModelSteps/helpers.js b/src/components/ModelSteps/helpers.js
index 94a58386..682e4eab 100644
--- a/src/components/ModelSteps/helpers.js
+++ b/src/components/ModelSteps/helpers.js
@@ -432,3 +432,10 @@ export const normalizeFormdataMetadata = ({
 
   return merge({}, formData, metadataNormalized)
 }
+
+/**
+ * Check if a file is a PDF file
+ * @param {File} file - File object
+ * @returns {boolean}
+ */
+export const isPDFFile = file => file.type === 'application/pdf'
diff --git a/src/locales/en.json b/src/locales/en.json
index 0eedcba9..3779d5a0 100644
--- a/src/locales/en.json
+++ b/src/locales/en.json
@@ -35,23 +35,23 @@
   },
   "Acquisition": {
     "repeat": "Scan an additional page",
-    "rotate": "Rotate the image 90 degrees counter-clockwise",
-    "desktop": {
-      "retry": "Change picture",
-      "success": "The picture was added.",
+    "pdf": {
+      "success": "The document was added.",
+      "edit": "Change document",
       "tooltip": {
-        "back": "Check if it is indeed the back side and if the picture is well oriented as on the model",
-        "default": "Check if the picture is well oriented as on the model",
-        "front": "Check if it is indeed the front side and if the picture is well oriented as on the model"
+        "back": "Check if it is indeed the back side and if the document is well oriented as on the model",
+        "default": "Check if the document is well oriented as on the model",
+        "front": "Check if it is indeed the front side and if the document is well oriented as on the model"
       }
     },
-    "mobile": {
-      "retry": "Change photo",
-      "success": "The photo was added.",
+    "image": {
+      "rotate": "Rotate the image 90 degrees counter-clockwise",
+      "success": "The image was added.",
+      "edit": "Change image",
       "tooltip": {
-        "back": "Check if it is indeed the back side and if the photo is well oriented as on the model",
-        "default": "Check if the photo is well oriented as on the model",
-        "front": "Check if it is indeed the front side and if the photo is well oriented as on the model"
+        "back": "Check if it is indeed the back side and if the image is well oriented as on the model",
+        "default": "Check if the image is well oriented as on the model",
+        "front": "Check if it is indeed the front side and if the image is well oriented as on the model"
       }
     }
   },
diff --git a/src/locales/fr.json b/src/locales/fr.json
index 545966d2..f47f3de2 100644
--- a/src/locales/fr.json
+++ b/src/locales/fr.json
@@ -35,24 +35,24 @@
   },
   "Acquisition": {
     "repeat": "Scanner une page supplémentaire",
-    "rotate": "Tourner l'image à 90 degrés dans le sens anti-horaire",
-    "desktop": {
-      "retry": "Changer l'image",
+    "pdf": {
+      "success": "Le document a bien été ajouté.",
+      "edit": "Modifier le document",
+      "tooltip": {
+        "back": "Vérifier s’il s’agit bien de la face arrière et si le document est bien orienté comme sur le modèle",
+        "default": "Vérifier si le document est bien orienté comme sur le modèle",
+        "front": "Vérifier s’il s’agit bien de la face avant et si le document est bien orienté comme sur le modèle"
+      }
+    },
+    "image": {
+      "rotate": "Tourner l'image à 90 degrés dans le sens anti-horaire",
       "success": "L'image a bien été ajoutée.",
+      "edit": "Modifier l'image",
       "tooltip": {
         "back": "Vérifier s’il s’agit bien de la face arrière et si l'image est bien orientée comme sur le modèle",
         "default": "Vérifier si l'image est bien orientée comme sur le modèle",
         "front": "Vérifier s’il s’agit bien de la face avant et si l'image est bien orientée comme sur le modèle"
       }
-    },
-    "mobile": {
-      "retry": "Changer la photo",
-      "success": "La photo a bien été ajoutée.",
-      "tooltip": {
-        "back": "Vérifier s’il s’agit bien de la face arrière et si la photo est bien orientée comme sur le modèle",
-        "default": "Vérifier si la photo est bien orientée comme sur le modèle",
-        "front": "Vérifier s’il s’agit bien de la face avant et si la photo est bien orientée comme sur le modèle"
-      }
     }
   },
   "action": {