Opened 8 years ago

Closed 8 years ago

#4709 closed Bug (fixed)

[IE6] Inconsistent scrollbar behavior

Reported by: Wiktor Walc Owned by: Garry Yao
Priority: Normal Milestone: CKEditor 3.1
Component: General Version: 3.0.1
Keywords: IE Confirmed Review+ Cc:

Description

This issue does not occur if you don't click on the editing area before clicking on the scrollbar (e.g. switch from source to wysiwyg mode and click on a scrollbar).

Steps to reproduce:

  • open http://ckeditor.com/demo
  • copy the sample content in the editor and paste it at the bottom to have two copies in the editing area (just to have more content in the editor)
  • click on the first "Little Red Riding Hood" header at the very top
  • click on the scroll down arrow (or below the scrollbar)
  • result: the scrollbar jumps down and up and it is impossible to scroll down.

Attachments (2)

4709.patch (1.4 KB) - added by Garry Yao 8 years ago.
4709_2.patch (2.1 KB) - added by Garry Yao 8 years ago.

Download all attachments as: .zip

Change History (9)

comment:1 Changed 8 years ago by Frederico Caldeira Knabben

Keywords: IE added; IE6 removed
Milestone: CKEditor 3.1

Confirmed with IE8 also.

That's quite a serious issue. It should be related to the focus handling.

comment:2 Changed 8 years ago by Garry Yao

Owner: set to Garry Yao
Status: newassigned

'PageUp/PageDown' keys are broken too.

Changed 8 years ago by Garry Yao

Attachment: 4709.patch added

comment:3 Changed 8 years ago by Garry Yao

Keywords: Review? added

'PageUp/PageDown' keys are broken too.

This's been proved to be yet another issue, new ticket for it at #4736.

comment:4 Changed 8 years ago by Frederico Caldeira Knabben

Keywords: Review- added; Review? removed

The patch brings another issue:

  1. In a long text, make a selection.
  2. Click and drag the scroolbar bar.

It will not work. It works only on a second click.

Changed 8 years ago by Garry Yao

Attachment: 4709_2.patch added

comment:5 Changed 8 years ago by Garry Yao

Keywords: Review? added; Review- removed

comment:6 Changed 8 years ago by Frederico Caldeira Knabben

Keywords: Review+ added; Review? removed

It seems that that code removed from lines 143-148 was simply related to the visual behavior of the editor, to make it work more like normal fields. I think we can live with it for now and wait for feedback.

comment:7 Changed 8 years ago by Garry Yao

Resolution: fixed
Status: assignedclosed

Fixed with [4669].

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