Opened 8 years ago

Closed 7 years ago

#5280 closed Bug (fixed)

Vertical scroll bounces up when clicking in-between the vertical scroll bar and the bottom arrow

Reported by: markuspaek Owned by: Frederico Caldeira Knabben
Priority: Normal Milestone: CKEditor 3.4
Component: General Version: 3.2
Keywords: IE Cc: philip_andrew@…

Description (last modified by Tobiasz Cudnik)

XP + IE6

It happens at http://ckeditor.com/demo

When I click in-between the vertical scroll bar and the bottom arrow, the page scrolls down but bounces back. The same goes when I click in-between the vertical scroll bar and the top arrow.

Additional reproduction scenario moved from #5581:

In ie7, I go to http://ckeditor.com/demo

Click on the word Red in little red riding hood, just put the cursor somewhere there by clicking on the text.

Use the scrollbar to scroll down (down button on scrollbar, not the scrollbar itself, use down button).

Cannot scroll down, the focus jumps back to the cursor at the top of the page, so the whole page jumps back upwards.

This is a serious problem for my customer as they can only use IE7.

Attachments (1)

5280.patch (1.6 KB) - added by Frederico Caldeira Knabben 7 years ago.

Download all attachments as: .zip

Change History (16)

comment:1 Changed 7 years ago by Frederico Caldeira Knabben

Keywords: Confirmed IE added
Milestone: CKEditor 3.4

Confirmed with IE8.

comment:2 Changed 7 years ago by Frederico Caldeira Knabben

#5581 has been marked as DUP. It provides a different way to reproduce it.

comment:3 Changed 7 years ago by Frederico Caldeira Knabben

May be related to #5735.

comment:4 Changed 7 years ago by Garry Yao

Owner: set to Garry Yao
Status: newassigned

Supposed to be fixed by #5735.

comment:5 Changed 7 years ago by phil

Cc: philip_andrew@… added

Your patch for this didn't fix #5581. Please look at #5581 to see the fix.

comment:6 Changed 7 years ago by Tobiasz Cudnik

Description: modified (diff)

Quoting philip_andrew from #5581:

I found I can fix this bug by commenting out the following code in plugin.js in /plugins/selection/plugin.js

// IE before version 8 will leave cursor blinking inside the document after
// editor blurred unless we clean up the selection. (#4716)
if ( CKEDITOR.env.ie && CKEDITOR.env.version < 8 )
{
doc.getWindow().on( 'blur', function( evt )
{
editor.document.$.selection.empty();
});
}

comment:7 Changed 7 years ago by phil

I fixed the bug by deleting the above code AFTER applying garry.yaos patch. I mean, it needed the patch as well.

comment:8 Changed 7 years ago by Frederico Caldeira Knabben

Owner: changed from Garry Yao to Frederico Caldeira Knabben

comment:9 Changed 7 years ago by Frederico Caldeira Knabben

Unfortunately not fixed with #5735.

comment:10 Changed 7 years ago by phil

I have fixed this bug in my FCKEditor, as my customer is in-company and must use IE7, its easy to fix this bug, please do so - so that I can use future versions of FCKEditor and not be stuck with my bug fixed one.

The customer cannot upgrade IE as they are inside company. If you need any help, then contact me at philip(underscore)andrew at hotmail thanks

comment:11 Changed 7 years ago by Frederico Caldeira Knabben

In fact, things should have already been fixed after #5735, but [5742] reintroduced the issue.

comment:12 in reply to:  11 Changed 7 years ago by Frederico Caldeira Knabben

Replying to fredck:

In fact, things should have already been fixed after #5735, but [5742] reintroduced the issue.

Sorry, [5738], not [5742].

Changed 7 years ago by Frederico Caldeira Knabben

Attachment: 5280.patch added

comment:13 Changed 7 years ago by Frederico Caldeira Knabben

Status: assignedreview

That one was hard to find a solution, but fortunately there is a way to workaround things.

For that I had to revert [5738], moving it to another place. It still satisfies #5634 as well as fixes this ticket.

comment:14 Changed 7 years ago by Sa'ar Zac Elias

Keywords: Confirmed removed
Status: reviewreview_passed

Please add the ID of this ticket to the comment for refrence.

comment:15 Changed 7 years ago by Frederico Caldeira Knabben

Resolution: fixed
Status: review_passedclosed

Fixed with [5760].

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