Fix route params overwriting route properties

[marvinhagemeister]

When a query param was named the same as an existing key on the route like "path", we'd overwrite that.

Imagine that we are on /foo/bar as the url and our route looks like this:

<Route path="/foo/:id" ... />
// -> { path: "/foo/bar", query: {}, id: "bar" }

But when we name the param the same as an existing key we'd overwrite those:

<Route path="/:path/:query" ... />
// -> { path: "bar", query: "bar" }

By moving params to their own object we avoid that issue all together. This is also in line with how most other routers in the wild store params.

<Route path="/:path/:query" ... />
// -> { path: "/foo/bar", query: {}, params: { path: "foo", query: "bar" }}

[changeset-bot]

🦋 Changeset detected

Latest commit: 8e8690a

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
preact-iso	Major

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR