-
Notifications
You must be signed in to change notification settings - Fork 171
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
Browser Support #414
Comments
Ideally we should align that with our server support matrix, oe say something like "same as server except IE". Otherwise every app has separate compatibility and it will be just confusing. |
Agreed! 👍 |
Both IE11 and Safari (7/8/9) prevent us from using any ES6 features. There would be the option of babel or some other cross-compiler, but in my opinion we as a community project can allow ourselves stricter requirements than the server project. Also I don't believe that our code currently properly supports either old Safari or IE and I also don't think that it should have any priority to change this. |
Ok, sounds fine. cc @karlitschek @MorrisJobke Also it might make sense to align this with other community apps like Notes, Calendar @georgehrke, News @BernhardPosselt, Tasks @raimund-schluessler and probably Mail too @ChristophWurst @Gomez. |
I'd just leave that up to the individual developers. Everyone uses different platforms (Mac, Linux, Windows, Android, iOS) and it's hard to test them (especially Safari which requires a Mac). Apart from that I don't really want to do work on Safari and IE related issues in my free time and I don't want to fiddle around with Firefox ESR which would require me to use a VM. Choose whatever platform you feel you can/are paid to support. PS: my current active support baseline for News is latest Chrome and Firefox (but I don't do any real work on it anymore), App Store requires latest Firefox, Chrome, Safari or Edge. |
There is no official support for Opera at the calendar right now. |
@BernhardPosselt I have assumed that Firefox ESR is pretty much just a normal old version of Firefox (at least in regards of web feature support). Is that assumption wrong? |
Right that's why installing it in parallel is shitty :) |
@georgehrke IE11 has issues, as reported by one customer today. Ping me for details if you need more info. Original report:
|
@mathiasconradt Georg is talking about calendar. |
I would like to clarify in the README which browser versions we actively support.
This doesn't mean that our app only runs on these browser versions. Older versions might still be supported (possibly with a reduced feature set). Also PRs are always welcome to improve support, even for older browsers.
My suggestion would be:
Mobile Devices:
same as Desktop: iOS = Safari, Android = Chrome
@nextcloud/contacts @jancborchardt what are your thoughts on this?
The text was updated successfully, but these errors were encountered: