#6639 closed Bug (fixed)
Pasting from Word a nested list with an empty paragraph doesn't work
| Reported by: | Wiktor Walc | Owned by: | Garry Yao |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.5.1 |
| Component: | Plugin : Paste from Word | Version: | 3.1 |
| Keywords: | Cc: |
Description
When there is a list with another nested list, that contains an empty paragraph created with Shift+Enter, pasting from Word generates an invalid result.
Word document:
1. One
a. Nested list 1
b. Nested list 2
c. Nested list 3
d. Nested list 4
2. Two
3. Three
4. Four
result after pasting from Word into CKEditor:
1.One
1.Nested list 1
2.Nested list 2
1.Nested list 3
2.Nested list 4
3.Two
4.Three
5.Four
Attachments (4)
Change History (17)
Changed 14 years ago by
| Attachment: | nested_list_with_empty_lines.docx added |
|---|
comment:1 Changed 14 years ago by
| Milestone: | CKEditor 3.6 → CKEditor 3.5.1 |
|---|
comment:2 Changed 14 years ago by
| Status: | new → confirmed |
|---|---|
| Version: | → 3.1 |
comment:3 Changed 14 years ago by
| Owner: | set to Tobiasz Cudnik |
|---|---|
| Status: | confirmed → assigned |
Changed 14 years ago by
| Attachment: | 6639.patch added |
|---|
comment:4 Changed 14 years ago by
| Status: | assigned → review |
|---|
The solution is really simple, although finding the reason weren't. I think there can be more places suffering from lack of the XML namespace support in our codebase...
Changed 14 years ago by
| Attachment: | 6639_2.patch added |
|---|
comment:5 Changed 14 years ago by
| Status: | review → review_failed |
|---|
I don't see the patch fix the problem, also, we need to keep namespaced elements intact with "cke" as prefix, see flash plugin.
comment:6 Changed 14 years ago by
| Owner: | changed from Tobiasz Cudnik to Garry Yao |
|---|---|
| Status: | review_failed → review |
comment:7 Changed 14 years ago by
| Status: | review → review_failed |
|---|
I'm not sure if we should mix up our own things in the dtd.
comment:8 Changed 14 years ago by
| Status: | review_failed → review |
|---|
Dollar prefixed dtd entries are all "none-generated" definitions that were added by us as additional semantics.
comment:9 Changed 14 years ago by
| Status: | review → review_failed |
|---|
We can inline the tag name for that, no need for another dtd entry.
Changed 14 years ago by
| Attachment: | 6639_3.patch added |
|---|
comment:10 Changed 14 years ago by
| Status: | review_failed → review |
|---|
comment:11 Changed 14 years ago by
| Status: | review → review_passed |
|---|
comment:12 Changed 14 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | review_passed → closed |
Fixed with [6317].
comment:13 Changed 11 years ago by
| Component: | Core : Pasting → Plugin : Paste from Word |
|---|
To not make the TC complicated, the bug is just about pasting the nested list, we must understand how line-break (SHIFT-ENTER) affects our list resolution.