(PC-30155)[PRO] fix: protect canvas from being to big to fix error se…

[Aliochka]

…ntry

But de la pull request

Ticket Jira (ou description si BSR) : https://passculture.atlassian.net/browse/PC-30155

Vérifications

• J'ai écrit les tests nécessaires
• J'ai relu attentivement les migrations, en particulier pour éviter les locks, et je préviens les équipes Shérif et Data
• J'ai ajouté des screenshots pour d'éventuels changements graphiques

[github-actions]

Visit the preview URL for this PR (updated for commit d474834):

https://pc-pro-testing--pr12915-pc-30155-canveas-siz-opvemvi8.web.app

_{(expires Thu, 27 Jun 2024 08:14:01 GMT)}

_{🔥 via Firebase Hosting GitHub Action 🌎}

_{Sign: 032d233ee67e1c50d6af12e29c936c7076770eb1}

[AmineLouveau]

C'est pas énorme 2000 pour une valeur initiale ?

[Aliochka]

l'erreur arrive quand le canvas est beaucoup trop énorme, on m'a proposé de mettre un grand max height du coup pour que ça arrive pas

[AmineLouveau]

🤔 mais ça veut dire qu'on peut mettre une taille "normale", non ?

Si le problème c'est quand il est trop grand, pourquoi mettre une grande valeur ?

[xdekoninck-pass]

Yes c'est ce que j'avais en tête. On peut aussi rajouter un max-width pour couvrir tous les cas. Et si le problème persiste on pourra tenter un truc du genre

canvas.width = Math.min(canvas.parentElement.clientWidth, 1200);
canvas.height = Math.min(canvas.parentElement.clientHeight, 2000);