Skip to content

Commit 445f6d6

Browse files
committed
Handle all browsers without pagereveal
older chrome might also be an issue. pagereveal is fairly new so we treat inspecting while streaming as progressive enhancement.
1 parent de21c75 commit 445f6d6

1 file changed

Lines changed: 3 additions & 6 deletions

File tree

  • packages/react-devtools-extensions/src/contentScripts

packages/react-devtools-extensions/src/contentScripts/proxy.js

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -32,12 +32,9 @@ function injectProxy({target}: {target: any}) {
3232
}
3333
}
3434

35-
if (__IS_FIREFOX__) {
36-
// Firefox does not implement `pagereveal' yet: https://developer.mozilla.org/en-US/docs/Web/API/Window/pagereveal_event#browser_compatibility
37-
window.addEventListener('pageshow', injectProxy);
38-
} else {
39-
window.addEventListener('pagereveal', injectProxy);
40-
}
35+
window.addEventListener('pagereveal', injectProxy);
36+
// For backwards compat with browsers not implementing `pagereveal` which is a fairly new event.
37+
window.addEventListener('pageshow', injectProxy);
4138

4239
window.addEventListener('pagehide', function ({target}) {
4340
if (target !== window.document) {

0 commit comments

Comments
 (0)