Skip to content
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

TypeError: Cannot assign to read only property 'target' of object '#<AnimationPlaybackEvent>' #123

Closed
web-padawan opened this issue Apr 13, 2017 · 27 comments

Comments

@web-padawan
Copy link

This error appears with latest 2.2.3 release.

TypeError: Cannot assign to read only property 'target' of object '#<AnimationPlaybackEvent>'

Not yet managed to create a small repro, will investigate.

@ztzven
Copy link

ztzven commented Apr 13, 2017

+1

Had to downgrade our version to 2.2.2 to get things working. Had the boss yelling down my ear at this one! (CI live site deploy for-the-fail 😭 )

@LarsDenBakker
Copy link

We have this error as well.

@gmcinalli
Copy link

Me too, I had to downgrade to 2.2.2.

@misterzorgy
Copy link

Yeah..downgraded to 2.2.2

@ghost
Copy link

ghost commented Apr 13, 2017

Just curious, what behavior is affected by this error? Currently I only see this error when clicking on paper-icon-buttons, and at least the functionality appears to be intact.

@web-padawan
Copy link
Author

Some elements using animations like paper-menu-button, paper-dropdown-menu, paper-dialog etc are broken and might not open at all.

@abdonrd
Copy link

abdonrd commented Apr 13, 2017

I have the same problem!

@ronnyroeller
Copy link

Same issue here

@DarkSideOfMo0n
Copy link

Up! 2.2.3 Breaks almost every Polymer app.

@samuelli
Copy link
Contributor

This has been fixed in #124. Please update to 2.2.4!

@emilio-martinez
Copy link

emilio-martinez commented Apr 14, 2017

@samuelli FYI, I'm not seeing 2.2.4 deployed in npm. I see it on the repo releases but not npm.

@samuelli
Copy link
Contributor

cc @alancutter: looks like NPM publishing didn't work. Can you re-publish to NPM? Thanks.

@alancutter
Copy link
Contributor

Apologies. The npm release should be at 2.2.4 now.

@emilio-martinez
Copy link

@alancutter thank you!

@lousley
Copy link

lousley commented Apr 18, 2017

this problem isn't fix with 2.2.4 --> tested with paper-dialog and i have the same error message!

@samuelli
Copy link
Contributor

@lousley: can you provide reproduction steps?

Can you run cat bower_components/web-animations-js/History.md | head -1 to confirm the version of web-animations that you have? I tried paper-dialog and don't see any errors.

@lousley
Copy link

lousley commented Apr 19, 2017

I found the problem in the library web-animations-next-lite.min.js.gz! Is it possible that you have forgot update the .gz file?

@alancutter
Copy link
Contributor

Uhoh, the .gz file isn't even supposed to be there. The release instructions should probably be updated to use *.min.js{,.map} instead of *.min.js* to avoid picking up more files than intended.

@alancutter alancutter reopened this Apr 20, 2017
@lousley
Copy link

lousley commented Apr 20, 2017

ah ok :) do you make a new version?

@alancutter
Copy link
Contributor

Thanks for reporting the mishap, removed the erroneous *.gz files in https://github.com/web-animations/web-animations-js/releases/tag/2.2.5.

@lousley
Copy link

lousley commented Apr 20, 2017

@alancutter thank you!

@BorntraegerMarc
Copy link

I am experiencing this issue again when I updated all my dependencies to Polymer 2.0. Has anybody also experienced this issue again in Polymer 2.0?

@BorntraegerMarc
Copy link

BTW: The warning in the console disappears when I add the import <link rel="import" href="assets/bower_components/neon-animation/web-animations.html"> manually but the scroll issue persists. So it breaks my app at the moment...

@samuelli
Copy link
Contributor

samuelli commented Jun 1, 2017

@BorntraegerMarc: can you remove your bower_components and run bower install again?

If it's still broken can you dump your bower.json and the contents of bower list?

@BorntraegerMarc
Copy link

@samuelli Thanks for the fast feedback. I did as you asked but the error still persists. Here is the dump of my dependencies:

"dependencies": {
    "app-layout": "PolymerElements/app-layout#2.0.0",
    "iron-flex-layout": "PolymerElements/iron-flex-layout#2.0.0",
    "paper-checkbox": "PolymerElements/paper-checkbox#2.0.0",
    "paper-styles": "PolymerElements/paper-styles#2.0.0",
    "paper-icon-button": "PolymerElements/paper-icon-button#2.0.0",
    "polymer": "Polymer/polymer#2.0.0",
    "iron-icons": "PolymerElements/iron-icons#2.0.0",
    "paper-button": "PolymerElements/paper-button#2.0.0",
    "paper-dialog": "PolymerElements/paper-dialog#2.0.0",
    "paper-input": "PolymerElements/paper-input#2.0.0",
    "paper-card": "PolymerElements/paper-card#2.0.0",
    "paper-toast": "PolymerElements/paper-toast#2.0.0",
    "paper-item": "PolymerElements/paper-item#2.0.0",
    "paper-ripple": "PolymerElements/paper-ripple#2.0.0",
    "paper-menu-button": "PolymerElements/paper-menu-button#2.0.0",
    "paper-dialog-scrollable": "PolymerElements/paper-dialog-scrollable#2.0.0",
    "vaadin-upload": "v2.0.0-alpha3",
    "iron-collapse": "PolymerElements/iron-collapse#2.0.0",
    "font-roboto": "PolymerElements/font-roboto-local#1.0.1",
    "iron-dropdown": "PolymerElements/iron-dropdown#2.0.0",
    "paper-spinner": "PolymerElements/paper-spinner#2.0.0",
    "paper-search": "Collaborne/paper-search#1.4.0",
    "paper-listbox": "PolymerElements/paper-listbox#2.0.0",
    "web-animations-js": "web-animations/web-animations-js#2.2.5",
    "paper-avatar": "Abe90/paper-avatar#1.0.3"
  }

I already had the issue with version 2.2.3 and Polymer 1. But then with the next released the issue got fixed and now after the upgrade I experience the error again.

@BorntraegerMarc
Copy link

I think this element PolymerElements/iron-dropdown#132 uses web-animations-js and caused it to fail, maybe?

@Serginho
Copy link

Serginho commented Jan 20, 2020

Same problem using [email protected] with "web-animations-js": "^2.3.2" when I use in the tsconfig.json "target": "es2015"

Uncaught TypeError: Cannot assign to read only property 'target' of object '[object AnimationPlaybackEvent]'
    at push../node_modules/web-animations-js/web-animations-next.min.js.b.Animation.<anonymous> (web-animations-next.min.js:15)
    at ZoneDelegate.invokeTask (zone-evergreen.js:391)
    at Object.onInvokeTask (core.js:39680)
    at ZoneDelegate.invokeTask (zone-evergreen.js:390)
    at Zone.runTask (zone-evergreen.js:168)
    at ZoneTask.invokeTask [as invoke] (zone-evergreen.js:465)
    at invokeTask (zone-evergreen.js:1603)
    at Animation.globalZoneAwareCallback (zone-evergreen.js:1629)

Any fix?

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

No branches or pull requests