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

OC_EventSource does not support multiple streams on IE8 #15599

Closed
oparoz opened this issue Apr 14, 2015 · 7 comments
Closed

OC_EventSource does not support multiple streams on IE8 #15599

oparoz opened this issue Apr 14, 2015 · 7 comments

Comments

@oparoz
Copy link
Contributor

oparoz commented Apr 14, 2015

This is to make it clearer that the fallback implemented in OC_EventSource does not work, instead of just mentioning it in another ticket.

Afaict, EventSource is used in OC.update and Pictures (Gallery+ has switched to a different library).

The best thing to do from my pov is to switch to a 3rd party polyfill since that would be one less thing to maintain.

@PVince81
Copy link
Contributor

Hmm, but OC_EventSource is also used by the upgrade system when clicking "Start update" and I remember that it worked in the past with IE8.

So, possibly a regression.

@oparoz
Copy link
Contributor Author

oparoz commented Apr 14, 2015

I don't have access to IE8, but what could happen is that it works fine when one EventSource call is made at a time, but fails when there is more than one, like in the Pictures app.

@PVince81
Copy link
Contributor

You can easily get an IE8 VM here: https://www.modern.ie/en-us/virtualization-tools#downloads

@oparoz
Copy link
Contributor Author

oparoz commented Apr 14, 2015

Ah, thanks :)

@oparoz oparoz changed the title OC_EventSource is not compatible with IE OC_EventSource does not support multiple streams on IE8 Jul 23, 2015
@oparoz
Copy link
Contributor Author

oparoz commented Jul 23, 2015

Updated the title since it can be made to work with IE9-11 using black magic.
IE8 is still a no go though when using multiple streams, like in Pictures.

@oparoz oparoz added this to the 8.2-current milestone Aug 13, 2015
@ghost ghost modified the milestones: backlog, 8.2-current Sep 25, 2015
@davitol
Copy link
Contributor

davitol commented Jan 7, 2016

As IE8 support is dropped in 9.0 I will close this issue.

@lock
Copy link

lock bot commented Aug 7, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Aug 7, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants