Opened 12 years ago
Last modified 12 years ago
#10073 closed Bug
ACF should clean up empty elements — at Initial Version
Reported by: | Piotrek Koszuliński | Owned by: | |
---|---|---|---|
Priority: | Normal | Milestone: | CKEditor 4.1 RC |
Component: | General | Version: | |
Keywords: | Drupal | Cc: | wim.leers@… |
Description
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.