Ticket #5280 (closed Bug: fixed)

Opened 5 years ago

Last modified 4 years ago

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

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

Description (last modified by tobiasz.cudnik) (diff)

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

5280.patch (1.6 KB) - added by fredck 4 years ago.

Change History

comment:1 Changed 4 years ago by fredck

  • Keywords Confirmed IE added
  • Milestone set to CKEditor 3.4

Confirmed with IE8.

comment:2 Changed 4 years ago by fredck

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

comment:3 Changed 4 years ago by fredck

May be related to #5735.

comment:4 Changed 4 years ago by garry.yao

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

Supposed to be fixed by #5735.

comment:5 Changed 4 years ago by philip142au

  • Cc philip_andrew@… added

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

comment:6 Changed 4 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 4 years ago by philip142au

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 4 years ago by fredck

  • Owner changed from garry.yao to fredck

comment:9 Changed 4 years ago by fredck

Unfortunately not fixed with #5735.

comment:10 Changed 4 years ago by philip142au

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 follow-up: ↓ 12 Changed 4 years ago by fredck

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

comment:12 in reply to: ↑ 11 Changed 4 years ago by fredck

Replying to fredck:

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

Sorry, [5738], not [5742].

Changed 4 years ago by fredck

comment:13 Changed 4 years ago by fredck

  • Status changed from assigned to review

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 4 years ago by Saare

  • Status changed from review to review_passed
  • Keywords Confirmed removed

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

comment:15 Changed 4 years ago by fredck

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

Fixed with [5760].

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