CKEDITOR.env.isCompatible should be a blacklist rather than a whiletlist

[Piotrek Koszuliński]

The reason is pretty simple – we know on what browsers CKEditor does not work (IE7-), but we do not know on what browsers it works and will work in the future. So given the problems with browsers recognition it will be much safer to have a blacklist.

[Piotrek Koszuliński]

The most important issue that this change will fix is the fact that on many Android devices CKEditor does not work on Chrome if that browser is configured to be seen as a mobile one.

[Anna Tomanek]

A big fat yes from me, more and more people are turning away from CKEditor since it "does not work on mobile" even though it may actually work quite all right with minor glitches, it's just disabled on purpose.

[Wiktor Walc]

+1 from me although I have some mixed feelings about enabling it on browsers like Konqueror that we don't test.

[Piotrek Koszuliński]

I have some mixed feelings about enabling it on browsers like Konqueror that we don't test.

I had too. But the truth is:

• How do we recognise Konqueror?
• Why do we assume that CKEditor does not work there?
• How many people does it concern?

[Frederico Caldeira Knabben]

++1

[Wiktor Walc]

Replying to Reinmar:

I have some mixed feelings about enabling it on browsers like Konqueror that we don't test.

I had too. But the truth is:

• How do we recognise Konqueror?
• Why do we assume that CKEditor does not work there?
• How many people does it concern?

Yep, the list of advantages is much bigger than disadvantages. Adding my comment here just so that anyone was aware that we took it into consideration.

[Piotrek Koszuliński]

Pushed ​branch:t/13316, but I think that we'll need to discuss what we place here - ​http://docs.ckeditor.com/#!/guide/dev_browsers

[Piotrek Koszuliński]

We decided to completely rewrite ​​http://docs.ckeditor.com/#!/guide/dev_browsers and that will be held in another ticket. For now, let's review the dev changes.

Pushed ​branch:t/13316.

To the reviewer: Please make sure that the new test passes on the Edge.

[Olek Nowodziński]

Merged into major in ​git:384147f.