Opened 14 years ago
Closed 14 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 14 years ago by
Milestone: | → CKEditor 3.4.1 |
---|---|
Owner: | set to Tobiasz Cudnik |
Status: | new → assigned |
Changed 14 years ago by
Attachment: | 6117.patch added |
---|
comment:2 Changed 14 years ago by
Status: | assigned → review |
---|
comment:3 Changed 14 years ago by
comment:4 Changed 14 years ago by
Status: | review → review_failed |
---|
The direction CSS should also be considered.
Changed 14 years ago by
Attachment: | 6117_2.patch added |
---|
comment:5 Changed 14 years ago by
Status: | review_failed → review |
---|
comment:6 Changed 14 years ago by
Status: | review → review_passed |
---|
comment:7 Changed 14 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.