FCKEditor 2.6.5 Does Not work with Safari 3.2.1

[ruben.g]

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.g]

Screenshot of issue

[domenm]

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.

[alfonsoml]

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.

[fredck]

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.

[fredck]

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

[alfonsoml]

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).

[fredck]

Fixed with [5123].