Merge pull request #2555 from jowlo/feature/fix-multi-location-back-button

Fix browser back button with multiple dcc.Location components in layout

Author: T4rk1n

CHANGELOG.md

@@ -6,6 +6,7 @@ This project adheres to [Semantic Versioning](https://semver.org/).
 
 ## Fixed
 
+- [#2555](https://github.com/plotly/dash/pull/2555) Fix browser back button when removing one of multiple location components from layout, fix [#1312](https://github.com/plotly/dash/issues/1312)
 - [#2565](https://github.com/plotly/dash/pull/2565) Fix sorting for > 10 pages, fix [#2564](https://github.com/plotly/dash/issues/2564)
 
 ## [2.10.2] - 2023-05-31

components/dash-core-components/src/components/Location.react.js

@@ -112,7 +112,7 @@ export default class Location extends Component {
     }
 
     componentDidMount() {
-        window.onpopstate = this.onLocationChange;
+        window.addEventListener('popstate', this.onLocationChange);
 
         window.addEventListener(
             '_dashprivate_pushstate',
@@ -122,7 +122,7 @@ export default class Location extends Component {
     }
 
     componentWillUnmount() {
-        window.onpopstate = () => {};
+        window.removeEventListener('popstate', this.onLocationChange);
         window.removeEventListener(
             '_dashprivate_pushstate',
             this.onLocationChange