(PC-34260)[PRO] feat: Temporarly set the recurrence forms dialog as d…

…rawer.
pass-culture · Feb 7, 2025 · 1400482 · 1400482
1 parent 0f2f53d
commit 1400482
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 2 deletions.
diff --git a/pro/src/components/IndividualOffer/StocksEventCreation/AddRecurrencesButton.tsx b/pro/src/components/IndividualOffer/StocksEventCreation/AddRecurrencesButton.tsx
@@ -47,6 +47,7 @@ export const AddRecurrencesButton = ({
       <DialogBuilder
         onOpenChange={setIsRecurrenceModalOpen}
         open={isRecurrenceModalOpen}
+        variant="drawer"
         trigger={
           <Button
             variant={ButtonVariant.PRIMARY}

diff --git a/pro/src/ui-kit/DialogBuilder/DialogBuilder.spec.tsx b/pro/src/ui-kit/DialogBuilder/DialogBuilder.spec.tsx
@@ -42,7 +42,7 @@ describe('DialogBuilder', () => {
     ).toBeInTheDocument()
     expect(screen.getByRole('button', { name: 'Annuler' })).toBeInTheDocument()
     expect(
-      screen.getByRole('button', { name: 'Fermer la modale' })
+      screen.getByRole('button', { name: 'Fermer la fenêtre modale' })
     ).toBeInTheDocument()
   })
 
@@ -54,7 +54,7 @@ describe('DialogBuilder', () => {
     )
 
     await userEvent.click(
-      screen.getByRole('button', { name: 'Fermer la modale' })
+      screen.getByRole('button', { name: 'Fermer la fenêtre modale' })
     )
     expect(
       screen.queryByRole('heading', { name: 'Dialog title' })

diff --git a/pro/src/ui-kit/DialogBuilder/DialogBuilder.tsx b/pro/src/ui-kit/DialogBuilder/DialogBuilder.tsx
@@ -111,6 +111,7 @@ export function DialogBuilder({
               // Detect if click actually happened within the bounds of content.
               // This can happen if click was on an absolutely positioned element overlapping content,
               // such as a click on the scroll bar (https://github.com/radix-ui/primitives/issues/1280)
+              /* istanbul ignore next */
               const actuallyClickedInside =
                 e.detail.originalEvent.clientX > contentRect.left &&
                 e.detail.originalEvent.clientX <