Opened 11 years ago

Last modified 11 years ago

#3681 closed Bug

[Webkit] Shift enter broken — at Version 2

Reported by: Garry Yao Owned by: Garry Yao
Priority: Must have (possibly next milestone) Milestone: CKEditor 3.0
Component: General Version:
Keywords: Webkit Confirmed Review+ Cc:

Description (last modified by Garry Yao)

Reproducing Procedures

  1. Open the replace by class example page in FF;
  2. Move cursor inside the text, and press 'Shift-Enter';
    • Expected Result: Link break introduced with cursor blinking in front of next paragraph, and If you continue to press this keystroke many times until the the content is outside viewport, latest linebreak will also be scrolled into view;
    • Actual Result: The cursor is still at the end of the first paragraph.

Change History (3)

Changed 11 years ago by Garry Yao

Attachment: 3681.patch added

comment:1 Changed 11 years ago by Garry Yao

Keywords: Review? added
Status: newassigned

Several fixes in this patch:

  1. Fixing a logic error where bogus BR is not guaranteed in webkit when shift enter.
  2. The dummy node for hacking scroll into view function should have some content inside;
  3. Another defensive logic error within CKDITOR.dom.element::getDocumentPosition.

comment:2 Changed 11 years ago by Garry Yao

Description: modified (diff)
Priority: NormalHigh

Adding necessary 'Expected Result' regard auto scrolling into view.

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