Provides hooks that can be used to determine the active state of a route #12
Conversation
| ``` | ||
|
|
||
| --- | ||
| ### `useIsPartiallyCurrent` |
There was a problem hiding this comment.
Kom niet meteen op een betere naam, maar vind het stukje PartiallyCurrent wat lastig. Ik zou zelf gaan voor het woord match een dan misschien gaan voor useIsPartiallyMatched() Maar ja matched betekent in de context van een router ook weer iets anders.
There was a problem hiding this comment.
Ik heb ze van jou en jij hebt ze van React router. Dat volgen lijkt me prima
| function useIsCurrent(route, params = {}): boolean | ||
| ``` | ||
|
|
||
| Returns a boolean indicating whether the route and params match the route and params of the current location. Note that if you don't want to match the params you can use the following: |
There was a problem hiding this comment.
Het duurde voor mij even voor dat ik snapte waar dit over ging, ik zou zelf even wat extra voorbeelden toevoegen waar bij je laat zien waarom je dit wel of niet zo willen. Zeker het verschil tussen de twee hooks wordt voor mij niet helemaal duidelijk.
There was a problem hiding this comment.
"Je kijkt dus eigenlijk niet of dat de routemapRoute.toString() matched of zo, maar meer of dat het route object en de params objecten het zelfde zijn."
|
|
||
| ```js | ||
| const { route: currentRoute } = useLocationMatch() | ||
| const isCurrent = route === currentRoute |
| ### `useIsCurrent` | ||
|
|
||
| ```js | ||
| function useIsCurrent(route, params = {}): boolean |
There was a problem hiding this comment.
Leest alsof params optioneel is, maar dat is het dus eigenlijk niet?
There was a problem hiding this comment.
Sommige routes hebben geen params
Fixes #5