Opened 14 years ago
Closed 14 years ago
#8305 closed Bug (fixed)
Adding a LTR Paragraph to RTL List changing language direction of list
| Reported by: | Satya Minnekanti | Owned by: | Garry Yao |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.6.3 |
| Component: | Core : Lists | Version: | 3.0 |
| Keywords: | IBM | Cc: | Damian, Teresa Monahan |
Description
- Paste the following code in to souce view
<ol dir="rtl"> <li> first list item</li> </ol> <p dir="ltr"> LTR Paragraph </p>
- Go back to rich text keep cursor in LTR Paragraph and click on Numbered List button.
Expected Result: List remains RTL and dir=rtl applied to second list item.
<ol dir="rtl"> <li> first list item</li> <li dir="ltr"> LTR Paragraph</li> </ol>
Actual Result: List becomes LTR
<ol> <li> first list item</li> <li dir="ltr"> LTR Paragraph</li> </ol>
Attachments (1)
Change History (6)
comment:1 Changed 14 years ago by
| Status: | new → confirmed |
|---|---|
| Version: | → 3.0 |
comment:2 Changed 14 years ago by
Changed 14 years ago by
| Attachment: | 8305.patch added |
|---|
comment:3 Changed 14 years ago by
| Owner: | set to Garry Yao |
|---|---|
| Status: | confirmed → review |
New impl handles both #7448 and this case, also, with an additional change that perserve the root element of existing list instead of the newly created one when merging, in short, merge new to old, this should be more intuitive.
Because of the relativity, new tcs are added back to the origin tt:
comment:4 Changed 14 years ago by
| Component: | General → Core : Lists |
|---|---|
| Milestone: | → CKEditor 3.6.3 |
| Status: | review → review_passed |
comment:5 Changed 14 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | review_passed → closed |
Fixed with [7403].

Reproducible in all browsers from CKE 3.0.