Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

🏓 Permettre l'export PDF ou impression du comparateur de statut #2406 #2835

Merged

Conversation

Grunt471
Copy link
Contributor

fix #2406

Contexte :
US
En tant qu'usager je souhaite pouvoir extraire en PDF ou via une impression les résultats du comparateur de statut

Complexité
S

Description du retour utilisateur
Un utilisatrice regrette qu'il ne soit pas possible d'exporter au format PDF ou d'impression les résultats du comparateur de statut.

URL de la page
https://mon-entreprise.urssaf.fr/simulateurs/comparaison-r%C3%A9gimes-sociaux

Description de la PR
Le bouton est désormais affiché sur la page comparateur de statut. Je me suis inspiré des travaux de la PR " 🐛🎨 Fix des éléments qui apparaissent à l'impression #1966 " afin d'améliorer la mise en page de l'impression.

Il reste cependant un bug d'affichage au niveau de l'impression, certains points de comparaison du comparateur étant absents ou incomplets (le point Adoption notamment).

Copy link

netlify bot commented Nov 25, 2023

Deploy Preview for nostalgic-mahavira-52b682 canceled.

Name Link
🔨 Latest commit cb75d31
🔍 Latest deploy log https://app.netlify.com/sites/nostalgic-mahavira-52b682/deploys/65b53a3cf44e710008ed144a

@johangirod johangirod force-pushed the issue-2406-export-pdf-comparateur-statut branch from 00f13d4 to c4b83ad Compare December 13, 2023 09:04
@johangirod
Copy link
Collaborator

Bonjour @Grunt471, je reviens vers vous sur cette PR. Après avoir testé localement, je ne peux pas merger en l'état. Le PDF de rendu lorsque l'on clique sur le bouton n'est pas assez abouti (testé sur Firefox).

  • Il manque des données cruciales comme le revenu pour chaque statut
  • Il y a trop d'espace, il faudrait que les valeurs soit affichées côte à côte (comme sur la page web) plutôt qu'en dessous les unes des autres.
  • Le bouton « Modifier mes options » ne devrait pas être affiché

comparaison-régimes-sociaux.pdf

Je vous remercie néanmoins pour votre contribution, et vous prie de m'excuser pour le délai de ma réponse. Comme expliqué précédemment, l'équipe est très réduite en ce moment.

Bien à vous,

@Grunt471
Copy link
Contributor Author

Merci pour vos retours. Je continue de travailler sur cette issue afin de résoudre les problèmes que vous avez soulevés.

Bien à vous,

@Grunt471 Grunt471 force-pushed the issue-2406-export-pdf-comparateur-statut branch from c4b83ad to dcc3f32 Compare December 20, 2023 17:35
@Grunt471
Copy link
Contributor Author

Bonsoir,
J'ai mis en place des correctifs en prenant compte des retours de votre message précédent.

Bien à vous,

@johangirod
Copy link
Collaborator

Bonjour,

Je reviens vers vous un peu tardivement, je n'ai pas eu beaucoup de temps ces dernières semaines.
Je viens de tester sur Firefox, et il manque des données dans l'impression :
comparaison-régimes-sociaux.pdf

Voici mes remarques :

  • EI et AE n'apparaissent pas dans la section revenu
  • AE n'apparait pas pour la retraite
  • La retraite complémentaire n'apparait pas
  • Les boutons « tout déplier » et « modifier mes options » n'ont pas de sens dans un PDF et devrait être caché
  • Les montants sont étendus sur la ligne entière, c'est beaucoup trop d'espace vide. On voudrait les voir côte à côte comme sur le site

Il est possible que ces problèmes ne soient présents que sur Firefox (je me souviens de différences que l'on avait dû régler lors de l'ajout du bouton sur les simulateurs). Quoi qu'il en soit, merci pour votre travail sur ce sujet difficile :)

@Grunt471 Grunt471 force-pushed the issue-2406-export-pdf-comparateur-statut branch from 37474ce to 164e363 Compare January 12, 2024 23:32
@Grunt471
Copy link
Contributor Author

Bonsoir,

Merci pour votre réponse. Le dernier commit résout le problème de l'affichage indésirable du bouton "tout déplier".
Concernant le reste des points que vous avez énoncé, de mon côté, voici le résultat de l'impression PDF issue de plusieurs navigateurs :
Chrome
CHROME-comparaison-régimes-sociaux.pdf
Firefox
FIREFOX-comparaison-régimes-sociaux.pdf

On peut constater l'absence, dans ces impressions, des problèmes d'affichage que vous décrivez dans votre précédent message.
Ces rendus correspondent-ils à vos attentes ?

Bien à vous,

@johangirod
Copy link
Collaborator

Oui, c'est bien mieux. Un peu de peaufinage pour réduire la taille du texte des blocs à l'impression, supprimer le padding/les margin et les bordures bleues de l'accordéon et ce sera parfait !
Dans tous les cas, merci pour le travail !

@Grunt471 Grunt471 force-pushed the issue-2406-export-pdf-comparateur-statut branch from 164e363 to cb75d31 Compare January 27, 2024 17:15
@Grunt471
Copy link
Contributor Author

Bonsoir,

Le dernier commit résout les problèmes soulevés dans votre précédent message.
Voici le rendu de l'impression pdf sous CHROME :
CHROME-comparaison-regimes-sociaux.pdf

Ce rendu correspond-il à vos attentes ?

Bien à vous,

@johangirod johangirod enabled auto-merge (squash) January 31, 2024 08:42
@johangirod
Copy link
Collaborator

C'est bien mieux ! Je merge la PR, un grand merci pour tout le travail effectué !

@johangirod johangirod disabled auto-merge January 31, 2024 08:42
@johangirod johangirod merged commit d7fb260 into betagouv:master Jan 31, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

🏓 Permettre l'export PDF ou impression du comparateur de statut
2 participants