Opened 18 years ago
Closed 15 years ago
#617 closed Bug (wontfix)
Table: TD should use style to set borderColor
Reported by: | Geir Helge Tjøstolvsen-Schmidt | Owned by: | Martin Kou |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | General | Version: | FCKeditor 2.4.3 |
Keywords: | Review- | Cc: |
Description
When selecting borderColor for a TD it's set as an attribute. There's no borderColor attribute for the TD element (works in most browsers, but not valid). Style="bordercolor:#xxxxxx" should be used instead.
Attaching suggested fix
Attachments (3)
Change History (16)
Changed 18 years ago by
Attachment: | fck_tablecell.html.patch added |
---|
comment:1 Changed 17 years ago by
Milestone: | → FCKeditor 2.6 |
---|---|
Owner: | set to Martin Kou |
Status: | new → assigned |
comment:2 Changed 17 years ago by
Keywords: | HasPatch added |
---|
comment:3 Changed 17 years ago by
Note on this: it must be backward compatible, looking for the color value in the borderColor attribute on load, and removing the attribute when setting the style on save.
comment:4 Changed 17 years ago by
The proposed patch does not work under Firefox, so I'm proposing another patch.
comment:5 Changed 17 years ago by
Keywords: | Review? added; HasPatch removed |
---|
comment:6 Changed 17 years ago by
The problem from my point of view would be that if someone pasted content from another source, usually Word, (or manually edited the source), and that cell have a custom border like different colors for each border or width!=1px or style!=solid, then as soon as he edited the cell those properties would be lost.
I would rather have settings to also change the width and style of the border, and being able to set all the borders the same or each one independently.
But in fact I would rather have a style editor instead of putting everything in the main dialog for the cell (because people also want to customize color, background-color, background-image...)
comment:7 Changed 17 years ago by
Keywords: | Review? removed |
---|
Another problem I just found is that different browsers are giving out vastly different HTML outputs for the border styles. Firefox gives the cleanest HTML output, while Safari attaches a huge assortment of CSS styles to the HTML output.
comment:8 follow-up: 10 Changed 17 years ago by
Keywords: | Review? added |
---|
I've added an updated patch for fixing this bug. The changes are moderately extensive for a seemingly simple bug.
The updated patch improves 2 things:
- If there's any already existing border styles in the table cell ( e.g. a dashed 2px border), the non-color border styles will not be destroyed.
- The HTML output of border styles is now the same across all browsers.
comment:10 Changed 17 years ago by
Keywords: | Review- added; Review? removed |
---|
Replying to martinkou:
The changes are moderately extensive for a seemingly simple bug.
You are correct Martin, and the extension of the changes could bring more problems than benefits. I think we must review the strategy for this ticket.
To easily give a Review-, the changes on _GetMainXmlString should be moved to GetXHTML, together with all other string processing we are doing there. In any case, I would ask a hold in this ticket for better understanding over the current patch.
In any case, this ticket has been mistakenly targeted to the 2.6. I'm re-targeting it to the 2.7, which is related to the table support review.
comment:11 Changed 17 years ago by
Milestone: | FCKeditor 2.6 → FCKeditor 2.7 |
---|
comment:12 Changed 17 years ago by
comment:13 Changed 15 years ago by
Resolution: | → wontfix |
---|---|
Status: | assigned → closed |
Fixed in CKEditor with #4893
Suggested fix