Opened 5 years ago

Closed 5 years ago

#7174 closed Bug (fixed)

IE9: Cursor path does not load corectly when editor is switched back from source to wysiwyg

Reported by: krst Owned by: garry.yao
Priority: Normal Milestone: CKEditor 3.5.2
Component: General Version: 3.5.3
Keywords: IE9 Cc:

Description (last modified by krst)

Environment

IE9 Browser mode IE9, document mode IE9 standards CKEditor set to source

TC

  • clear contents
  • insert default table
  • in first row type some text
  • switch to source and back to wysiwyg
  • click on first cell in second column (as on attached screenshot)

Expected

caret position is shown in 'status bar of editor'. Example body blockquote table tbody tr td

Actual

The bottom line in Editor is empty, until next operation is done. User is allowed to make nested blockquotes, which is not possible in other browsers (by repeating two last steps and clicking blockquote button).

<blockquote>
  <blockquote>
    <blockquote>
      <table border="1" cellpadding="1" cellspacing="1" style="width: 500px;">
	<tbody>
	  <tr>
		<td>
...

Attachments (2)

krst_20110214-01.png (33.3 KB) - added by krst 5 years ago.
Empty bottom bar
7174.patch (463 bytes) - added by garry.yao 5 years ago.

Download all attachments as: .zip

Change History (10)

Changed 5 years ago by krst

Empty bottom bar

comment:1 Changed 5 years ago by krst

  • Description modified (diff)

comment:2 Changed 5 years ago by Saare

  • Status changed from new to confirmed

Changed 5 years ago by garry.yao

comment:3 Changed 5 years ago by garry.yao

  • Owner set to garry.yao
  • Status changed from confirmed to review

comment:4 Changed 5 years ago by wwalc

  • Milestone set to CKEditor 3.5.2

comment:5 Changed 5 years ago by Saare

  • Status changed from review to review_failed

The patch does not resolve the issue for me.

comment:6 Changed 5 years ago by garry.yao

  • Status changed from review_failed to review

Rational behind the fix:

  1. We guard the selection change as ( to filter those equivilent ones):

if ( !currentPath.compare( this._.selectionPreviousPath ) ) { ... }

  1. IE9 doesn't allow to compare live nodes (in current doc) with offline ones (in old doc because of switch mode), throws an error but swalled by a catch.
  2. Invalidate the last path cache before unload the doc make it works.

comment:7 Changed 5 years ago by Saare

  • Status changed from review to review_passed

Path does work, but it seems like SVN puts the line in the wrong place, which failed the patch from working.

comment:8 Changed 5 years ago by garry.yao

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

Fixed with [6441].

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