Opened 7 years ago

Closed 7 years ago

Last modified 7 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 Sa'ar Zac Elias 7 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: newassigned

Manual Ticket Test added:
run OR view source.

Changed 7 years ago by Garry Yao

Attachment: 5778.patch added

comment:3 Changed 7 years ago by Frederico Caldeira Knabben

Keywords: IE added; Confirmed IE? removed
Milestone: CKEditor 3.4CKEditor 3.5

Changed 7 years ago by Sa'ar Zac Elias

Attachment: 5778_2.patch added

Updated patch

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

Status: reviewreview_passed

comment:5 Changed 7 years ago by Tobiasz Cudnik

Resolution: fixed
Status: review_passedclosed

Fixed with [5876].

comment:6 Changed 7 years ago by Tobiasz Cudnik

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

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