Opened 16 years ago
Closed 16 years ago
#3576 closed Bug (fixed)
Style system incorrectly remove element
| Reported by: | Garry Yao | Owned by: | Garry Yao |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.3 |
| Component: | Core : Styles | Version: | SVN (CKEditor) - OLD |
| Keywords: | Confirmed Review+ | Cc: |
Description
If you're applying any 'span' based style on the following range:
<p>^this <span>tag</span> should not be removed.^</p>
Then the span will always be removed, which is wrong.
Attachments (3)
Change History (16)
comment:1 Changed 16 years ago by
| Keywords: | Review? added |
|---|---|
| Status: | new → assigned |
Changed 16 years ago by
| Attachment: | 3576.patch added |
|---|
comment:2 Changed 16 years ago by
| Milestone: | CKEditor 3.0 → CKEditor 3.1 |
|---|---|
| Priority: | High → Normal |
This doesn't look like a big issue really. We have the same behavior in V2 and we have never had complains about it.
I'm postponing this ticket so we can still think if the change is valid. If we have few code changes for it, we may even consider this.
comment:3 Changed 16 years ago by
| Keywords: | Review- added; Review? removed |
|---|
I'm still getting this issue on FF 3.5 with this result:
<p> <span style="background-color: rgb(255, 215, 0);">this tag should not be removed.</span></p>
comment:4 Changed 16 years ago by
Defer this fixing since at least span with any style/attribute won't be affected.
comment:5 Changed 16 years ago by
| Owner: | Garry Yao deleted |
|---|---|
| Status: | assigned → new |
comment:6 Changed 16 years ago by
| Milestone: | CKEditor 3.1 → CKEditor 3.2 |
|---|---|
| Version: | → SVN (CKEditor) |
comment:7 Changed 16 years ago by
| Milestone: | CKEditor 3.2 → CKEditor 3.3 |
|---|
comment:8 Changed 16 years ago by
| Owner: | set to Garry Yao |
|---|---|
| Status: | new → assigned |
Changed 16 years ago by
| Attachment: | 3576_2.patch added |
|---|
comment:9 Changed 16 years ago by
| Component: | General → Core : Styles |
|---|---|
| Keywords: | Review? added; Review- removed |
Align previous patch with trunk.
comment:10 Changed 16 years ago by
| Keywords: | Review- added; Review? removed |
|---|
Both Firefox 3.6 and IE8 remove the span in the testcase.
Changed 16 years ago by
| Attachment: | 3576_3.patch added |
|---|
comment:11 Changed 16 years ago by
| Keywords: | Review? added; Review- removed |
|---|
Adding the missed checks for 'removeEmpty', I wonder why browsers don't implement the return value of 'removeAttribute' :(
comment:12 Changed 16 years ago by
| Keywords: | Review+ added; Review? removed |
|---|
Beside the fix, the patch also align the styles TS to the current trunk.