Unexpected breaking change in URL encoding.

# Expected
1.2.1: https://codesandbox.io/s/reach-router-starter-v1-forked-vsw3w?file=/src/index.js

after calling `navigate(encodeURIComponent("foo$bar"));`, I was brought to `/foo%24bar`

# Actual

1.3.4: https://codesandbox.io/s/reach-router-starter-v1-forked-1fyh2?file=/src/index.js

after calling `navigate(encodeURIComponent("foo$bar"));`, I was brought to `/foo%2524bar`

---

After trying several versions, this issue was introduced in 1.3.3.

The reproduction uses navigate(encodeURIComponent(...)) in the linked /src/index.js example; start by comparing behavior between versions 1.3.2 and 1.3.3 and inspect the URL-encoding path. Done means the input reaches /foo%24bar rather than /foo%2524bar, with a regression test if the project’s test location is identified.

Written by the indexing model from the issue text.