Opened 17 years ago
Closed 17 years ago
#1502 closed Bug (fixed)
Remove format doesn't remove attributes
| Reported by: | Alfonso Martínez de Lizarrondo | Owned by: | Alfonso Martínez de Lizarrondo |
|---|---|---|---|
| Priority: | Normal | Milestone: | FCKeditor 2.6 |
| Component: | Core : Styles | Version: | FCKeditor 2.5 Beta |
| Keywords: | Confirmed Review+ | Cc: |
Description
The "Remove format" introduced in #75 doesn't remove any attribute, so it leaves the class and styles, so in the end the result is almost no clean up at all.
This is a list of attributes that should be cleaned at least to have some clean output:
- class / className
- style
- lang
- width
- height
- align
- v:shapes
- x:str
- x:num
- hspace
- valign
Attachments (2)
Change History (6)
comment:1 Changed 17 years ago by
| Keywords: | Confirmed added |
|---|
comment:2 Changed 17 years ago by
| Keywords: | Review? added |
|---|---|
| Owner: | set to Alfonso Martínez de Lizarrondo |
| Status: | new → assigned |
The patch does take care of removing that list of attributes (it's configurable in fckconfig.js) when the remove Format is executed, so the output code is really clean.
comment:3 Changed 17 years ago by
| Keywords: | Review+ added; Review? removed |
|---|
comment:4 Changed 17 years ago by
| Milestone: | → FCKeditor 2.6 |
|---|---|
| Resolution: | → fixed |
| Status: | assigned → closed |
Fixed with [1603]
Note: See
TracTickets for help on using
tickets.
Confirmed in IE and FF2 using version 2.5 SVN Build 18185