Ticket #5404 (closed New Feature: fixed)
FillEmptyBlocks in CKEditor 3.x
| Reported by: | wwalc | Owned by: | garry.yao |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.5 |
| Component: | Core : Output Data | Version: | |
| Keywords: | Cc: |
Description
It looks like FillEmptyBlocks option is not available in CKEditor 3.x, thus it is impossible to configure CKEditor to leave empty elements without nbsp; inside of it.
Attachments
Change History
comment:1 Changed 3 years ago by garry.yao
- Status changed from new to assigned
- Keywords Confirmed Review? added
- Component changed from General to Core : Output Data
- Owner set to garry.yao
- Milestone set to CKEditor 3.4
comment:4 Changed 3 years ago by garry.yao
- Status changed from review to assigned
With the new patch, fillEmptyBlocks config could be even a mediate function that allows fully controlling over empty blocks (a community feature request).
comment:6 Changed 3 years ago by fredck
- Status changed from review to review_failed
- This setting bus be enabled by default.
- The documentation syntax is wrong (@type doesn't accept description and there is semicolon after @name).
comment:7 Changed 3 years ago by fredck
Btw... the fix is targeted to the 3.5, but it looks like the patch has been made for trunk.
comment:8 Changed 3 years ago by fredck
Also... better to commit #6248 and update the 3.5.x branch before the new patch, to avoid conflicts later.
comment:9 Changed 3 years ago by garry.yao
- Status changed from review_failed to review
- Version 3.0 deleted
comment:10 Changed 2 years ago by Saare
- Keywords Confirmed removed
- Status changed from review to review_passed
Two minor things:
- L517 could be removed.
- Let's add to the documentation the fact that we're passing our CKEDITOR.htmlParser.element representation there, and not CKEDITOR.dom.element.
comment:12 Changed 2 years ago by tobiasz.cudnik
- Status changed from review to review_failed
Using following content:
<p> foo</p> <p> </p> <p> bar</p>
Second paragraph is not visible and thus not editable inside WYSIWYG area.
comment:13 Changed 2 years ago by garry.yao
- Status changed from review_failed to review
You're right, this option should have NO impact on wysiwyg.
comment:14 Changed 2 years ago by tobiasz.cudnik
- Status changed from review to review_passed
It's OK now, just change the example inside the JSDOC comment, as it causes an error for elements without a class.
comment:15 Changed 2 years ago by garry.yao
- Status changed from review_passed to closed
- Resolution set to fixed
Fixed with [6138].
