-
Notifications
You must be signed in to change notification settings - Fork 24.8k
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
fix(devtools): prevent devTools to load when not text/html document #48021
Conversation
df6dffd
to
fb12509
Compare
a6d123b
to
b4b7b4a
Compare
On Firefox, Angular DevTools was breaking the XML display because of the script injected Based on contentType, we won't inject that script anymore Fixes angular#48017
b4b7b4a
to
cd70397
Compare
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.
Good catch! Thank you @JeanMeche
This PR was merged into the repository by commit f8f8928. |
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
…ngular#48021) On Firefox, Angular DevTools was breaking the XML display because of the script injected Based on contentType, we won't inject that script anymore Fixes angular#48017 PR Close angular#48021
The Angular DevTools extension breaks the XML formatting on Firefox.
This happens because a script tag is injected into the DOM even if it's a XML Document.
React DevTools had the same issue. They fixed it by checking the
content-type
before injecting the script.Fixes #48017
PR Type
What kind of change does this PR introduce?
Does this PR introduce a breaking change?