UX/UI : Privilégier l’ouverture d’onglet [ GEN-1872 ] #4450
Conversation
calummackervoy
added
the
no-changelog
| @@ -26,6 +26,6 @@ <h3>{{ company.display_name }}</h3> | |||
| {% include "companies/includes/_company_details.html" with company=company only %} | |||
| {% endif %} | |||
| </ul> | |||
| <a href="{% url 'companies_views:card' siae_id=company.pk %}{% if back_url|default:"" %}?back_url={{ back_url|urlencode }}{% endif %}" class="btn btn-secondary btn-block mt-4">Voir la fiche de l'entreprise</a> | |||
| <a href="{% url 'companies_views:card' siae_id=company.pk %}{% if back_url|default:"" %}?back_url={{ back_url|urlencode }}{% endif %}" class="btn btn-secondary btn-block mt-4" target="_blank">Voir la fiche de l'entreprise</a> | |||
There was a problem hiding this comment.
Cela implique que le changement est mis sur plusieurs pages qui n'étaient pas spécifié sur le ticket, mais il me semble qu'on serait content de repliquer ce comportement sur tous ces pages ?
There was a problem hiding this comment.
Est-ce qu’on souhaite conserver le back_url ? Si on ouvre dans un nouvel onglet, on ne veut probablement pas que l’utilisateur revienne à la dernière étape d’un formulaire (qui a pu être soumis entre temps) ?
There was a problem hiding this comment.
Attention : il ne faut pas le faire partout !
Dans le début du parcours, on veut pouvoir passer d'un élément à un autre (l'entreprise, un des postes, l'entreprise à nouveau) sans quitter le parcours
There was a problem hiding this comment.
Tu peux probablement passer un paramètre supplémentaire au template open_in_tab et utiliser {% if open_in_tab|default:False %} dans ce template.
Cela te permettra de choisir quand est-ce qu'il faut l'ouvrir dans un nouvel onglet.
Sachant que dans ce cas là, il faudra sans doute ne pas mettre le back url.
There was a problem hiding this comment.
Modifié ! J'ai ajouté dans le description du PR les endroits où les liens vont ouvrir un onglet. Il y a toujours plus des endroits que spécifié sur le ticket, mais si j'ai bien suivis la motivation ces endroits sont impliqués par les mêmes besoins
There was a problem hiding this comment.
C'est validé ! Si tout va bien pour vous on pourrait le merger ? :)
calummackervoy
force-pushed
the
calum/priviliger-ouverture-onglets
branch
from
4137e9e to
24ef52b
Compare
calummackervoy
added
modifié
and removed
no-changelog
calummackervoy
force-pushed
the
calum/priviliger-ouverture-onglets
branch
from
bd629dd to
190de69
Compare
tonial
added
the
1-recette-jetable
|
Je crée une recette jetable pour tester |
calummackervoy
added
1-recette-jetable
fix(_company_info.html): open_in_tab controls link behaviour
calummackervoy
force-pushed
the
calum/priviliger-ouverture-onglets
branch
from
190de69 to
0c4e0b4
Compare
🤔 Pourquoi ?
Éviter que l’utilisateur ne sorte du parcours sur quelques composants clés (attention que j'ai proposé des pages non-définit sur le ticket):
job_seeker_info.html: liens vers son CV sur les pages des applications et approbationsprocess_internal_transfer.html,transfer_job_application.html: "voir la fiche de l'enterprise" sur la confirmation de transfert du candidataccept_section.html: confirmation de l'acceptation du candidatedit_siae_preview.html: sous la page "modifier les coordonnées de votre structure"application/base.html: un lien vers l'information de l'enterprise concernée et inclus sur plusieurs gabarits à la soumission de candidature🚨 À vérifier