FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

feat: options configurables pour routerLinkActive by gabrielroberge · Pull Request #7 · Recursyve/ngx-navigation · GitHub

Repository navigation

feat: options configurables pour routerLinkActive - #7

Open
gabrielroberge wants to merge 1 commit into
masterfrom
fix/router-link-active-exact
Open

gabrielroberge wants to merge 1 commit into
masterfrom
fix/router-link-active-exact

Conversation

Copy link
Copy Markdown
Contributor

Résumé

  • Corrige le comportement de [exact]="true" : match strict sur le chemin uniquement, query params ignorés (contrairement au raccourci Angular { exact: true } qui exige aussi une correspondance exacte des query params).
  • Ajoute l'input [activeMatchOptions] pour un contrôle complet via IsActiveMatchOptions.
  • Exporte la constante NICE_NAVIGATION_EXACT_PATH_MATCH_OPTIONS pour réutilisation.
  • Ajoute des tests couvrant query params, override de exact, et options custom.

Problème

Après #6, [exact]="true" utilisait { exact: this.exact() }, le raccourci Angular qui mappe vers queryParams: 'exact'. Résultat : un item /dashboard n'était plus actif sur /dashboard?tab=overview.

De plus, un simple booléen exact ne suffit pas pour tous les cas — certaines apps ont besoin de définir finement comment un lien est considéré actif.

Solution

Trois niveaux de configuration, par priorité :

  1. [activeMatchOptions] — passe un IsActiveMatchOptions Angular complet
  2. [exact]="true" — raccourci : { paths: 'exact', queryParams: 'ignored', ... }
  3. Défaut — comportement Angular standard (match par préfixe)

Usage

<!-- Raccourci (cas VCL) -->
<nice-basic-navigation link="/dashboard" [exact]="true">

<!-- Contrôle complet -->
<nice-basic-navigation
    link="/dashboard"
    [activeMatchOptions]="{ paths: 'exact', queryParams: 'exact', fragment: 'ignored', matrixParams: 'ignored' }"
>

Test plan

  • npm test -- --project=ngx-navigation --no-watch
  • ng build ngx-navigation
  • Vérifier dans VCL que /dashboard reste actif avec query params quand [exact]="true"

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant


Back | FazBrowse Home | New Git URL