Backspace/delete doesn't work with many empty lines in Firefox

[Josh Nisly]

To duplicate, open the divreplace sample. Delete all the text, then hold down the enter key until the scrollbar is smaller than the down-arrow button on the scrollbar. Now try pressing backspace, delete, etc. Nothing will remove those lines. Now type another letter. Suddenly, all the lines can be deleted.

I can only reproduce this in FF on Windows (not linux), but I can reproduce it consistently there. V2 works correctly. This can be reproduced in the nightly sample.

[Garry Yao]

A even more easy way to reproduce with nightly build:

Reproducing Procedures

1. Open the replace by class example page;
2. Click on 'New Page' button to clear all content;
3. Hit 'enter' key, then hit 'backspace' key.
   • Expected Result: The new paragraph has been deleted.
   • Actual Result: The new paragraph is still there.

[Garry Yao]

After some investigation, this bug has been caused by the appearance of <br _moz_editor_bogus_node="TRUE" _moz_dirty=""/> inside the document, this happened when you clear the document content with the following ways:

1. Switch to 'source', delete all codes and switch back;
2. Click 'New Page' command.

Actually there's no way of preventing FF from generating this bogus node, and my testing shows that remove this node could never help to resolve the issue.
Since it belongs to some sort of browser bug and this case doesn't happen very often, we can hold this one for a while.

[Josh Nisly]

It does seem like a browser weirdness, but V2 works correctly. Is there really no way to work around it?

[Garry Yao]

There's a workaround for this by guarding on the empty document data, putting our own padding br.
Highjinx, it's great if you can also reviewing this.

[Josh Nisly]

Did you mean to attach a patch?

[Garry Yao]

Replying to highjinx_53:

Did you mean to attach a patch?

I mean please test the patch against your profile, and of course, attach a better fix at your will.

[Frederico Caldeira Knabben]

Garry, you have asked review, but no patch is available here? I'm a bit confused. Can you clarify it?

[Garry Yao]

I saw confusion here all because of missing of the patch file, my fault.

[Frederico Caldeira Knabben]

I'll wait to review it only after #3505.

[Frederico Caldeira Knabben]

I'm not able to reproduce this one, and Garry also thinks this one is hard to reproduce now, as e have introduced other features in the editor.

I'll still leave this opened for a while, so we can check whether it's needed or not.

[Frederico Caldeira Knabben]

Milestone CKEditor 3.x deleted

[Frederico Caldeira Knabben]

Expired.