Opened 8 years ago

Closed 8 years ago

#2525 closed Bug (fixed)

Chrome: error if FCKConfig.StartupShowBlocks = true

Reported by: fournaise Owned by: martinkou
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 martinkou)

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 martinkou 8 years ago.

Download all attachments as: .zip

Change History (10)

comment:1 Changed 8 years ago by fournaise

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

comment:2 Changed 8 years ago by arczi

  • Keywords Confirmed Chrome added
  • Milestone FCKeditor 2.6.4 deleted
  • Version changed from SVN to FCKeditor 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 8 years ago by aplennevaux

  • Cc alexandre@… added
  • Keywords Firefox added
  • Priority changed from Normal to High
  • Summary changed from Google Chrome bug if FCKConfig.StartupShowBlocks is set to true. to Google Chrome AND Firefox bug if FCKConfig.StartupShowBlocks is set to true.
  • Version changed from FCKeditor 2.6.3 Beta to FCKeditor 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 8 years ago by fredck

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

comment:5 Changed 8 years ago by martinkou

  • Owner set to martinkou
  • Status changed from new to assigned

comment:6 Changed 8 years ago by martinkou

  • Description modified (diff)

Tidied up the description a bit.

Changed 8 years ago by martinkou

comment:7 Changed 8 years ago by martinkou

  • Keywords Review? added

comment:8 Changed 8 years ago by alfonsoml

  • Keywords Review+ added; Review? removed

comment:9 Changed 8 years ago by martinkou

  • Resolution set to fixed
  • Status changed from assigned to closed

Fixed with [2563].

Click here for more info about our SVN system.

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