Skip to content

Conversation

@hellodeloo
Copy link
Contributor

@hellodeloo hellodeloo commented Sep 2, 2024

🤔 Quoi ?

  • Maj du thème pour supporter les règles globales d'impression
  • Ajout du header_print
  • Adaptation de la page "attestation imprimable" aux règles globales d'impression
  • Ajout du bouton "Imprimer” sur la page de détail Candidat

💻 Captures d'écran

capture 2024-09-02 à 15 39 02

@hellodeloo hellodeloo self-assigned this Sep 2, 2024
@hellodeloo hellodeloo added the ajouté Ajouté dans le changelog. label Sep 2, 2024
@notion-workspace
Copy link

@hellodeloo hellodeloo changed the title Deloo/add css print rules and print button UX/UI: Ajout du bouton "Imprimer” sur la page de détail Candidat Sep 2, 2024
@hellodeloo hellodeloo added the 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC label Sep 2, 2024
@github-actions
Copy link

github-actions bot commented Sep 2, 2024

🥁 La recette jetable est prête ! 👉 Je veux tester cette PR !

Copy link
Member

@francoisfreitag francoisfreitag left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pour nourrir les castors, pourrait-on faire un peu plus compact sur l’impression, histoire que ça tienne sur une page ?
image

@hellodeloo hellodeloo force-pushed the deloo/add-css-print-rules-and-print-button branch 2 times, most recently from 40e66bb to 828a8d0 Compare September 3, 2024 10:25
@hellodeloo
Copy link
Contributor Author

hellodeloo commented Sep 3, 2024

Pour nourrir les castors, pourrait-on faire un peu plus compact sur l’impression, histoire que ça tienne sur une page ? !

Pour l'instant c'est des règles print globales, pour que ça "marche" sur toutes les pages.
Ensuite, la mise en page change un peu selon la config d'impression de l'utilisateur et les navigateurs (ex sous ff et chrome chez moi)
capture 2024-09-03 à 12 04 44
capture 2024-09-03 à 12 04 35

Si je voulais faire ça, je devrais adapter plus longuement la page :/

Par contre, j'ai tassé un peu plus l'interlignage et reduit la taille par default du texte

@xavfernandez
Copy link
Contributor

Je me demande si l'encart "Ce candidat n'a pas de CV ?" ne devrait pas être en d-print-none (si j'ai bien compris la doc)

@hellodeloo hellodeloo force-pushed the deloo/add-css-print-rules-and-print-button branch from 828a8d0 to 0a81b6b Compare September 4, 2024 08:35
@hellodeloo
Copy link
Contributor Author

Je me demande si l'encart "Ce candidat n'a pas de CV ?" ne devrait pas être en d-print-none (si j'ai bien compris la doc)

L'idée était de mettre en place des règles communes d'impression puis d'ajouter un bouton d'impression sur la page des candidatures. Pour l'instant, il n'était pas prévu de faire un page de détail de candidature complètement adaptée à l'impression.

Par contre, si le bouton est beaucoup utilisé, il faudrait surement imaginer une version de la page spécifique à l'impression (comme printable_approval.html). Du coup, je viens d'ajouter du matomo sur le bouton, histoire de pouvoir traquer ça.

@francoisfreitag
Copy link
Member

Je ne suis pas au clair sur le cas d’utilisation. On ajoute un bouton pour permettre d’imprimer, mais on se dit que l’impression de la page est supportée à minima comme sur le reste du site ?

C’est pour mesurer s’il y a de la demande pour ce genre de fonctionnalité ?

@hellodeloo
Copy link
Contributor Author

Je ne suis pas au clair sur le cas d’utilisation. On ajoute un bouton pour permettre d’imprimer, mais on se dit que l’impression de la page est supportée à minima comme sur le reste du site ?

C’est pour mesurer s’il y a de la demande pour ce genre de fonctionnalité ?

J'imagine que oui, sinon une maquette d'impression de la page aurait été designée afin d'être ensuite intégrée. Et dans ce cas, on se serait surement rendu compte que le plus simple était de faire une page spécifique afin d'avoir toutes les libertés de mise en page pour l'impression (comme printable_approval.html)

Copy link
Member

@francoisfreitag francoisfreitag left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

C’est déjà une belle amélioration 👍

@hellodeloo hellodeloo force-pushed the deloo/add-css-print-rules-and-print-button branch from 0a81b6b to 962c7b7 Compare September 4, 2024 13:48
@hellodeloo hellodeloo added this pull request to the merge queue Sep 9, 2024
Merged via the queue into master with commit 7f7a799 Sep 9, 2024
@hellodeloo hellodeloo deleted the deloo/add-css-print-rules-and-print-button branch September 9, 2024 08:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC ajouté Ajouté dans le changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants