[app] Gérer l'actualisation du statut current de la navbar #46

New Issue

florian_briand · 2024-08-06T22:03:08+02:00

florian_briand commented

2024-08-06 22:03:08 +02:00

Objectif

Quand on clic sur un élément de la navbar, on souhaite, en plus de mettre à jour le titre et le contenu de la page, mettre à jour la navbar pour que l'élément ayant current=true soit l'élément cliqué.

Suggestion d'approche

On peut, peut-être, faire ça élégamment avec hx-swap-oob, en incluant dans la réponse faite, la nouvelle version du menu. Ça demande juste de trouver un moyen d'ajouter le menu à la réponse, sans dupliquer de code.

On pourrait aussi le gérer purement en JS ? Ou trigger un rechargement du menu lors d'un clic sur un de ses élément ?

Complément

Une fois ce comportement implémenté, on peut mettre en place un mécanisme pour éviter que des clics successif sur l'élément qui est current=true déclenche, à chaque fois, une nouvelle requête.
(problème initialement évoqué sur #42 (comment))

# Objectif Quand on clic sur un élément de la navbar, on souhaite, en plus de mettre à jour le titre et le contenu de la page, mettre à jour la navbar pour que l'élément ayant `current=true` soit l'élément cliqué. # Suggestion d'approche On peut, peut-être, faire ça élégamment avec `hx-swap-oob`, en incluant dans la réponse faite, la nouvelle version du menu. Ça demande juste de trouver un moyen d'ajouter le menu à la réponse, sans dupliquer de code. On pourrait aussi le gérer purement en JS ? Ou trigger un rechargement du menu lors d'un clic sur un de ses élément ? # Complément Une fois ce comportement implémenté, on peut mettre en place un mécanisme pour éviter que des clics successif sur l'élément qui est `current=true` déclenche, à chaque fois, une nouvelle requête. (problème initialement évoqué sur https://forge.p4pillon.org/P4Pillon/Krys4lide/pulls/42#issuecomment-426)

florian_briand added this to the 0 - POC project 2024-08-06 22:03:24 +02:00

florian_briand added this to the 0 - POC milestone 2024-08-06 22:03:26 +02:00

florian_briand added the

bug

label 2024-08-06 22:03:32 +02:00

theo commented

2024-08-08 16:42:02 +02:00

Ressource HTMX qui propose peut etre une solution: https://htmx.org/examples/tabs-hateoas/

Ca correspond ?

Ressource HTMX qui propose peut etre une solution: https://htmx.org/examples/tabs-hateoas/ Ca correspond ?

florian_briand commented

2024-08-08 17:53:54 +02:00

Ressource HTMX qui propose peut etre une solution: https://htmx.org/examples/tabs-hateoas/

Ca correspond ?

Trop bien, bonne idée les exemples HTMX! Je vais creuser cet exemple, ils ont l'air de d'utiliser juste hx-get et rien d'autre, c'est surprenant (sans doute lié aux valeurs par défaut des paramètres hx-

> Ressource HTMX qui propose peut etre une solution: https://htmx.org/examples/tabs-hateoas/ > > Ca correspond ? Trop bien, bonne idée les exemples HTMX! Je vais creuser cet exemple, ils ont l'air de d'utiliser juste hx-get et rien d'autre, c'est surprenant (sans doute lié aux valeurs par défaut des paramètres hx-

florian_briand self-assigned this 2024-08-17 15:43:52 +02:00

florian_briand referenced a pull request that will close this issue

2024-08-18 23:34:58 +02:00

Fix #46 : mise à jour de la navbar pour prendre en compte la "page courante" #53

florian_briand referenced a pull request that will close this issue

2024-08-23 20:30:00 +02:00

Implémentation "HATEOAS" de l'interface pour HTMX et update des URLs qui fonctionne ! #57

florian_briand closed this issue

2024-08-30 18:25:54 +02:00

florian_briand referenced this issue from a commit

2024-08-30 18:25:54 +02:00

Merge pull request 'Implémentation "HATEOAS" de l'interface pour HTMX et update des URLs qui fonctionne !' (#57) from feat/54_update_url_on_navbar_navigation into main