pass-culture / pass-culture-institutional

pass Culture institutional web site
Mozilla Public License 2.0
3 stars 3 forks source link

fix: active les items dans le menu et le megamenu selon la route #379

Closed lheneman-pass closed 3 months ago

lheneman-pass commented 3 months ago

Description

Cette pull request règle le problème des items actifs du menu (header et megamenu). Lorsque l'on navigue sur une page fille, l'item du menu de la page parent doit être actif, ainsi que l'item du megamenu de la page fille.

Changement

Deux fonctions findCollectionIdByPath et. findInMenu ont été écrites dans Header.tsx pour faire matcher la route active et l'item du menu. Une classe active a été ajoutée dans sur les link dans MegaMenu.tsx. Une fonction utile qui compare deux chaînes de string a été écrite (isStringAreEquals) Issue relative Notion ici.

Note

Il est à noter que si coté back des espaces vides sont laissées en fin d'url, ceux-ci remontent en front. Un "trim" a donc été ajouté dans le traitement des url dans findCollectionIdByPath.

sonarcloud[bot] commented 3 months ago

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
87.2% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud