Opened 15 years ago
Closed 15 years ago
#6117 closed Bug (fixed)
BIDI: Language direction is changing when we come out of Numbered/Bulleted list
| Reported by: | Satya Minnekanti | Owned by: | Tobiasz Cudnik |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.4.1 |
| Component: | General | Version: | 3.4 Beta |
| Keywords: | IBM | Cc: | Damian, joek |
Description
To reproduce the defect:
Scenario 1:
- Open Ajax sample.
- Type few lines of text, select all the lines & click on RTL icon in the Tool bar.
- See that the language direction of the lines are changed to Right and RTL Icon in the Tool bar is selected.
- Now select all the lines and click on Numbers/Bullets icon in the Tool bar.
- See that a Numbered/Bulleted list appears with RTL language direction.
- Now select all the list items and press Decrease Indent icon.
Expected Result:
Numbered/Bulleted list is removed but the Language direction of text in the lines should still be from Right to Left(RTL).
Actual Result:
Numbered/Bulleted list is removed but Language direction of text in in the lines is removed and they have no language direction set.
Scenario 2:
- Open Ajax sample.
- Type few lines of text, select all the lines & click on RTL icon in the Tool bar.
- See that the language direction of the lines are changed to Right and RTL Icon in the Tool bar is selected.
- Now select all the lines and click on Numbers/Bullets icon in the Tool bar.
- See that a Numbered/Bulleted list appears with RTL language direction.
- go to the last list item and press Enter Twice.
Expected Result:
See that you are out of Numbered/Bulleted but the direction of text in the New Paragraph should still be from Right to Left(RTL).
Actual Result:
we are out of Numbered/Bulleted list is removed but New Paragraph has no explicit Language direction set.
Attachments (2)
Change History (9)
comment:1 Changed 15 years ago by
| Milestone: | → CKEditor 3.4.1 |
|---|---|
| Owner: | set to Tobiasz Cudnik |
| Status: | new → assigned |
Changed 15 years ago by
| Attachment: | 6117.patch added |
|---|
comment:2 Changed 15 years ago by
| Status: | assigned → review |
|---|
comment:3 Changed 15 years ago by
comment:4 Changed 15 years ago by
| Status: | review → review_failed |
|---|
The direction CSS should also be considered.
Changed 15 years ago by
| Attachment: | 6117_2.patch added |
|---|
comment:5 Changed 15 years ago by
| Status: | review_failed → review |
|---|
comment:6 Changed 15 years ago by
| Status: | review → review_passed |
|---|
comment:7 Changed 15 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | review_passed → closed |
Fixed with [5857].

The fix looks like correct, but I'm not able to test it due to #6138. We need to fix that one first.