-
Notifications
You must be signed in to change notification settings - Fork 48
fix: Illegal invocation calling window.getComputedStyle #146
base: master
Are you sure you want to change the base?
Conversation
d922786 to
77365c9
Compare
|
Hello @CaerusKaru @jsaguet! Do you have any updates about the PR? We're trying to upgrade Angular 12 to the latest minor version. Unfortunately, we're not able to downgrade Angular to 12.1.4 because we use @angular-builders/custom-webpack which uses the latest minor version of Angular by default and the issue is blocking the Angular upgrade. Thank you! |
|
Hi @jsaguet Thanks a lot for the fix! Are there any updates on this? Looks like the missing CLA might prevent this PR from being merged. |
DavidBowdoin
left a comment
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.
This fix works for me
rezonant
left a comment
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.
Tested, fix works
|
@jsaguet Could you sign the CLA? |
|
I already did |
This is a fix for #135
I've tested this change in an application of my own that was affected by this error and it seems to be working properly this way.
Illegal invocation error is thrown as described in #135
Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
no
Other information: