Opened 6 years ago

Closed 5 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 Owned by: Saare
Priority: Normal Milestone: CKEditor 3.6.1
Component: General Version: 3.4.1
Keywords: IBM Safari Cc: damo, joek

Description

To reproduce the defect:

  1. In Safari Open Ajax sample.
  1. Copy a Table with content from MS Word and paste it in to editor.
  1. 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)

krst_15.02.29.png (68.0 KB) - added by krst 6 years ago.
Confirmed on 3.4.1 nightly
simple table.doc (23.5 KB) - added by satya 5 years ago.
6263.patch (776 bytes) - added by Saare 5 years ago.

Download all attachments as: .zip

Change History (15)

Changed 6 years ago by krst

Confirmed on 3.4.1 nightly

comment:1 follow-up: Changed 6 years ago by 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.

comment:2 Changed 6 years ago by satya

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 in reply to: ↑ 1 Changed 6 years ago by krst

  • Keywords Safari added
  • Status changed from new to confirmed
  • Version set to 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 6 years ago by mani

  • Owner set to mani
  • Status changed from confirmed to assigned

comment:5 Changed 6 years ago by mani

The problem is that the Safari automatically select text on which was clicked right mouse button.

comment:6 follow-up: Changed 5 years ago by garry.yao

  • Cc changed from damo,joek to damo, joek
  • Status changed from assigned to pending

WFM, can you provide a sample page?

comment:7 in reply to: ↑ 6 Changed 5 years ago by satya

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 5 years ago by satya

comment:8 Changed 5 years ago by fredck

  • Status changed from pending to 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 5 years ago by Saare

comment:9 Changed 5 years ago by Saare

  • Owner changed from mani to Saare
  • Status changed from confirmed to review

comment:10 Changed 5 years ago by fredck

  • Status changed from review to review_passed

comment:11 Changed 5 years ago by fredck

  • Milestone set to CKEditor 3.6.1

comment:12 Changed 5 years ago by Saare

  • Resolution set to fixed
  • Status changed from review_passed to closed

Fixed with [6918].

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