Opened 13 years ago
Last modified 13 years ago
#10073 closed Bug
ACF should clean up empty elements — at Version 2
| Reported by: | Piotrek Koszuliński | Owned by: | |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 4.1 RC |
| Component: | General | Version: | |
| Keywords: | Drupal | Cc: | wim.leers@… |
Description (last modified by )
TC:
- Create editor without image button (e.g. use one of editors in filter sample).
- Switch to source.
- Set:
<p><em><img src="" alt=""></em></p>. - Switch to wysiwyg mode.
- Check with dev tools that editable contains
<p><em></em></p>.
There are few cases:
<b><img></b>: after filtering out<img>entire<b>should be removed, because it's inDTD.$removeEmpty.<a href=""><img></a>is a different case because it's similar to<a name=""><img></a>and<a name=""></a>shouldn't be removed. So it's an exception, because 'a' isn't included in$removeEmpty. However, why doesn't data processor fix this case? It fixes the case I described in the next point, but why not this? Especially, that I see a filter's rule for htmlFilter that fixes this on output. I think that identical rule should exist for toHtml(). And perhaps previous point should also be handled by data processor...<p><img></p>}: after filtering out<img>filter will left empty<p></p>which then will be correctly handled by data processor, so in this case we don't have to act.
Important note: http://dev.ckeditor.com/ticket/9992#comment:7
Change History (2)
comment:1 Changed 13 years ago by
| Status: | new → confirmed |
|---|
comment:2 Changed 13 years ago by
| Description: | modified (diff) |
|---|
