Opened 14 years ago
Closed 14 years ago
#8087 closed Bug (fixed)
RTL: Indenting list adds extra dir attributes to list items.
Reported by: | Wiktor Walc | Owned by: | Garry Yao |
---|---|---|---|
Priority: | Normal | Milestone: | CKEditor 3.6.2 |
Component: | Core : Lists | Version: | 3.6.1 |
Keywords: | Cc: | satya_minnekanti@… |
Description
Caused by [6928].
- Start with
<ul dir="rtl"> <li> one</li> <li> two</li> <li> three</li> <li> four</li> </ul>
- Select two list items in the middle ("two" and "three").
- Press "Increase Indent"
- Result:
<ul dir="rtl"> <li> one <ul> <li dir="rtl"> two</li> <li dir="rtl"> three</li> </ul> </li> <li> four</li> </ul>
Previously the result was:
<ul dir="rtl"> <li> one <ul> <li> two</li> <li> three</li> </ul> </li> <li> four</li> </ul>
Attachments (3)
Change History (14)
comment:1 Changed 14 years ago by
Status: | new → confirmed |
---|
comment:2 Changed 14 years ago by
Owner: | set to Garry Yao |
---|---|
Status: | confirmed → review |
Changed 14 years ago by
Attachment: | 8087.patch added |
---|
comment:4 Changed 14 years ago by
comment:5 Changed 14 years ago by
Cc: | satya_minnekanti@… added |
---|
comment:6 Changed 14 years ago by
Milestone: | CKEditor 3.6.1 → CKEditor 3.6.2 |
---|
Unless anything more serious will be found while working with lists, we can correct it in the next release.
The text direction is correct, so let's work on it straight after the release to keep the resulting HTML code as clean as possible.
Changed 14 years ago by
Attachment: | 8087_2.patch added |
---|
comment:8 Changed 14 years ago by
Keywords: | NeedsTest added |
---|
comment:9 Changed 14 years ago by
Changed 14 years ago by
Attachment: | 8087_3.patch added |
---|
comment:10 Changed 14 years ago by
Keywords: | NeedsTest removed |
---|
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 [7219].
At least #6861, #8042, #8053 must be re-checked.