Skip to content

UX/UI: Corriger les boutons des cartes candidats#4951

Merged
hellodeloo merged 2 commits intomasterfrom
deloo/fix-list-job-applications-cta-cards-ui
Oct 30, 2024
Merged

UX/UI: Corriger les boutons des cartes candidats#4951
hellodeloo merged 2 commits intomasterfrom
deloo/fix-list-job-applications-cta-cards-ui

Conversation

@hellodeloo
Copy link
Copy Markdown
Contributor

@hellodeloo hellodeloo commented Oct 18, 2024

🤔 Pourquoi ?

Les boutons “Proposer un rdv”, “invitation envoyée”, “Réessayer un envoi” etc des box de fiches candidates sont mal mis en forme (espacement, alignement horizontal et fullwith en mobile)

Pour infos :

  • J'ai retiré les {% if job_application.pending_for_weeks >= job_application.WEEKS_BEFORE_CONSIDERED_OLD %} sur les classes car ils ne sont plus nécessaires avec la nouvelle construction du DOM/Flex
  • @leo-naeka Je me suis aussi permis de remplacer des boutons, qui me semblaient ne pas être le bon élément sémantique, par du texte simple
    capture 2024-10-18 à 15 48 21

🤔 J'aimerais bien tester l'affichage sur la recette jetable, mais je ne sais pas comment faire apparaitre les boutons pour proposer un rdv

💻 Captures d'écran

Avant
capture 2024-10-18 à 15 38 44
capture 2024-10-18 à 15 38 56

Apres
capture 2024-10-18 à 16 00 03
capture 2024-10-18 à 16 00 56

@hellodeloo hellodeloo self-assigned this Oct 18, 2024
@hellodeloo hellodeloo added the modifié Modifié dans le changelog. label Oct 18, 2024
@hellodeloo hellodeloo changed the title fix: list jobs applications cards cta ui UX/UI: Corriger les boutons des cartes candidats Oct 18, 2024
@notion-workspace
Copy link
Copy Markdown

@hellodeloo hellodeloo force-pushed the deloo/fix-list-job-applications-cta-cards-ui branch from c9e176f to c896803 Compare October 18, 2024 12:31
@hellodeloo hellodeloo added the 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC label Oct 18, 2024
@github-actions
Copy link
Copy Markdown

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

@hellodeloo hellodeloo force-pushed the deloo/fix-list-job-applications-cta-cards-ui branch from c896803 to cda67cb Compare October 18, 2024 14:28
@leo-naeka
Copy link
Copy Markdown
Contributor

@hellodeloo pour que les boutons apparaissent il faut "activer" RDVI depuis l'admin pour une entreprise.

L'ID RDVS de démo est 654.
Je l'ai ajouté à l'EI : https://c1-review-deloo-fix-list-job-applications-cta-cards-ui.cleverapps.io/admin/companies/company/3850/change/
Attention aux téléphones/emails des candidats, RDVI envoie vraiment des mails/SMS.

Je n'ai pas MAJ les fixtures car ce faisant l'intégration n'est pas complète : il faut relayer les webhooks sur les recettes jetables, et à ce stade, ça reste manuel (fait dans le cas présent).

@hellodeloo hellodeloo force-pushed the deloo/fix-list-job-applications-cta-cards-ui branch 2 times, most recently from f2405ed to 3e82480 Compare October 28, 2024 10:34
@hellodeloo hellodeloo force-pushed the deloo/fix-list-job-applications-cta-cards-ui branch from 3e82480 to a005cec Compare October 28, 2024 13:00
@hellodeloo hellodeloo force-pushed the deloo/fix-list-job-applications-cta-cards-ui branch from a005cec to ed3b2d4 Compare October 29, 2024 17:44
@hellodeloo hellodeloo added this pull request to the merge queue Oct 30, 2024
Merged via the queue into master with commit 978311d Oct 30, 2024
@hellodeloo hellodeloo deleted the deloo/fix-list-job-applications-cta-cards-ui branch October 30, 2024 10:13
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 modifié Modifié dans le changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants