-
Notifications
You must be signed in to change notification settings - Fork 47.5k
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
Reapply changes from #22631 #22645
Reapply changes from #22631 #22645
Conversation
@@ -56,7 +55,6 @@ function setup(hook, extensionId) { | |||
{ | |||
source: 'react-devtools-bridge', | |||
payload: {event, payload}, | |||
extensionId, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oooh so this was something we missed in #22638?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah, although it shouldn't really cause any issues
@@ -149,7 +150,7 @@ function createPanelIfReactLoaded() { | |||
|
|||
store = new Store(bridge, { | |||
isProfiling, | |||
supportsReloadAndProfile: isChrome, | |||
supportsReloadAndProfile: isChrome || isEdge, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
change
@Sarkar44 Not sure what your comment is referencing but this PR has already been merged. |
Accidentally reverted these in #22638
Audited the commits listed here to make sure none of those changes were missed: #22638 (comment)