Opened 15 years ago
Closed 15 years ago
#3551 closed Bug (duplicate)
FCK Editor 2.6.4 IE8 HTML4 scrolling errors
| Reported by: | Harry | Owned by: | |
|---|---|---|---|
| Priority: | Normal | Milestone: | |
| Component: | General | Version: | FCKeditor 2.6.4 |
| Keywords: | Cc: |
Description
Descriptive summary:
JavaScript errors appearing when scrolling the editor window in IE8, when clicking on the scrollbar. The error only occurs when the doctype is set to HTML4 (FCKConfig.DocType).
Steps to reproduce:
It is quite easy to reproduce - edit fckeditor\_samples\html\assets\sample15.config.js and add the line:
FCKConfig.DocType = '<!DOCTYPE html PUBLIC "-W3CDTD HTML 4.01 TransitionalEN" "http://www.w3.org/TR/html4/loose.dtd">';
Then run sample 15 and put some new lines in to make a scrollbar appear in the editing area. If you click and drag the scrollbar, IE8 throws an exception and asks if you want to debug the web page. The error message given is "'nodeName' is null or not an object".
Browser name and OS: Bug appears in: IE8, Windows XP or Vista. Bug does not appear: IE6 or IE7. Also does not appear in IE8 with no doctype, a doctype of HTML3.2 or XHTML1/1.1. Only HTML 4!
Fix:
The problem appears to be in fck_ie.js, function Doc_OnMouseDown, line 451
if ( e.nodeName.IEquals( 'input' ) && e.type.IEquals( ['radio', 'checkbox'] ) && !e.disabled )
If this is changed to check that nodeName is defined first, then the errors go away, i.e:
if (typeof e.nodeName != 'undefined' && e.nodeName.IEquals( 'input' ) && e.type.IEquals( ['radio', 'checkbox'] ) && !e.disabled )
This bug has not been fixed on the nightly build.
dup of #3429