Opened 7 years ago

Closed 6 years ago

#4709 closed Bug (fixed)

[IE6] Inconsistent scrollbar behavior

Reported by: wwalc 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 6 years ago.
4709_2.patch (2.1 KB) - added by garry.yao 6 years ago.

Download all attachments as: .zip

Change History (9)

comment:1 Changed 7 years ago by fredck

  • Keywords IE added; IE6 removed
  • Milestone set to CKEditor 3.1

Confirmed with IE8 also.

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

comment:2 Changed 6 years ago by garry.yao

  • Owner set to garry.yao
  • Status changed from new to assigned

'PageUp/PageDown' keys are broken too.

Changed 6 years ago by garry.yao

comment:3 Changed 6 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 6 years ago by fredck

  • 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 6 years ago by garry.yao

comment:5 Changed 6 years ago by garry.yao

  • Keywords Review? added; Review- removed

comment:6 Changed 6 years ago by fredck

  • 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 6 years ago by garry.yao

  • Resolution set to fixed
  • Status changed from assigned to closed

Fixed with [4669].

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