Opened 15 years ago
Closed 14 years ago
#7448 closed Bug (fixed)
Create list on RTL paragraph
| Reported by: | Garry Yao | Owned by: | Garry Yao |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.6.2 |
| Component: | Core : Lists | Version: | 3.0 |
| Keywords: | IBM | Cc: | satya_minnekanti@… |
Description
- Load editor with the following source and selection:
<ol> <li> ltr</li> </ol> <p dir="rtl"> rtl^</p>
- Click "Numbered List" button;
- Actual Result:
<ol dir="rtl"> <li> ltr</li> <li> rtl</li> </ol>
- Expected:
<ol> <li> ltr</li> <li dir="rtl"> rtl</li> </ol>
Attachments (1)
Change History (12)
comment:1 Changed 15 years ago by
| Status: | new → confirmed |
|---|---|
| Version: | 3.4 → 3.0 |
comment:2 Changed 14 years ago by
| Component: | Core : BiDi → Core : Lists |
|---|
comment:3 Changed 14 years ago by
| Cc: | satya_minnekanti@… added |
|---|---|
| Keywords: | IBM added |
comment:4 Changed 14 years ago by
Changed 14 years ago by
| Attachment: | 7448.patch added |
|---|
comment:5 Changed 14 years ago by
| Owner: | set to Garry Yao |
|---|---|
| Status: | confirmed → review |
comment:8 Changed 14 years ago by
Replying to garry.yao:
Ticket Test: run OR view source.
This test is currently PASSing on FF without patch, while the original TC can be confirmed manual. It makes it not reliable and must be verified.
comment:10 Changed 14 years ago by
| Milestone: | → CKEditor 3.6.2 |
|---|---|
| Status: | review → review_passed |
comment:11 Changed 14 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | review_passed → closed |
Fixed with [7210].

Merge list logic should now consider text direction.