#14894 closed Bug (fixed)

Editor scrolls to top when focused or when dialog is opened.

Reported by: Jakub Ś Owned by: Tade0
Priority: Normal Milestone: CKEditor 4.7.0
Component: General Version:
Keywords: Chrome Cc:

Description

Steps to reproduce

These are two issues in fact. First one concerns opening dialog when editor is not focused and second one is the continuation of #14659.

Test Case 1

  1. http://dev.ckeditor.com/ticket/14659#comment:6. Test case from that comment is still reproducible (starting from 4.0.0).

Test Case 2

  1. Open sample page - replacebycode.html or divarea.html
  2. Scroll editor a little bit without focusing it
  3. Open Link dialog

This test case can be reproduced in both classic (starting from 4.4.6) and divarea editor (starting from 4.0.0)

Expected result

Editor should not scroll.

Actual result

In both cases editor will be scrolled to top.

Other details (browser, OS, CKEditor version, installed plugins)

Problem can be reproduced in Chrome browser only

Change History (8)

comment:1 Changed 14 months ago by Jakub Ś

Status: newconfirmed

comment:2 Changed 12 months ago by Marek Lewandowski

Milestone: CKEditor 4.6.1CKEditor 4.6.2

Moving to 4.6.2 minor release, as 4.6.1 is mostly about polishing 4.6.0.

comment:3 Changed 11 months ago by Marek Lewandowski

Milestone: CKEditor 4.6.2CKEditor 4.7.0

comment:4 Changed 10 months ago by Tade0

Owner: set to Tade0
Status: confirmedassigned

comment:5 Changed 10 months ago by Tade0

Status: assignedreview

Here's an admittedly blunt, yet working solution.

Reasoning: we have no control over what exaclty happens when a native method is called, so we need to work around it.

Changes pushed to branch:t/14894.

comment:6 Changed 10 months ago by Tomasz Jakut

Status: reviewreview_failed

Fix is working fine, but it lacks unit test. It shouldn't be too hard to create. I'm also not sure if it wouldn't be better to apply this fix only to Chrome.

comment:7 in reply to:  6 Changed 10 months ago by Tade0

Status: review_failedreview

Replying to t.jakut:

I'm also not sure if it wouldn't be better to apply this fix only to Chrome.

Yeah, I probably should.

Added unit test, changes pushed to branch:t/14894.

comment:8 Changed 10 months ago by Tomasz Jakut

Resolution: fixed
Status: reviewclosed

LGTM, I've just polished it a bit and merged to major with git:af78e85.

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