Opened 14 years ago
Closed 14 years ago
#6263 closed Bug (fixed)
Safari : Cell Properties option is disabled in Pasted Table Cells when we invoke context menu by keeping cursor inside text
Reported by: | Satya Minnekanti | Owned by: | Sa'ar Zac Elias |
---|---|---|---|
Priority: | Normal | Milestone: | CKEditor 3.6.1 |
Component: | General | Version: | 3.4.1 |
Keywords: | IBM Safari | Cc: | Damian, joek |
Description
To reproduce the defect:
- In Safari Open Ajax sample.
- Copy a Table with content from MS Word and paste it in to editor.
- Keep the cursor inside text in Table Cells, Activate context menu and go to Cell sub menu.
Error:
Cell Properties option is disabled. but when we keep cursor out side the text or in an Empty cell & Activate Context menu Cell properties option is enabled.
Expected Result:
Cell Properties option should be enabled all the time irrespective of the cursor position inside Table Cell.
Attachments (3)
Change History (15)
Changed 14 years ago by
Attachment: | krst_15.02.29.png added |
---|
comment:1 follow-up: 3 Changed 14 years ago by
we can't even Delete Rows,Columns & Cells when we keep the cursor inside the text. Even though the option is shown as enabled when we click on that option nothing happens.
comment:2 Changed 14 years ago by
we can't even Split Cells Horizontally or Vertically when we keep the cursor inside the text in Table Cells. Even though the option is shown as enabled when we click on that option nothing happens.
comment:3 Changed 14 years ago by
Keywords: | Safari added |
---|---|
Status: | new → confirmed |
Version: | → 3.4.1 (SVN - trunk) |
Replying to satya:
we can't even Delete Rows,Columns & Cells when we keep the cursor inside the text. Even though the option is shown as enabled when we click on that option nothing happens.
Confirmed for table pasted from Word. In table created in CKE, ewerything works fine
we can't even Split Cells Horizontally or Vertically when we keep the cursor inside the text in Table Cells. Even though the option is shown as enabled when we click on that option nothing happens.
Confirmed, as above.
- Confirmed for pasting table via keyboard Ctrl+V and Context menu - Paste
- I was able to split and delete cells in table created in CKE, with cursor inside text.
- In Chrome7 ewerything works fine, so it seems it is safari bug
comment:4 Changed 14 years ago by
Owner: | set to Martin |
---|---|
Status: | confirmed → assigned |
comment:5 Changed 14 years ago by
The problem is that the Safari automatically select text on which was clicked right mouse button.
comment:6 follow-up: 7 Changed 14 years ago by
Cc: | damo,joek → damo, joek |
---|---|
Status: | assigned → pending |
WFM, can you provide a sample page?
comment:7 Changed 14 years ago by
Replying to garry.yao:
WFM, can you provide a sample page?
Use any sample and copy the table from attached word doc and paste in to editor. As mentioned in step 3 keep cursor inside the text in table cell.Invoke context menu and see Cell Properties option disabled. I could still reproduce this on nightly build.
Changed 14 years ago by
Attachment: | simple table.doc added |
---|
comment:8 Changed 14 years ago by
Status: | pending → confirmed |
---|
Confirmed with Safari.
The same issue can be seen with Firefox. Simply select the full text of the cell before opening the context menu. In Safari this is not needed because the selection happens automatically on right-click.
This happens because the cell contents are included in a <p> tag.
Changed 14 years ago by
Attachment: | 6263.patch added |
---|
comment:9 Changed 14 years ago by
Owner: | changed from Martin to Sa'ar Zac Elias |
---|---|
Status: | confirmed → review |
comment:10 Changed 14 years ago by
Status: | review → review_passed |
---|
comment:11 Changed 14 years ago by
Milestone: | → CKEditor 3.6.1 |
---|
comment:12 Changed 14 years ago by
Resolution: | → fixed |
---|---|
Status: | review_passed → closed |
Fixed with [6918].
Confirmed on 3.4.1 nightly