Opened 7 years ago

Closed 7 years ago

#5984 closed Bug (fixed)

FireFox 3.6.3: Hitting Ctrl-Home in the CKEditor puts a newline before the first line if the first line has text on it.

Reported by: Jude Allred Owned by: Frederico Caldeira Knabben
Priority: Normal Milestone: CKEditor 3.4
Component: General Version: 3.3
Keywords: Firefox Cc:

Description (last modified by Frederico Caldeira Knabben)

  1. go to http://ckeditor.com/demo
  2. click 'new page' (so that the editor is blank)
  3. type 'test' into the article
  4. hit ctrl+home

[expected] Cursor is moved to the beginning of the document

[actual] Cursor is moved to the beginning of the document, and a "<p>&nbsp;</p>" is prepended.

Attachments (1)

5984.patch (1.5 KB) - added by Frederico Caldeira Knabben 7 years ago.

Download all attachments as: .zip

Change History (8)

comment:1 Changed 7 years ago by Frederico Caldeira Knabben

Description: modified (diff)
Keywords: Firefox added
Status: newconfirmed

comment:2 Changed 7 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.4

That's definitely annoying.

comment:3 Changed 7 years ago by Frederico Caldeira Knabben

Version: 3.3

Works well with version 3.2.2.

comment:4 Changed 7 years ago by Frederico Caldeira Knabben

The issue has been introducing when merging trunk stuff into the 3.3.x branch with [5488]. Something there was incompatible with the changes we've made in the branch, much probably [5391].

Changed 7 years ago by Frederico Caldeira Knabben

Attachment: 5984.patch added

comment:5 Changed 7 years ago by Frederico Caldeira Knabben

Owner: set to Frederico Caldeira Knabben
Status: confirmedreview

This issue has been introduced with [5462].

I've fixed this issue with some code refactoring, which basically reverted those changes. I've checked the #5381 tcs thought and it works well, so we should be ok with it.

comment:6 Changed 7 years ago by Sa'ar Zac Elias

Status: reviewreview_passed

comment:7 Changed 7 years ago by Frederico Caldeira Knabben

Resolution: fixed
Status: review_passedclosed

Fixed with [5737].

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