Opened 14 years ago
Closed 14 years ago
#5766 closed Bug (fixed)
‘ignoreEmptyParagraph’ behaves wrong
| Reported by: | Garry Yao | Owned by: | Garry Yao |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.4.2 |
| Component: | Core : Output Data | Version: | 3.0 |
| Keywords: | Confirmed | Cc: |
Description
Reproducing Procedures
- Open the 'replacebyclass' sample page;
- Create several new paragraphs at the end of document;
- Switch to 'source' mode and back multiple times.
- Actual Result: Each time with one paragraph lost at the end of document.
- Expected Result: All paragraphs remains.
Attachments (2)
Change History (10)
Changed 14 years ago by
| Attachment: | 5766.patch added |
|---|
comment:1 Changed 14 years ago by
| Owner: | set to Garry Yao |
|---|---|
| Status: | new → assigned |
| Version: | → 3.0 |
comment:2 Changed 14 years ago by
| Milestone: | CKEditor 3.3 → CKEditor 3.4 |
|---|
comment:3 Changed 14 years ago by
| Keywords: | Review? added |
|---|
comment:4 Changed 14 years ago by
| Milestone: | CKEditor 3.4 → CKEditor 3.5 |
|---|
comment:6 Changed 14 years ago by
New patch adds full page handling, the drawback now is the original 'strip last empty' behavior is lost, but maybe not a drawback at all, as the major principle of an text editor is not touching user's content unnoticed.
Changed 14 years ago by
| Attachment: | 5766_2.patch added |
|---|
comment:7 Changed 14 years ago by
| Status: | review → review_passed |
|---|
comment:8 Changed 14 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | review_passed → closed |
Fixed with [5960].
Note: See
TracTickets for help on using
tickets.
We should give priority, look at the affected version.