Opened 8 years ago

Closed 8 years ago

#3950 closed Bug (fixed)

[Safari] Insert column before/after problematic

Reported by: Damian Owned by: Garry Yao
Priority: Normal Milestone: CKEditor 3.1
Component: General Version: SVN (CKEditor) - OLD
Keywords: IBM Confirmed WebKit Review+ Cc:

Description (last modified by Frederico Caldeira Knabben)

To reproduce:

  1. Open nightly sample
  2. Insert table with 3 columns
  3. Populate the cells in the first row to keep track of them.
  4. Select and right click in the empty area of a cell in the first row (to the right of the text in the cell)
  5. Attempt to insert a column before the current column

Result is that the column is added to the beginning of the table and not before the selected column

Attachments (1)

3950.patch (1.5 KB) - added by Garry Yao 8 years ago.

Download all attachments as: .zip

Change History (9)

comment:1 Changed 8 years ago by Garry Yao

Keywords: WorksForMe added

WFM,Could you tell me on which cell did you trigger the menu exactly?

comment:2 Changed 8 years ago by Damian

I tried on the last cell in the first row, and on a middle cell in the first row.

The important thing is to right click in the empty part of the cell.

I am using Safari 3.2.2 on WinXP.

comment:3 in reply to:  2 Changed 8 years ago by Frederico Caldeira Knabben

Keywords: Confirmed WebKit added; WorksForMe removed
Version: SVN (CKEditor)

Replying to damo:

I am using Safari 3.2.2 on WinXP.

That's all we needed to know. Please always include the browser version in the reports.

I've confirmed it on Safari. Ok with FF and IE.

comment:4 Changed 8 years ago by Garry Yao

Owner: set to Garry Yao
Status: newassigned

Changed 8 years ago by Garry Yao

Attachment: 3950.patch added

comment:5 Changed 8 years ago by Garry Yao

Keywords: Review? added

It could be considered as a regression of [3456], where determinate the selectedElement for selection below:

<tr><td>cell1[</td><td>]cell2</td></tr>

The cell2 <td> is reported instead of the first one, I'm proposing of limit #3231's fix to only inline elements to avoid this problem temporarily.

The real criminal is the context menu on safari( chrome doesn't have the problem), on 'contextmenu' event, instead of keeping the current document selection, the browser is trying to grabbing a word range in place, which is really cumbersome and troublesome. Even worse, this behavior is very hard to prevent, I've tried various way without reaching an idea result, we can have it under investigation later.

comment:6 Changed 8 years ago by Frederico Caldeira Knabben

Description: modified (diff)

comment:7 Changed 8 years ago by Frederico Caldeira Knabben

Keywords: Review+ added; Review? removed

comment:8 Changed 8 years ago by Garry Yao

Resolution: fixed
Status: assignedclosed

Fixed with [4516]. Ticket Manual Test added at :
http://ckeditor.t/tt/3950/1.html.

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