Opened 13 years ago
Closed 12 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 13 years ago by
Status: | new → confirmed |
---|---|
Version: | 3.4 → 3.0 |
comment:2 Changed 12 years ago by
Component: | Core : BiDi → Core : Lists |
---|
comment:3 Changed 12 years ago by
Cc: | satya_minnekanti@… added |
---|---|
Keywords: | IBM added |
comment:4 Changed 12 years ago by
Changed 12 years ago by
Attachment: | 7448.patch added |
---|
comment:5 Changed 12 years ago by
Owner: | set to Garry Yao |
---|---|
Status: | confirmed → review |
comment:8 Changed 12 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 12 years ago by
Milestone: | → CKEditor 3.6.2 |
---|---|
Status: | review → review_passed |
comment:11 Changed 12 years ago by
Resolution: | → fixed |
---|---|
Status: | review_passed → closed |
Fixed with [7210].
Merge list logic should now consider text direction.