Opened 14 years ago
Closed 14 years ago
#7037 closed Bug (fixed)
Incorrect remove preformatted block
Reported by: | Garry Yao | Owned by: | Garry Yao |
---|---|---|---|
Priority: | Normal | Milestone: | CKEditor 3.5.1 |
Component: | Core : Styles | Version: | 3.5.1 |
Keywords: | Cc: |
Description
- Load with the following content and selection:
<p> pargraph[ one</p> <p> pargraph ]two</p>
- Open format combo and click on preformatted block;
- Click again to remove formatted block;
- Actual Result: Two paragraphs merged into one.
- Expected Result: The original content is reverted.
Attachments (2)
Change History (13)
comment:1 Changed 14 years ago by
Status: | new → confirmed |
---|
comment:2 Changed 14 years ago by
Milestone: | → CKEditor 3.5.1 |
---|
comment:3 Changed 14 years ago by
Owner: | set to Sa'ar Zac Elias |
---|---|
Status: | confirmed → assigned |
Changed 14 years ago by
Attachment: | 7037.patch added |
---|
comment:4 Changed 14 years ago by
Owner: | changed from Sa'ar Zac Elias to Garry Yao |
---|---|
Status: | assigned → review |
comment:5 Changed 14 years ago by
Status: | review → review_failed |
---|
In enter mode = BR a div element is left in the source, when preformatted element is removed (if it has no extra attributes it makes no sense to leave it there).
Changed 14 years ago by
Attachment: | 7037_2.patch added |
---|
comment:6 Changed 14 years ago by
Status: | review_failed → review |
---|
comment:7 Changed 14 years ago by
Status: | review → review_failed |
---|
Use the following:
<p> Pa[ra 1</p> <p> </p> <p> Pa]ra 2</p>
Apply preformatted style and remove it.
Expected: Original content is reverted.
Actual:
<p> Para 1</p> <p> Para 2</p>
comment:8 Changed 14 years ago by
Status: | review_failed → review |
---|
comment:10 Changed 14 years ago by
Status: | review → review_passed |
---|
(For reference, the very same can be reproduced when applying "normal" instead of removing the preformatted style).
comment:11 Changed 14 years ago by
Resolution: | → fixed |
---|---|
Status: | review_passed → closed |
Fixed with [6370].
Starting [6344].