Opened 14 years ago

Closed 14 years ago

Last modified 14 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 14 years ago.
5778_2.patch (1.4 KB) - added by Sa'ar Zac Elias 14 years ago.
Updated patch

Download all attachments as: .zip

Change History (8)

comment:1 Changed 14 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 14 years ago by Garry Yao

Keywords: Review? added
Owner: set to Garry Yao
Status: newassigned

Manual Ticket Test added:
run OR view source.

Changed 14 years ago by Garry Yao

Attachment: 5778.patch added

comment:3 Changed 14 years ago by Frederico Caldeira Knabben

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

Changed 14 years ago by Sa'ar Zac Elias

Attachment: 5778_2.patch added

Updated patch

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

Status: reviewreview_passed

comment:5 Changed 14 years ago by Tobiasz Cudnik

Resolution: fixed
Status: review_passedclosed

Fixed with [5876].

comment:6 Changed 14 years ago by Tobiasz Cudnik

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

Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy