Opened 16 years ago
Closed 15 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 )
To reproduce:
- Open nightly sample
- Insert table with 3 columns
- Populate the cells in the first row to keep track of them.
- Select and right click in the empty area of a cell in the first row (to the right of the text in the cell)
- 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)
Change History (9)
comment:1 Changed 16 years ago by
Keywords: | WorksForMe added |
---|
comment:2 follow-up: 3 Changed 16 years ago by
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 Changed 16 years ago by
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 16 years ago by
Owner: | set to Garry Yao |
---|---|
Status: | new → assigned |
Changed 16 years ago by
Attachment: | 3950.patch added |
---|
comment:5 Changed 16 years ago by
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 15 years ago by
Description: | modified (diff) |
---|
comment:7 Changed 15 years ago by
Keywords: | Review+ added; Review? removed |
---|
comment:8 Changed 15 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Fixed with [4516].
Ticket Manual Test added at :
http://ckeditor.t/tt/3950/1.html.
WFM,Could you tell me on which cell did you trigger the menu exactly?