-
Notifications
You must be signed in to change notification settings - Fork 56
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
Document supported platforms in the README file #457
Comments
@paddybyers @mattheworiordan need your help on the TBD ^ |
When this was first implemented, we intended to support all desktop browsers with the exception of IE6 and earlier, and all versions of iOS Safari and Android's browser. I think we officially stopped supporting IE7 (which I think was the only browser that didn't support JSON natively). We also officially stopped supporting some transports (iframe and flash websockets) - but this didn't stop any browsers from working; it only meant that certain browsers would fall back to JSONP. @SimonWoolf has done most of the cross-browser work and fixed multiple issues over the years but I wasn't aware of specific problems that would cause any of the "supported" browsers (ie IE8+, Safari, Chrome, Firefox, Opera, Android) to fail. Then we have unsupported features. For example, we use local storage to persist connection state for So then the question is which ones do we test - and if we find problems, how much effort do we put into fixing them? If a browser (eg IE8) shows a large number of test failures, and we think the IE8 population is too small to worry about, then we can list IE8 as unsupported. Otherwise, we could document known issues for certain browsers if those issues affect only certain use-cases (eg "crypto is unsupported on IE8"). So TBH I think to advertise a formal supported browsers list we need more concrete data so we know where the issues are, and we can decide which are worth fixing. |
@paddybyers Cool. Let's resume this when I finish #452 |
As per call today with @paddybyers we'll document supported platforms in the README file. |
@funkyboy don't forget to update the |
Resolved in #498 |
The text was updated successfully, but these errors were encountered: