Opened 7 years ago

Closed 6 years ago

Last modified 6 years ago

#5778 closed Bug (fixed)

[IE] Unwanted scroll on first mouse right-click

Reported by: garry.yao Owned by: garry.yao
Priority: Normal Milestone: CKEditor 3.4.1
Component: General Version: 3.0
Keywords: IE Cc:

Description

Environment

All IE versions

Reproducing Procedures

  1. Open our online demo page;
  2. Scroll down the document to bottom without puting focus into the editing document;
  3. Mouse right-click (contextmenu) at the last paragraph;
    • Actual Result: The document is scrolling to top and the context menu is opened at the new position.
    • Expected Result: Context menu opens at the clicked location with no document scrolling occurs.

Attachments (2)

5778.patch (1.4 KB) - added by garry.yao 7 years ago.
5778_2.patch (1.4 KB) - added by Saare 6 years ago.
Updated patch

Download all attachments as: .zip

Change History (8)

comment:1 Changed 7 years ago by garry.yao

It's pretty much a browser bug here of IE's behavior on right mouse click:

  1. Mousedown: Focus the document first if window doesn't gain focus yet;
  2. Mouseup: Make a collapsed selection at the mouse position in document;

There's an obviously problem with this is that the first step may cause document scroll and thus the at the second step the document position indicated by the mouse is already changed.

Comparing with other browsers, e.g. Firefox, is actually making the selection at the first step.

comment:2 Changed 7 years ago by garry.yao

  • Keywords Review? added
  • Owner set to garry.yao
  • Status changed from new to assigned

Manual Ticket Test added:
run OR view source.

Changed 7 years ago by garry.yao

comment:3 Changed 6 years ago by fredck

  • Keywords IE added; Confirmed IE? removed
  • Milestone changed from CKEditor 3.4 to CKEditor 3.5

Changed 6 years ago by Saare

Updated patch

comment:4 Changed 6 years ago by Saare

  • Status changed from review to review_passed

comment:5 Changed 6 years ago by tobiasz.cudnik

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

Fixed with [5876].

comment:6 Changed 6 years ago by tobiasz.cudnik

Correction: Fixed with [5877] not [5876].

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