UX/UI : Lorsque je consulte une candidature depuis l’onglet “candidat” d’une fiche candidat, je devrais revenir sur ce même onglet#5310
Merged
Conversation
dejafait
changed the title
dejafait
force-pushed
the
dejafait/fix_navbar
branch
2 times, most recently
from
eae5b00 to
87cce32
Compare
dejafait
force-pushed
the
dejafait/fix_navbar
branch
5 times, most recently
from
292920f to
5865cec
Compare
dejafait
changed the title
leo-naeka
reviewed
Jan 9, 2025
dejafait
force-pushed
the
dejafait/fix_navbar
branch
2 times, most recently
from
ae28990 to
da2b23f
Compare
Contributor
Author
|
@leo-naeka Nouvelle revue stp, j'ai utilisé un partiel comme discuté. |
leo-naeka
approved these changes
Jan 20, 2025
leo-naeka
left a comment
leo-naeka
left a comment
Contributor
There was a problem hiding this comment.
Tout bon :)
Nit : j'aurais éventuellement utilisé l'anglais pour la valeur de active_nav_tab : pour moi c'est du "code", pas du display… À toi de voir, pas bloquant
dejafait
force-pushed
the
dejafait/fix_navbar
branch
from
da2b23f to
27d9753
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
https://www.notion.so/plateforme-inclusion/Lorsque-je-consulte-une-candidature-depuis-l-onglet-candidat-d-une-fiche-candidat-je-devrais-reve-126e8fa5c35b80dd87bbdf105a7856cb
🤔 Pourquoi ?
Les deux onglets de la page Candidat (voir screenshots ci-dessous) n'ayant pas d'URL propre, si on clique sur un lien puis qu'on revient en arrière, on revient systématiquement sur l'onglet par défaut (informations générales) et pas sur l'onglet secondaire (Candidatures) même si on était dessus avant de cliquer sur le lien.
🍰 Comment ?
On donne à chaque onglet son URL propre en séparant l'unique route actuelle en deux routes. Notez qu'on conserve une unique vue mais qu'on sépare son gabarit en deux.
Chaque route pointe vers son gabarit directement dans le
urls.py. Pour cela j'ai d'abord tenté unpath("details/<uuid:public_id>", views.JobSeekerDetailView.as_view(), {"template_name": "job_seekers_views/details.html"}, name="details")mais je n'ai pas réussi à le faire fonctionner en aval. Au final j'ai utilisé un simplepath("details/<uuid:public_id>", views.JobSeekerDetailView.as_view(template_name="job_seekers_views/details.html"), name="details")qui fonctionne très bien.💻 Captures d'écran