Opened 7 years ago

Closed 7 years ago

#6155 closed Bug (fixed)

[[FF]] Modifying Table Header Properties by selecting first Row, causing several issues

Reported by: Satya Minnekanti Owned by: Tobiasz Cudnik
Priority: Normal Milestone: CKEditor 3.4.1
Component: General Version: 3.4 Beta
Keywords: IBM Cc: Damian, joek

Description

To reproduce the defect:

  1. open Ajax sample.
  1. Insert a Table with 3 rows & 2 columns.
  1. Select Cells in the first row & click on Table Properties option.
  1. Try to change Table Headers option to both and Click OK button.

Expected Result:

Both First Row and Column should be set as Headers.

Actual Result:

we are getting the following error in firebug when we press OK button first time.

t is null [Break on this error] if(/[\s\ufeff]/.test(D))z=null;else{G...)])t=t.getParent(),u=t.getIndex()+1;

we are getting below error every time we click OK button from second time onwards

u is undefined [Break on this error] if(r.equals(t.getPrevious())){o.startO...kground-color: $color !important;}';

Attachments (2)

6155.patch (892 bytes) - added by Tobiasz Cudnik 7 years ago.
6155_2.patch (892 bytes) - added by Tobiasz Cudnik 7 years ago.

Download all attachments as: .zip

Change History (9)

comment:1 Changed 7 years ago by Sa'ar Zac Elias

Milestone: CKEditor 3.4.1
Status: newconfirmed

comment:2 Changed 7 years ago by Tobiasz Cudnik

Owner: set to Tobiasz Cudnik
Status: confirmedassigned

Changed 7 years ago by Tobiasz Cudnik

Attachment: 6155.patch added

comment:3 Changed 7 years ago by Tobiasz Cudnik

Status: assignedreview

comment:4 Changed 7 years ago by Sa'ar Zac Elias

Status: reviewreview_failed

The name of the attribute has just been changed to _cke_bookmark.

Changed 7 years ago by Tobiasz Cudnik

Attachment: 6155_2.patch added

comment:5 Changed 7 years ago by Tobiasz Cudnik

Status: review_failedreview

comment:6 Changed 7 years ago by Sa'ar Zac Elias

Status: reviewreview_passed

comment:7 Changed 7 years ago by Tobiasz Cudnik

Resolution: fixed
Status: review_passedclosed

Fixed with [5849].

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