Skip to content
This repository has been archived by the owner on Dec 19, 2024. It is now read-only.

IE11 support for "new URL()" is gone... #74

Closed
1 of 8 tasks
Ado77 opened this issue Mar 24, 2017 · 5 comments
Closed
1 of 8 tasks

IE11 support for "new URL()" is gone... #74

Ado77 opened this issue Mar 24, 2017 · 5 comments
Labels

Comments

@Ado77
Copy link

Ado77 commented Mar 24, 2017

Description

It looks like support for new URL() is gone from IE11

https://developer.mozilla.org/en/docs/Web/API/URL#Browser_compatibility

Expected outcome

Navigation from anchor tags to not reload the entire page.

Actual outcome

Navigation is causing the entire page to reload.

Live Demo

Steps to reproduce

polymer init a PSK 2.0 and run it in ie 11
click navigation links on the left

Dev Note :)

It's in the _getSameOriginLinkHref function.

Browsers Affected

  • IE 11
  • Chrome
  • Firefox
  • Safari 9
  • Safari 8
  • Safari 7
  • Edge
  • IE 10
@Ado77 Ado77 changed the title IE11 support for new URL() is gone... IE11 support for "new URL()" is gone... Mar 24, 2017
@Ado77
Copy link
Author

Ado77 commented Mar 24, 2017

@e111077 Not sure if you are the right person for this but seems like a big problem if you want to do routing...

@e111077
Copy link
Contributor

e111077 commented Mar 24, 2017

Heya, I am the right person. We have received many complaints and have ran into huge issues when including our own polyfills in elements which is why the url polyfill was moved to the dev dependencies. Going on from 2.0 to further we are allowing developers to choose and use their own polyfills. If you want to use the same polyfill that we do:

bower install -S webcomponents/URL#^0.5.7

and then before you import app-route or iron-location you should include the polyfill. Here is an example on how we do this in the tests.

@e111077
Copy link
Contributor

e111077 commented Mar 24, 2017

I recommend leaving an issue to include the polyfill on the PSK2 repo

@Ado77
Copy link
Author

Ado77 commented Mar 24, 2017

Ok, thanks for the quick reply 👍 . Maybe put it on the README of this repo? I suspect it will trip up a lot of people... I couldn't find any information on this...

@e111077
Copy link
Contributor

e111077 commented Mar 24, 2017

That is a really good point. We've been putting breaking changes on the PRs. I've updated the PR for now.

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

No branches or pull requests

2 participants