Opened 16 years ago
Closed 15 years ago
#2874 closed Bug (fixed)
tables: in FF caption seems to be in the wrong place
Reported by: | trumbitta | Owned by: | Martin Kou |
---|---|---|---|
Priority: | Normal | Milestone: | FCKeditor 2.6.5 |
Component: | General | Version: | FCKeditor 2.6.4 |
Keywords: | Confirmed HasPatch Review+ | Cc: |
Description
FCKeditor generates the following
<table ....> <thead> <tr> <th scope="col">1</th> <th scope="col">2</th> </tr> </thead> <caption>My Caption</caption> <tbody> <tr> <td>...</td> <td>...</td> </tr> </tbody> </table>
The following would be the correct markup
<table ....> <caption>My Caption</caption> <thead> <tr> <th scope="col">1</th> <th scope="col">2</th> </tr> </thead> <tbody> <tr> <td>...</td> <td>...</td> </tr> </tbody> </table>
Going further with my tests, I just found that the bug doesn't show up if you use MSIE7. My conclusions:
- the problem is strictly related to the browser and its "DOM engine"
OR
- the problem is inside the portion of code dedicated to gecko
Attachments (2)
Change History (11)
comment:1 Changed 16 years ago by
Keywords: | Confirmed HasPatch added |
---|
comment:2 Changed 16 years ago by
Owner: | set to Martin Kou |
---|---|
Status: | new → assigned |
Changed 16 years ago by
Attachment: | 2874.patch added |
---|
comment:3 Changed 16 years ago by
Keywords: | Review? added |
---|
comment:4 Changed 16 years ago by
Keywords: | Review+ added; Review? removed |
---|
comment:5 Changed 16 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Fixed with [3897].
Click here for more info about our SVN system.
comment:6 Changed 16 years ago by
Please re-open this one. Tested it in Firefox 3.5 but it still produces an invalid result.
To reproduce: Create a table with a caption but without any thead and th-cell. Then, edit the table and make the first row a thead filled with th-cells. In that case, the caption is placed after the thead.
Changed 16 years ago by
Attachment: | 2874_2.patch added |
---|
comment:8 Changed 16 years ago by
Resolution: | fixed |
---|---|
Status: | closed → reopened |
comment:9 Changed 15 years ago by
Resolution: | → fixed |
---|---|
Status: | reopened → closed |
@kwillems, thanks for the fix. Committed with [4229].
A fix for it has been proposed at our forums: http://www.fckeditor.net/forums/viewtopic.php?f=8&t=12852&p=34037#p34102