Opened 10 years ago

Closed 10 years ago

#3692 closed Bug (fixed)

Find scrollIntoView incorrect

Reported by: Garry Yao Owned by: Garry Yao
Priority: Normal Milestone: CKEditor 3.0
Component: Core : Styles Version:
Keywords: Confirmed Review+ Cc:

Description

Reproducing Procedures

  1. Open the replace by class example page in FF;
  2. Press 'Enter' after the first paragraph until the text is not visible within current viewport;
  3. Now open 'Find' dialog and fill in 'Find What' field with some word inside the the paragraph;
  4. Prompt with 'The specified text not found', click 'Ok', then click the 'Find' button again;
    • Expected Result: The matched word in the first paragraph is now scrolled into view;
    • Actual Result: It's matched, but never scroll into view.

Attachments (2)

3692.patch (547 bytes) - added by Garry Yao 10 years ago.
3692_2.patch (895 bytes) - added by Garry Yao 10 years ago.

Download all attachments as: .zip

Change History (6)

Changed 10 years ago by Garry Yao

Attachment: 3692.patch added

comment:1 Changed 10 years ago by Garry Yao

Keywords: Review? added
Status: newassigned

Changed 10 years ago by Garry Yao

Attachment: 3692_2.patch added

comment:2 Changed 10 years ago by Garry Yao

The matching scrolling should keep the by default align to bottom' setting, while the correct fix should be somewhere else.

comment:3 Changed 10 years ago by Martin Kou

Keywords: Review+ added; Review? removed

comment:4 Changed 10 years ago by Garry Yao

Resolution: fixed
Status: assignedclosed

Fixed with [3653].

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