Opened 17 years ago
Closed 16 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 17 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 16 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