Opened 9 years ago

Closed 9 years ago

#2525 closed Bug (fixed)

Chrome: error if FCKConfig.StartupShowBlocks = true

Reported by: fournaise Owned by: Martin Kou
Priority: Normal Milestone: FCKeditor 2.6.4
Component: General Version: FCKeditor 2.6.3 Beta
Keywords: Confirmed Chrome Firefox Review+ Cc: alexandre@…

Description (last modified by Martin Kou)

Hello, If you execute FCKEditor on Google Chrome on the latest trunk (seem appear in the 2.6.3), I have an error if FCKConfig.StartupShowBlocks is set to true.

Uncaught TypeError: Cannot read property 'nodeType' of null http://127.0.0.1/www.lib/fckeditor/edit ... wblocks.js (line 59)
[...]
if ( FCKBrowserInfo.IsIE )
{
try
{
FCK.EditorDocument.selection.createRange().select() ;
}
catch ( e )
{}
}
else
{
var focus = FCK.EditorWindow.getSelection().focusNode ;
if ( focus.nodeType != 1 )
Uncaught TypeError: Cannot read property 'nodeType' of null
focus = focus.parentNode ;
FCKDomTools.ScrollIntoView( focus, false ) ;
}
[...]

To fix the problem I have just added "try catch" in the "else" branch.

regards Frederic

Attachments (1)

2525.patch (1.3 KB) - added by Martin Kou 9 years ago.

Download all attachments as: .zip

Change History (10)

comment:1 Changed 9 years ago by fournaise

file is fckeditor/editor/_source/commandclasses/fckshowblocks.js (line 59)

comment:2 Changed 9 years ago by Artur Formella

Keywords: Confirmed Chrome added
Milestone: FCKeditor 2.6.4
Version: SVNFCKeditor 2.6.3 Beta

Confirmed. Google Chrome 0.2.149

Uncaught TypeError: Cannot read property 'nodeType' of null http://localhost/edytor/FCKeditor/tags/2.6.3/editor/_source/commandclasses/fckshowblocks.js (line 59)

comment:3 Changed 9 years ago by Alexandre Plennevaux

Cc: alexandre@… added
Keywords: Firefox added
Priority: NormalHigh
Summary: Google Chrome bug if FCKConfig.StartupShowBlocks is set to true.Google Chrome AND Firefox bug if FCKConfig.StartupShowBlocks is set to true.
Version: FCKeditor 2.6.3 BetaFCKeditor 2.6.3

I also have this issue in Firefox 3. It provokes this message:

Error : FCK.EditorWindow.getSelection() is null Source file : http://localhost/lab-au.com/dev/_js/_FCKeditor/editor/js/fckeditorcode_gecko.js Line : 71FCKSelection.GetSelection()

Disabling the option disables the bug.

comment:4 Changed 9 years ago by Frederico Caldeira Knabben

Milestone: FCKeditor 2.6.4
Priority: HighNormal
Summary: Google Chrome AND Firefox bug if FCKConfig.StartupShowBlocks is set to true.Chrome: error if FCKConfig.StartupShowBlocks = true
Version: FCKeditor 2.6.3FCKeditor 2.6.3 Beta

comment:5 Changed 9 years ago by Martin Kou

Owner: set to Martin Kou
Status: newassigned

comment:6 Changed 9 years ago by Martin Kou

Description: modified (diff)

Tidied up the description a bit.

Changed 9 years ago by Martin Kou

Attachment: 2525.patch added

comment:7 Changed 9 years ago by Martin Kou

Keywords: Review? added

comment:8 Changed 9 years ago by Alfonso Martínez de Lizarrondo

Keywords: Review+ added; Review? removed

comment:9 Changed 9 years ago by Martin Kou

Resolution: fixed
Status: assignedclosed

Fixed with [2563].

Click here for more info about our SVN system.

Note: See TracTickets for help on using tickets.
© 2003 – 2017 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy