Opened 18 years ago
Last modified 18 years ago
#36 closed Bug
bug in FCKTools.GetParentWindow — at Initial Version
Reported by: | Alfonso Martínez de Lizarrondo | Owned by: | Alfonso Martínez de Lizarrondo |
---|---|---|---|
Priority: | Normal | Milestone: | FCKeditor 2.4 |
Component: | General | Version: | |
Keywords: | Cc: |
Description
I've run the automated test (first I've had to patch the test files adding a
FCKScriptLoader._FCKeditorPath = document.location.pathname.substring(0,document.location.pathname.lastIndexOf('_test')) ;
so it does work in any directory) and then I've found that Firefox had an error in FCKTools.GetParentWindow I've looked at it and I think that it was wrong because it tries to use document.contentWindow but contentWindow is a property of an iframe, it doesn't point to the parent window of the document in any browser. Instead I've changed to
FCKTools.GetParentWindow = function( document ) { document.parentWindow ; return document.parentWindow ? document.parentWindow : document.defaultView ; }
That should work in IE, Firefox and Opera.
All of these changes are in my development branch