Opened 10 years ago

Closed 10 years ago

#2977 closed Bug (fixed)

Focus issues when inserting a numbered list

Reported by: Damian Owned by: Martin Kou
Priority: Must have (possibly next milestone) Milestone: CKEditor 3.0
Component: General Version:
Keywords: IBM Confirmed Review- Cc:

Description (last modified by Garry Yao)

When adding a new numbered list at the end of the document, the list item is inserted correctly but the carat is positioned before the first list item and it is not possible to navigate to the first list item within WYSIWYG mode.

Steps to reproduce:

  1. Open nightly build. Currently (http://nightly.ckeditor.com/3105/_samples/replacebyclass.html)
  2. Add new line to the existing text in the editor
  3. Insert numbered list

Expected behavior: The new list is created with one item and the carat is positioned just after the number of the first item.

Actual behavior: The new list is created as expected. The position of the carat is placed before the first item in the list. It is not possible to navigate to the first item in the list.

Attachments (1)

2977.patch (663 bytes) - added by Martin Kou 10 years ago.

Download all attachments as: .zip

Change History (9)

comment:1 Changed 10 years ago by Damian

Applies to bullet lists also

comment:2 Changed 10 years ago by Garry Yao

Description: modified (diff)
Resolution: duplicate
Status: newclosed

DUP of #2952

comment:3 Changed 10 years ago by Martin Kou

Resolution: duplicate
Status: closedreopened

No, it isn't the same issue as #2952.

comment:4 Changed 10 years ago by Martin Kou

Keywords: Confirmed added
Owner: set to Martin Kou
Status: reopenednew

Changed 10 years ago by Martin Kou

Attachment: 2977.patch added

comment:5 Changed 10 years ago by Martin Kou

Keywords: Review? added

comment:6 Changed 10 years ago by Frederico Caldeira Knabben

Keywords: Review- added; Review? removed

Even if the specific case described by the reporter works after the patch, applying lists is still quite unstable.

Cases after applying the patch:

  1. Load replacebyclass sample.
  2. Place the caret at the end of the existing paragraph.
  3. Hit the list button (ERROR!)
  1. Load replacebyclass sample.
  2. Place the caret at the end of the existing paragraph.
  3. Hit ENTER to add a new paragraph.
  4. Hit the list button. List created (after the patch).
  5. Hit the list button. List removed.
  6. Hit the list button. (ERROR!)

comment:7 Changed 10 years ago by Martin Kou

Priority: NormalHigh

comment:8 Changed 10 years ago by Martin Kou

Resolution: fixed
Status: newclosed

The issue no longer exists in IE6/7, Firefox, Safari and Opera. It should have been fixed by a previous patch.

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