FCKEditor 2.6.5 Does Not work with Safari 3.2.1

[Ruben Gamez]

I just updated to FCKEditor 2.6.5 to fix a height issue with Safari 4, but now Safari 3.2.1 doesn't display anything in the editor -- I also can't type anything in there.

This was working without any problems on both versions of Safari before the update. I'm using the replace text field method in case it helps. Don't hesitate to email me for more information.

[Ruben Gamez]

Screenshot of issue

[Domen Mali]

Hi,

i had the same problem. i review all the changes in my svn and found a "bug/solution".

in file "fckeditor.html (line 270) and fckeditor.original.html (line 378)" in "fckeditor/editor" folder

If i change

if ( FCKBrowserInfo.IsGecko || FCKBrowserInfo.IsSafari )

to

if ( FCKBrowserInfo.IsGecko && !FCKBrowserInfo.IsOpera )

FCK works again in Safari 3.2.1, but i guess that in Safari 4 the height problem appears again.

[Alfonso Martínez de Lizarrondo]

It's not possible to download Safari 3 from ​http://www.apple.com/safari/download/ so testing such environments isn't easy as they would require to keep some system with updates disabled

The described change is not feasible as that would mean that now IE also might execute that code, and if we leave that aside and only consider Opera and Safari, testing for IsOpera is the same that checking for IsSafari.

[Frederico Caldeira Knabben]

Replying to alfonsoml:

It's not possible to download Safari 3 from ​http://www.apple.com/safari/download/

Here you have it, for ​Leopard, ​Tiger and ​Windows.

Testing such environments isn't easy as they would require to keep some system with updates disabled

That's true... but it looks like we have options for it:
​http://michelf.com/projects/multi-safari/

In fact I was able to reproduce and confirm the bug.

The described change is not feasible as that would mean that now IE also might execute that code, and if we leave that aside and only consider Opera and Safari, testing for IsOpera is the same that checking for IsSafari.

True... I'll be providing a patch for it.

[Frederico Caldeira Knabben]

I'm proposing the same version detection code we use on V3.

[Alfonso Martínez de Lizarrondo]

Those links are interesting. I've heard too many times of IE Tester but I never show this option for Safari, and virtualising Mac OS X isn't too easy (as it's expensive to buy the Server version).

[Frederico Caldeira Knabben]

Fixed with [5123].