move Agent select to left menu planning
Created by: adipasquale
Comportement
Lorsqu'on est sur une route hors-planning, par ex un users#show
:
- l'agent selectionné dans le select est toujours l'agent courant
- si on clique sur un des liens Agenda, Plages d'ouvertures, Absences, on arrive sur la vue de l'agent courant
- si on sélectionne un autre Agent dans le select, on arrive sur l'Agenda de cet agent
Lorsqu'on est sur une route dans le planning :
- L'agent sélectionné à gauche reste persisté entre toutes les routes au sein de ce sous-groupe du planning
- Si je suis sur une vue
index
(par exabsences#index
) ounew
(par explages_ouvertures#new
) et que je change d'agent a gauche, je reviens bien vers la meme vue courante (index
ounew
) avec le nouvel agent selectionné. Ce n'est pas vrai sur les routes edit car l'id de la resource ne serait plus valide - Si je suis sur une vue
edit
ou autre et que je change d'agent, j'arrive sur l'agenda de cet agent.
Détails :
- je détourne un peu l'usage de
content_for
pour passer le nom du path helper a utiliser dans le select de changement d'agent.content_for
est plutot prevu pour passer du contenu, pas des symboles ou des fonctions. Je n'ai pas trouvé de solution plus élégante. J'ai envisagé un moment de changer leagent_id
du path courant en JS mais c'est encore pire je pense, et plus risqué - la au moins on sait ce qui va se passer dans tous les cas. - j'en profite pour supprimer une route inutile sur l'agenda qui passait l'agent id en query param plutot que dans le path
- j'ai retouché le CSS du menu latéral pour bien aligner les chevrons à droite
TODO
-
il faudrait s'occuper du sujet du lien Liste des RDVs et le breadcrumb associé avant, la je l'ai purement supprimé mais ca casse le breadcrumb -
pour simplifier la transition pour les agents habitués a l'interface actuelle, est-ce que pendant une semaine ou deux on affiche un message dans le header bar ou se trouvait jusqu'ici le select avec un message "agent sélectionnable à gauche" -
valider la vue responsive => fait dans l'autre PR qui revoit tout le menu de gauche, il faudra merger