Opened 12 years ago

Closed 5 years ago

#444 closed New Feature (fixed)

Table: Editing cell property for several cells resets all values

Reported by: Geir Helge Tjøstolvsen-Schmidt Owned by:
Priority: Normal Milestone:
Component: General Version: 3.0
Keywords: Cc:

Description

I would like to start a discussion in order to find a better way to set and change cell properties in a table. With today's design you can't change one property for all cells without changing all the other available properties at the same time.

Example:

  1. Create a table with 6x6 cells/cols
  2. Set different background colours on each column
  3. Now you want to change the vertical alignment on all cells in the table. If you try to do that you will overwrite all background colours for all cells.

However, in some cases it's nice to be able to change the background colour for all cells in order to reset it.

I think the main problem is that you're able to change all the properties from the same window and therefore can't select cells with different properties to change just one of them.

Change History (6)

comment:1 Changed 12 years ago by Frederico Caldeira Knabben

Milestone: FCKeditor 3.0

There will be a complete review of the dialog system for version 3.0. We could think about finding a solution for it.

For example, in the example you have given, when opening the dialog, we should fill only the fields that have the same values for all cells. Then, when hitting ok, only those fields that have been changed would get applied. All other fields would be ignored. We should do that in a generic way for all dialogs.

comment:2 Changed 12 years ago by Wojciech Olchawa

Keywords: Confirmed added

comment:3 Changed 10 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.0CKEditor 3.x

comment:4 Changed 9 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.x

Milestone CKEditor 3.x deleted

comment:5 Changed 7 years ago by Jakub Ś

Version: FCKeditor 2.4.23.0

This is still an issue in CKEditor 4.x.

Use below code:

<table border="1" cellpadding="1" cellspacing="1" style="width: 500px;">
	<tbody>
		<tr>
			<td style="background-color: rgb(0, 153, 153);">&nbsp;</td>
			<td style="background-color: rgb(204, 255, 0);">&nbsp;</td>
			<td style="background-color: rgb(255, 51, 255);">&nbsp;</td>
		</tr>
		<tr>
			<td style="background-color: rgb(0, 153, 153);">&nbsp;</td>
			<td style="background-color: rgb(204, 255, 0);">&nbsp;</td>
			<td style="background-color: rgb(255, 51, 255);">&nbsp;</td>
		</tr>
		<tr>
			<td style="background-color: rgb(0, 153, 153);">&nbsp;</td>
			<td style="background-color: rgb(204, 255, 0);">&nbsp;</td>
			<td style="background-color: rgb(255, 51, 255);">&nbsp;</td>
		</tr>
	</tbody>
</table>

Select all cells, choose cell properties and set alignment.

Result: Colour from first column will be used for whole table. IMHO this is to aggressive and properties not changed in dialog should not be touched.

comment:6 Changed 5 years ago by Jakub Ś

Resolution: fixed
Status: confirmedclosed

I have checked latest CKEditor 4.4.6 and this seems to work fine now. This has most likely been fixed by #11439.

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