Opened 16 years ago
Closed 16 years ago
#2854 closed New Feature (fixed)
Move table dialog to trunk.
Reported by: | Martin Kou | Owned by: | Martin Kou |
---|---|---|---|
Priority: | Normal | Milestone: | CKEditor 3.0 |
Component: | UI : Dialogs | Version: | SVN (FCKeditor) - Retired |
Keywords: | Review+ | Cc: |
Description
The table dialog didn't make it to v3 beta. It needs to be moved to the trunk with a review to both its UI and logic.
Attachments (3)
Change History (10)
comment:1 Changed 16 years ago by
Status: | new → assigned |
---|
comment:2 Changed 16 years ago by
Keywords: | Review? added |
---|
Quite a number of things were fixed in the patch:
- The table dialog is refactored to use the setup/commit system.
- Changing table alignment no longer crashes in IE.
- Changing cell padding and spacing are now effective immediately in IE.
- It's now possible to create table within table. If you want to test editing table properties, change the command for the table button in plugin.js to "tableProperties".
- Visual grouping of fields is improved.
- It's now possible to create and edit table headers as in v2.6.4.
- Fixed the issue where the dialog does not detect a selected table when a table as a control in IE.
- Fixed an issue in the wysiwygarea plugin which causes insertion of block elements inside a block limit element (e.g. inserting a table inside a td) to raise JavaScript errors.
Changed 16 years ago by
Attachment: | 2854.patch added |
---|
Changed 16 years ago by
Attachment: | 2854_2.patch added |
---|
comment:4 Changed 16 years ago by
Keywords: | Review- added; Review? removed |
---|
- As stated on #2792, we must not have the "defaultValues" configuration section. Any kind of customization in this sense must be done by using the JavaScript API.
- The dialogs are getting registered twice, one at plugin.js and again at table.js. Why is that done in this way?
- Most of the fields in the dialog are quite simple and their logic could be easily separated inside each UI element. There was no need to use the setupValue and commitValue artifact for all of them, as it just makes the dialog logic hard to follow. The dialog is functional, and we can even commit it as is, but this is really bad coding.
comment:5 Changed 16 years ago by
Keywords: | Review? added; Review- removed |
---|
Fixed the following issues:
- Removed all defaultValues in configuration.
- Moved all setup logic to setup() functions and most commit logic to commit() functions. The following 3 commit items are still in onOk() because of interdependency issues:
- Rows and columns
- Table headers, which depends on rows and columns
- Width and height, which are interdependent
- Simplified the validation logic in table dialog.
- Added alert messages when validation fails.
- Fixed the issue where the table caption cannot be cleared.
- Fixed the issue where table width cannot be set to percentages.
Changed 16 years ago by
Attachment: | 2854_3.patch added |
---|
comment:6 Changed 16 years ago by
Keywords: | Review+ added; Review? removed |
---|
Attention, the patch contains an unresolved conflict on _source/plugins/toolbar/plugin.js. Be sure to fix that before committing.
comment:7 Changed 16 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Thanks for the notifications, the conflict is resolved.
Fixed with [3047].
Some issues to be fixed: