Créneaux type : améliorations cosmétiques & d'url

[raphodn]

Modifications apportées :

• ajout d'une modal de confirmation lors de la suppression d'un créneau type (le bouton est parfois confondu avec la suppression d'un poste)
• bougé le bouton de suppression d'un poste de créneau type dans le collapsible-body
• refonte du mapping des URL pour être plus 'CRUD'

route	Avant	Après	commentaire
period_edit	/period/edit/{id}	/period/{id}/edit
add_position_to_period	/period/{id}/add_position/	/period/{id}/position/add	dans l'absolu ca devrait être /period/{id}/position ^^
remove_position_from_period	/period/{period}/remove_position/{position}	period/{id}/position/{position}
book_position_from_period	/period/book/{id}	/period/{id}/position/{position}/book
free_position_from_period	/period/free/{id}	/period/{id}/position/{position}/free
period_delete	/period/period/{id}	/period/{id}

[petitalb]

Tu utilises une modal pour faire la confirmation, est-ce qu'on ne devrait pas essayer d'uniformiser la façon de faire avec les autres vues ?

Ex : https://github.com/elefan-grenoble/gestion-compte/blob/master/app/Resources/views/admin/booking/_partial/bucket_modal.html.twig#L90

[raphodn]

tu voudrais uniformiser dans le sens du onsubmit="return confirm() ou du modal-trigger ?

je penche pour le modal-trigger, on l'utilise pour la suppression de compte, mais aussi à d'autres endroits pour avoir des formulaires dans la modale, ca permet de mettre d'avantage de contexte. Par exemple dans cette modale on pourrait lister en détails les impacts d'une suppression d'un créneau type, alors que dans une alert JS c'est très/trop minimaliste non ?

[raphodn]

@petitalb up :)
je pensais ensuite faire les modifs d'ajout de modal de confirmation dans bucket_modal (dans une autre PR)

[petitalb]

Est-ce qu'on veut vraiment modifier ça ? Je veux dire par là que c'est bien d'un point de vu UX de garder la même façon de faire pour la partie "semaine type" et la partie "créneau généré".

[raphodn]

Justement, en terme d'UX je trouve que ce fonctionnement ici (icone de croix rouge tout à droite + alerte JS) n'est pas homogène avec le reste de l'app (bouton Supprimer claire + situé en dessous des infos + modale de confirmation pour être sûr)