Add Support for @activeOnSubPaths to Link component
Per #308 this adds a named arg @activeOnSubPaths to allow links to be considered active when child routes are the current URL.
The new @activeOnSubPaths arg defaults to false to retain existing active behavior without a breaking change. In a major release it may make sense to default this to true.
In a major release it may make sense to default this to true.
I don't think the default will change, because sometimes you do want a differentiation between foo.index and foo.bar, which are foo/ and foo/bar respectively.
But this change is great! thanks for working on it! is the PR intended to still be draft?
is the PR intended to still be draft?
I was still checking that the docs rendered and giving some 👀 after coffee.
Gonna check the ternary.
I'm :/ on the query params state not having a test right now so gonna work on a test state for that to go along with the includeActiveQueryParams render test