Ticket #3830 (closed Bug: fixed)

Opened 5 years ago

Last modified 5 years ago

Multi-cell selection does not work

Reported by: damo Owned by: garry.yao
Priority: Normal Milestone: CKEditor 3.0
Component: General Version:
Keywords: Confirmed IBM Review+ Cc:

Description

When selecting a cell range in a table, the Cell Properties dialog should modify the properties of all cells in the range. Currently it only does for one cell.

Attachments

3830.patch (2.9 KB) - added by garry.yao 5 years ago.
3830_2.patch (3.1 KB) - added by garry.yao 5 years ago.
3830_3.patch (2.7 KB) - added by garry.yao 5 years ago.

Change History

comment:1 Changed 5 years ago by garry.yao

  • Keywords Confirmed added

comment:2 Changed 5 years ago by garry.yao

  • Status changed from new to closed
  • Resolution set to wontfix

Cell Properties, as an 'editing' command, as other similar commands, is designed to deal with one element each time, which will result in ambiguous result when applied to multiple, currently we would keep the current feature as it is.

comment:3 Changed 5 years ago by damo

This works as described in the ticket on FCKEditor 2.6. Why would CKEditor 3.0 be different?

comment:4 Changed 5 years ago by fredck

  • Status changed from closed to reopened
  • Resolution wontfix deleted

I agree with damo. This is a required feature.

comment:5 Changed 5 years ago by garry.yao

  • Status changed from reopened to new
  • Owner set to garry.yao

Changed 5 years ago by garry.yao

comment:6 Changed 5 years ago by garry.yao

  • Keywords Review? added
  • Status changed from new to assigned

comment:7 Changed 5 years ago by martinkou

  • Keywords Review+ added; Review? removed

comment:8 Changed 5 years ago by martinkou

Please fix the curly bracket at tabletools plugin line 428 before committing - it should go to the next line.

comment:9 Changed 5 years ago by fredck

  • Keywords Review- added; Review+ removed

Moving onShow and onOk out of the specific plugin is really strange. It sounds like a hack. What if I need another plugin that interact with the selected cell? Should I change tabletools again then?

Changed 5 years ago by garry.yao

comment:10 Changed 5 years ago by garry.yao

  • Keywords Review? added; Review- removed

comment:11 Changed 5 years ago by martinkou

  • Keywords Review+ added; Review? removed

comment:12 Changed 5 years ago by fredck

  • Keywords Review- added; Review+ removed

The changelog file has an incorrect change at line 1.

Changed 5 years ago by garry.yao

comment:13 Changed 5 years ago by garry.yao

  • Keywords Review? added; Review- removed

comment:14 Changed 5 years ago by fredck

  • Keywords Review+ added; Review? removed

comment:15 Changed 5 years ago by garry.yao

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

Fixed with [3774]. Click here for more info about our SVN system.

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