Opened 15 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 15 years ago by
| Status: | new → confirmed |
|---|
comment:2 Changed 15 years ago by
| Owner: | set to Garry Yao |
|---|---|
| Status: | confirmed → review |
Changed 15 years ago by
| Attachment: | 8087.patch added |
|---|
comment:4 Changed 15 years ago by
comment:5 Changed 15 years ago by
| Cc: | satya_minnekanti@… added |
|---|
comment:6 Changed 15 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 15 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.