Currently navigating back in the browser reloads the whole page, so we can get the correct filters & view again. This however only works on the first backward navigation, since the addEventListener for popstate is added in the click event.
So flow is as follows:
1. user clicks a few filters, url is updated accordingly, and window.addEventListener("popstate", function (e) {} is executed (since it is executed in the click handler).
2. User presses back, and browser reloads. No event listener is attached anymore.
3. User presses back again; view is not updated anymore since page doesn't reload anymore.
I propose we move the event listener outside of the click handler.
Needs work
1.0
Code
The change is currently missing an automated test that fails when run with the original code, and succeeds when the bug has been fixed.
Not all content is available!
It's likely this issue predates Contrib.social: some issue and comment data are missing.
Here's an update to the patch in #4 with the code that adds the event listener wrapped in a once() call so it doesn't get added multiple times (per the suggestion in #9).
Patch in #12 works nicely here to fix the problems with facet selections and the browser's Back button. Thanks!