Skip to content
This repository has been archived by the owner on Apr 20, 2018. It is now read-only.

IE8 support is broken #99

Open
jpotterm opened this issue Oct 20, 2015 · 2 comments
Open

IE8 support is broken #99

jpotterm opened this issue Oct 20, 2015 · 2 comments

Comments

@jpotterm
Copy link

First, IE8 doesn't support document.createEvent which is used in src/events/addeventlistenerpolyfill.js. The last working version is 6.0.0.

Second, requests via Rx.DOM.get never complete in IE8. Last working version for this is (ironically) 4.0.4. Here's a minimal example to demonstrate:

function logStart() {
    console.log('start');
}

function logEnd(data) {
    console.log('end');
}

var requests = Rx.Observable.just(1);
var responses = requests.flatMap(function() {
    return Rx.DOM.get('.');
});

requests.subscribe(logStart);
responses.subscribe(logEnd);

It should print

start
end

But in IE8 with any version of RxJS-DOM later than 4.0.4 it only prints start.

@mattpodwysocki
Copy link
Member

@jpotterm can you verify the fix with the latest RxJS and RxJS-DOM for events?

@jpotterm
Copy link
Author

I'm still having the problem with requests never returning (see the example code above). I can see the ajax request succeed in the inspector but the logEnd subscriber is never called.

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

No branches or pull requests

2 participants