-
-
Notifications
You must be signed in to change notification settings - Fork 7
Back Button Does Not Work #16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
There's also the additional complexity of: What happens if the user presses the back button enough to where they're not on a page that uses We don't want to globally intercept all page changes. We only want to intercept page changes that can be resolved by an existing router node. |
That's handled with an effect cleanup in an invisible element that's rendered as part of the router.
I'm not really sure I understand this one. How would one know a priori whether a given location could be evaluated by a router node without checking? Besides, if no route matches, then the router just renders |
It's possible to record whether the URL was successfully evaluated as a data attribute within the invisible element. But if the effect cleanup is able to handle edge cases on it's own then this is likely not needed. |
Right now, if you click the browser's back button, the page does not update. We need to subscribe to changes in browser history and send them to the sever. I'm not really sure how best to achieve this in the long run. In the short run, I think this may require creating a hidden JS component that we display at the level of the router that does this.
The text was updated successfully, but these errors were encountered: