devwebfeed
devwebfeed copied to clipboard
Bug: Only works in Chrome
Feed does not load on other browsers then Chrome, tried both Edge & FF — app shell loads but feed keeps loading.
Any plans to make it cross browser? Thank you.
Yea, apologies for this. It works in Safari and Chrome, but not FF and Edge. FF is tricky b/c they now support ES modules but not dynamic import(). So the feature detect has gotten harder.
Edge should hit the rollup'd nomodule fallback script. What errors do you see from it?
Unfortunately it does not work on Edge. Test GIF below.

With the following exception from the dev tools

Which traces to

The full uilicious test case used for the GIF is as followed
// Lets load year 2017, as we can predict
// the expected load result
I.goTo("https://devwebfeed.appspot.com/?year=2017");
// Validate the header loads
I.see("dev-web firehose");
I.see("Older");
// With the Dec 28, 2017 result
I.see("What's New In DevTools (Chrome 63)");
So... on the plus side, with the same test script, I can confirm its working on safari.

So it appears Edge doesn't support URLSearchParams on URL. This page suggests it does. This page suggests it doesn't. :\
Hi Ebidel,
apologies for my delayed responses & inability to follow up & debug the issue yet.
URLSearchParams is supported, the 2nd link has to be updated, the first link seems accurate.
I will try to look at this soon or alternatively open and edge bug so someone on the team can look an help us here.
Many thanks!
Took a deeper dive into this, URLSearchParam was just rolled out in Edge 17; While it is not supported in Edge 16. So in that expect the documentation is correct (it does say 17 onwards).
The following are screenshots from crossbrowsertesting of IE16

Fo IE17, while URLSearchParams is supported, I get a pretty much untracable "SCRIPT5022: Syntax Error" pointing to line 1,1 of the html file


Sorry for the confusion : didn't occur to me that I would need to update my windows (so it does something haha)