Ticket #6137 (closed Bug: fixed)

Opened 4 years ago

Last modified 4 years ago

Table dialog: width doesn't have the default value on second open

Reported by: fredck Owned by: Saare
Priority: Normal Milestone: CKEditor 3.4
Component: General Version: 3.4 Beta
Keywords: Cc:

Description

  1. Click the table button to create a new table. Note that the Width field presents the default "500" value.
  2. ESC to close the dialog.
  3. Click the table button again.

The Width field will be blank.

Works well with CKEditor 3.3.2.

Attachments

6137.patch (833 bytes) - added by Saare 4 years ago.
6137_2.patch (5.0 KB) - added by garry.yao 4 years ago.
6137_3.patch (5.7 KB) - added by Saare 4 years ago.

Change History

comment:1 Changed 4 years ago by Saare

  • Owner set to Saare
  • Status changed from new to assigned

Changed 4 years ago by Saare

comment:2 Changed 4 years ago by Saare

  • Status changed from assigned to review

comment:3 Changed 4 years ago by garry.yao

  • Status changed from review to review_failed

The patch provide a workaround for this particular field while it's still buggy when you have, e.g. a default table height.

Changed 4 years ago by garry.yao

comment:4 Changed 4 years ago by garry.yao

The patch is proposing the following change:

No 'change' event during internal fields committing and when fields resetting.

comment:5 Changed 4 years ago by fredck

#6148 has been marked as DUP.

comment:6 Changed 4 years ago by Saare

  • Status changed from review_failed to review

It's great seeing you here Garry.

comment:7 Changed 4 years ago by Saare

  • Status changed from review to review_failed

The change in the reset method causes problems with manual resetting.

Changed 4 years ago by Saare

comment:8 Changed 4 years ago by Saare

  • Status changed from review_failed to review

comment:9 Changed 4 years ago by fredck

  • Status changed from review to review_passed

comment:10 Changed 4 years ago by fredck

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

Fixed with [5822].

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