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

Candidature: mettre la vue tableau par défaut pour les employeurs #5499

Merged
merged 3 commits into from
Jan 31, 2025

Conversation

xavfernandez
Copy link
Contributor

🤔 Pourquoi ?

https://www.notion.so/gip-inclusion/Par-d-faut-afficher-la-liste-des-candidatures-en-vue-tableau-18a5f321b60481ecae82ff24b2958297?pvs=4

🍰 Comment ?

Décrivez en quelques mots la solution retenue et mise en oeuvre, les difficultés ou problèmes rencontrés. Attirez l'attention sur les décisions d'architecture ou de conception importantes.

🚨 À vérifier

  • Mettre à jour le CHANGELOG_breaking_changes.md ?
  • Ajouter l'étiquette « Bug » ?

🏝️ Comment tester ?

Les instructions pour reproduire le problème, les profils de test, le parcours spécifique à utiliser, etc. Si vous disposez d'une recette jetable, mettre l'URL pour tester dans cette partie.

💻 Captures d'écran

@xavfernandez xavfernandez self-assigned this Jan 30, 2025
@xavfernandez xavfernandez force-pushed the xfernandez/make_table_view_default_for_siae branch from 1bd9d1b to 218a544 Compare January 30, 2025 10:35
@hellodeloo hellodeloo self-assigned this Jan 30, 2025
@hellodeloo hellodeloo added the 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC label Jan 30, 2025
Copy link

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

@hellodeloo hellodeloo force-pushed the xfernandez/make_table_view_default_for_siae branch from 7fc74c4 to de4ebf7 Compare January 30, 2025 16:17
@hellodeloo hellodeloo force-pushed the xfernandez/make_table_view_default_for_siae branch from de4ebf7 to c455c6b Compare January 30, 2025 16:30
Copy link
Contributor

@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.

Le bouton pour quitter le parcours est difficilement accessible. Sa place évolue à chaque étape et il est souvent chevauché par d’autres éléments.

image
image

Il faudrait aussi décocher la candidature cochée pour l’exemple lorsqu’on quitte la présentation non ?

@hellodeloo hellodeloo force-pushed the xfernandez/make_table_view_default_for_siae branch 3 times, most recently from 03e108c to 24ccd3f Compare January 31, 2025 08:41
@hellodeloo
Copy link
Contributor

@francoisfreitag Effectivement, le bouton de fermeture X n'a pas aimé la dernière maj BS5.3.3.
C'est fix et j'en ai aussi profité pour définir un width fixe à la "modale" (plutôt que du min-width et max-with) afin qu'elles aient toujours la même largeur et j'ai aussi diminué le line-height des titres de "modales".

@xavfernandez xavfernandez force-pushed the xfernandez/make_table_view_default_for_siae branch from 24ccd3f to fd011f9 Compare January 31, 2025 09:27
@xavfernandez xavfernandez force-pushed the xfernandez/make_table_view_default_for_siae branch from fd011f9 to 7bdd1a6 Compare January 31, 2025 10:17
@xavfernandez xavfernandez changed the title Candidature: mettre la vue tableau par défaut pour les employeurs & prescripteurs Candidature: mettre la vue tableau par défaut pour les employeurs Jan 31, 2025
@xavfernandez xavfernandez added this pull request to the merge queue Jan 31, 2025
Merged via the queue into master with commit 1d7b939 Jan 31, 2025
9 checks passed
@xavfernandez xavfernandez deleted the xfernandez/make_table_view_default_for_siae branch January 31, 2025 13:48
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é
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants