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

florian_briand · 2024-08-18T23:34:58+02:00

florian_briand commented

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

Détails

Cette PR prend le parti de mettre à jour les éléments de menu dans les navbars (mobile & desktop), en injectant les nouvelles valeurs dans la page requêtée.
Par exemple, quand on appelle la page "CPS", elle contient son contenu, mais également des blocs à mettre à jour à d'autres endroits (grâce au paramètre hx-swap-oob)

Une autre possibilité aurait pu être de gérer ça totalement côté client, en JS, mais je souhaitais explorer l'approche htmx.

Pourquoi ?

Les classes permettant de mettre en avant l'élément "courant" de la navbar étaient configurées "en dur" pour la page d'accueil. Des modifications étaient donc nécessaires pour rendre ça dynamique.

Documentation

Documentation du paramètre HTMx hx-swap-oob

### Détails Cette PR prend le parti de mettre à jour les éléments de menu dans les navbars (mobile & desktop), en injectant les nouvelles valeurs dans la page requêtée. Par exemple, quand on appelle la page "CPS", elle contient son contenu, mais également des blocs à mettre à jour à d'autres endroits (grâce au paramètre `hx-swap-oob`) Une autre possibilité aurait pu être de gérer ça totalement côté client, en JS, mais je souhaitais explorer l'approche htmx. ### Pourquoi ? Les classes permettant de mettre en avant l'élément "courant" de la navbar étaient configurées "en dur" pour la page d'accueil. Des modifications étaient donc nécessaires pour rendre ça dynamique. ### Documentation Documentation du paramètre HTMx [`hx-swap-oob`](https://htmx.org/attributes/hx-swap-oob/)

florian_briand added the

bug

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

florian_briand added 1 commit 2024-08-18 23:34:59 +02:00

fix: inject menu items with correct "current" value 09f3e4cc39

florian_briand self-assigned this 2024-08-18 23:35:09 +02:00

theo commented

2024-08-20 20:21:19 +02:00

Pour faire le changement de tabs, qu'en penserait tu de cet approche ?
https://htmx.org/examples/tabs-hateoas/

A la place de faire un hx-swap-oob, chaque tab a son propre contenu, mais seulement le contenu du tab selectione est envoye, la tab etant marque comme selectionnee (avec les autres tabs sans contenu aussi)

Tu en penses quoi ?

Pour faire le changement de tabs, qu'en penserait tu de cet approche ? https://htmx.org/examples/tabs-hateoas/ A la place de faire un hx-swap-oob, chaque tab a son propre contenu, mais seulement le contenu du tab selectione est envoye, la tab etant marque comme selectionnee (avec les autres tabs sans contenu aussi) Tu en penses quoi ?

florian_briand commented

2024-08-20 20:28:01 +02:00

Pour faire le changement de tabs, qu'en penserait tu de cet approche ?
https://htmx.org/examples/tabs-hateoas/

A la place de faire un hx-swap-oob, chaque tab a son propre contenu, mais seulement le contenu du tab selectione est envoye, la tab etant marque comme selectionnee (avec les autres tabs sans contenu aussi)

Tu en penses quoi ?

Cette approche ne fonctionne pas dans le cas présent, car :

il y a deux endroits où la navbar doit être changée ;
ces deux endroits ne sont pas à proximité du contenu de l'onglet

Dans l'implémentation actuelle de notre interface, utiliser cette approche implique de render entièrement la page et de la remplacer quasi entièrement, ce qui n'est pas très intéressant.

> Pour faire le changement de tabs, qu'en penserait tu de cet approche ? > https://htmx.org/examples/tabs-hateoas/ > > A la place de faire un hx-swap-oob, chaque tab a son propre contenu, mais seulement le contenu du tab selectione est envoye, la tab etant marque comme selectionnee (avec les autres tabs sans contenu aussi) > > Tu en penses quoi ? Cette approche ne fonctionne pas dans le cas présent, car : - il y a deux endroits où la navbar doit être changée ; - ces deux endroits ne sont pas à proximité du contenu de l'onglet Dans l'implémentation actuelle de notre interface, utiliser cette approche implique de render entièrement la page et de la remplacer quasi entièrement, ce qui n'est pas très intéressant.

florian_briand changed title from ~~Fix #46 : mise à jour de la navbar pour prendre en compte la "page courante"~~ to WIP: Fix #46 : mise à jour de la navbar pour prendre en compte la "page courante"

2024-08-23 13:29:49 +02:00

florian_briand changed title from ~~WIP: Fix #46 : mise à jour de la navbar pour prendre en compte la "page courante"~~ to Fix #46 : mise à jour de la navbar pour prendre en compte la "page courante"

2024-08-23 13:30:04 +02:00

florian_briand requested review from kosssi 2024-08-23 13:30:13 +02:00

florian_briand requested review from julien.misiak 2024-08-23 13:30:13 +02:00

florian_briand requested review from theo 2024-08-23 13:30:14 +02:00

~~florian_briand referenced this pull request 2024-08-23 20:16:33 +02:00~~

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

florian_briand commented

2024-08-23 20:29:11 +02:00

Cette PR est remplacée par #57

florian_briand closed this pull request

2024-08-23 20:29:11 +02:00

florian_briand referenced this pull request