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