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

Fix URL redefined in polymer #97

Closed
wants to merge 1 commit into from
Closed

Fix URL redefined in polymer #97

wants to merge 1 commit into from

Conversation

jareguo
Copy link

@jareguo jareguo commented Jul 21, 2014

No description provided.

@eligrey
Copy link
Owner

eligrey commented Jul 21, 2014

Can you link me to a test case so I can try to reproduce your issue in Chrome?

@eligrey
Copy link
Owner

eligrey commented Jul 22, 2014

I'm unable to reproduce your issue, so without any further feedback I will be closing this issue.

@eligrey eligrey closed this Jul 22, 2014
@eligrey
Copy link
Owner

eligrey commented Jul 22, 2014

You say that Polymer defines a URL global--you should file a bug with them as URL should never be redefined unless the new value is also compatible with all DOM URL APIs.

@ghost
Copy link

ghost commented Jul 22, 2014

"Polymer is a big deal, we can't just dodge it." thought the mouse.

"If Polymer is installed is there another way to accomplish saveAs()? Perhaps they have provided something?" inquired the wizard.

@eligrey
Copy link
Owner

eligrey commented Jul 22, 2014

@ropidon No luck; Polymer breaks an essential interface required by FileSaver.js. Please file a bug with Polymer and link it here if you want this resolved.

@jareguo jareguo changed the title Fix view.URL.createObjectURL undefined in Chrome 31 Fix URL redefined in polymer Jul 23, 2014
@eligrey eligrey reopened this Jul 24, 2014
@eligrey
Copy link
Owner

eligrey commented Jul 24, 2014

webcomponents/URL#4 is going to address this.

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

Successfully merging this pull request may close these issues.

2 participants