Opened 11 years ago
Last modified 10 years ago
#11767 confirmed Bug
Table Tools spanning not updated
Reported by: | Eugen | Owned by: | |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | General | Version: | 3.0 |
Keywords: | Cc: |
Description
Testcase:
Use IE or FF to be able to easily visualize the issue, in the source its broken for all browsers thogh
- Go on http://ckeditor.com/demo
- Create a table with 4x4
- select Cell 1x1 and set row-spanning to 10
- You now have a broken spanning since it is not validated
A more common use-case for normal users would be
- Go on http://ckeditor.com/demo
- Create a table with 4x4
- 3. select Cell 1x1 and set row-spanning to 10
- Delete a row -> spanning is not set to 9, instead of getting recoundet and fixed
Basically there is no sense in not validating the data since this is an editor for a user, not developer. In a table with a lot of rows you can easily count wrong and end up having a broken table.
Solution:
a) caluculate max span for col/rowspan before entering the dialog and offer a select-widget rather then an input field b) validate on save
You could also reuse such a table validator: http://jsfiddle.net/UkV35/8/
Thanks for the validation code snippet. We will definitely need to introduce something like that for tables.