Opened 17 years ago
Closed 17 years ago
#1607 closed Task (fixed)
Remove FCKConfig.GeckoUseSPAN
| Reported by: | Alfonso Martínez de Lizarrondo | Owned by: | |
|---|---|---|---|
| Priority: | Normal | Milestone: | FCKeditor 2.6 |
| Component: | Core : Styles | Version: | FCKeditor 2.5 |
| Keywords: | Review+ | Cc: |
Description
With the new Style system the FCKConfig.GeckoUseSPAN setting shouldn't affect the output at all, so it can be removed.
Attachments (1)
Change History (6)
Changed 17 years ago by
| Attachment: | 1607.patch added |
|---|
comment:1 Changed 17 years ago by
| Keywords: | Review? added |
|---|---|
| Milestone: | → FCKeditor 2.6 |
I've attached the proposed patch. It removes the configuration entries and initializes the editing mode of Gecko to don't use CSS (we will take care of it).
comment:2 Changed 17 years ago by
| Keywords: | Review+ added; Review? removed |
|---|---|
| Resolution: | → fixed |
| Status: | new → closed |
Fixed with [1170].
Click here for more info about our SVN system.
comment:3 Changed 17 years ago by
| Resolution: | fixed |
|---|---|
| Status: | closed → reopened |
Is there any reason for still calling the styleWithCSS/useCSS commands? They should make no difference for us.
comment:4 Changed 17 years ago by
I've leave them there just in case the code generated by Gecko with CSS differs slightly from what is expected currently and maybe it does something in the future that isn't handled (I'm expecting that they could only change the behavior of StyleWithCSS, changing the output if it is disabled will bring them lots of bug reports quickly).
But if we are confident about the capabilities of the Style system then we can remove them.
comment:5 Changed 17 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | reopened → closed |
Proposed patch