Opened 16 years ago
Closed 16 years ago
#2430 closed New Feature (fixed)
Support THEAD in Tables
Reported by: | Alfonso Martínez de Lizarrondo | Owned by: | Alfonso Martínez de Lizarrondo |
---|---|---|---|
Priority: | Normal | Milestone: | FCKeditor 2.6.4 |
Component: | General | Version: | FCKeditor 2.3.2 |
Keywords: | Review+ | Cc: |
Description
Take the patch of https://sourceforge.net/tracker/index.php?func=detail&aid=1409409&group_id=75348&atid=543655 review it and update to current version.
Attachments (6)
Change History (16)
Changed 16 years ago by
Attachment: | fck_table.html added |
---|
comment:1 Changed 16 years ago by
I've updated the patch, fixing the issues that it had with IE and using some of the new FCKDomTools functions.
It's still missing the text entries in the language files and the corresponding what's new, but it should be enough to check that the code works as expected.
Does anyone have any suggestion or problem related to this patch?
comment:2 Changed 16 years ago by
Keywords: | Review? added |
---|
I've modified the patch including the language files, and I've changed the UI, instead of using two checkboxes I think that it might be easier for the user to have a single select box with the available combination of headers.
I see that I've forgot about the what's new entry, it should be something like: New features: In the table dialog it's possible to create header cells in the first row (included in a thead element) or the first column of the table.
comment:3 Changed 16 years ago by
Keywords: | Review- added; Review? removed |
---|
The patch has a few problems that need to be fixed:
- Create a default 2x3 table -> open table properties to set its first column to header -> open the table properties dialog again to set its first row to header. You'll see the top-left cell of the table remains a regular <td> cell. The culprit here is line 234 to line 242 in fck_table.html.
- Line 196 should be th.scope = 'row' instead of th.scope = 'col', right?
- The function FCKDomTools.RenameNode() is only used by fck_table.html so far. It seems to be more appropriate to define it inside fck_table.html to reduce core code size.
- The changes to fckconfig.js shouldn't be in the patch.
comment:4 Changed 16 years ago by
Keywords: | Review? added; Review- removed |
---|
- I've changed the logic so it only changes cells inside the tbody element(s).
- I think that line 196 was right, the problem was line 229, so that it matches the scopes generated if the headers are set at creation time.
- The idea is to reuse the function for the cell dialog to switch between td and th, but I've moved to fck_dialog_common.js so it isn't in the core anymore.
- Sorry, I didn't see it. I've attached the what's new changes this time.
comment:5 Changed 16 years ago by
Keywords: | Review? removed |
---|
ups, now something it's failing in IE...
comment:6 Changed 16 years ago by
Keywords: | Review? added |
---|
I've fixed a big problem in IE that placed the TH at the end of the row, and a little detail in Opera that sometimes it did include an extra empty <tbody>
comment:7 Changed 16 years ago by
I've started looking at the TH option for the cell dialog and based on the previous patch by Martinkou I've remembered that now I can use the existing function to copy all attributes properly.
comment:8 Changed 16 years ago by
Keywords: | Review+ added; Review? removed |
---|
comment:9 Changed 16 years ago by
Milestone: | → FCKeditor 2.6.4 |
---|
Previous patch