Skip to content
This repository was archived by the owner on Mar 20, 2024. It is now read-only.

fix: Illegal invocation calling window.getComputedStyle #146

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jsaguet
Copy link

@jsaguet jsaguet commented Nov 21, 2021

  • What modules are related to this pull-request
  • server side
  • client side
  • inline
  • build process
  • docs
  • tests
  • What kind of change does this PR introduce?

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.

  • What is the current behavior? (You can also link to an open issue here)

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:

@google-cla google-cla bot added the cla: no label Nov 21, 2021
@pavelrazuvalau
Copy link

pavelrazuvalau commented Nov 30, 2021

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!

@linusschwab
Copy link

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.

Copy link

@DavidBowdoin DavidBowdoin left a 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

Copy link

@rezonant rezonant left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested, fix works

@rezonant
Copy link

@jsaguet Could you sign the CLA?

@jsaguet
Copy link
Author

jsaguet commented Apr 23, 2022

I already did

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants