Hierarchical navigation menu #4

Merged
lx merged 3 commits from hier-nav-menu into main 1 year ago
lx commented 1 year ago
Owner

Fix #2

Fix #2
lx added 1 commit 1 year ago
lx commented 1 year ago
Poster
Owner

Bon le style est pas parfait, il faudrait retoucher le CSS, mais ça je sais pas comment ça marche :p

Bon le style est pas parfait, il faudrait retoucher le CSS, mais ça je sais pas comment ça marche :p
lx force-pushed hier-nav-menu from 83941aea7c to 0900f2910e 1 year ago
quentin commented 1 year ago
Owner

C'est ok pour moi. Perso je préfèrerais garder un menu simple avec un fil d'ariane en plus en haut, pour justement pas trop densifier la navigation.
Mais après c'est vraiment une histoire de gout perso et j'ai testé ta PR et ça marche donc ok pour merge pour moi, merci pour le travail :-)

C'est ok pour moi. Perso je préfèrerais garder un menu simple avec un fil d'ariane en plus en haut, pour justement pas trop densifier la navigation. Mais après c'est vraiment une histoire de gout perso et j'ai testé ta PR et ça marche donc ok pour merge pour moi, merci pour le travail :-)
lx added 1 commit 1 year ago
lx added 1 commit 1 year ago
lx commented 1 year ago
Poster
Owner

Je propose un petit fix pour alléger le menu de navigation: ne pas commencer au niveau 0 mais au niveau 1, i.e. quand on est par exemple dans la partie "Opérations", on ne voit pas les autres parties, le menu prend pour racine la section "Opérations".

Je propose un petit fix pour alléger le menu de navigation: ne pas commencer au niveau 0 mais au niveau 1, i.e. quand on est par exemple dans la partie "Opérations", on ne voit pas les autres parties, le menu prend pour racine la section "Opérations".
lx merged commit e69ee6ab7c into main 1 year ago
The pull request has been merged as e69ee6ab7c.
You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.
git checkout -b hier-nav-menu main
git pull origin hier-nav-menu

Step 2:

Merge the changes and update on Forgejo.
git checkout main
git merge --no-ff hier-nav-menu
git push origin main
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: Deuxfleurs/guide.deuxfleurs.fr#4
Loading…
There is no content yet.