Ticket #3950 (closed Bug: fixed)

Opened 8 months ago

Last modified 4 months ago

[Safari] Insert column before/after problematic

Reported by: damo Owned by: garry.yao
Priority: Normal Milestone: CKEditor 3.1
Component: General Version: SVN (CKEditor)
Keywords: IBM Confirmed WebKit Review+ Cc:

Description (last modified by fredck) (diff)

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

3950.patch Download (1.5 KB) - added by garry.yao 8 months ago.

Change History

  Changed 8 months ago by garry.yao

  • keywords WorksForMe added

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

follow-up: ↓ 3   Changed 8 months ago by damo

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.

in reply to: ↑ 2   Changed 8 months ago by fredck

  • keywords Confirmed WebKit added; WorksForMe removed
  • version set to 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.

  Changed 8 months ago by garry.yao

  • owner set to garry.yao
  • status changed from new to assigned

Changed 8 months ago by garry.yao

  Changed 8 months 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.

  Changed 4 months ago by fredck

  • description modified (diff)

  Changed 4 months ago by fredck

  • keywords Review+ added; Review? removed

  Changed 4 months ago by garry.yao

  • status changed from assigned to closed
  • resolution set to fixed

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

Note: See TracTickets for help on using tickets.