Opened 9 years ago
Closed 9 years ago
#14721 closed Bug (expired)
convertToPx function in RTL mode
| Reported by: | Irina | Owned by: | Tade0 |
|---|---|---|---|
| Priority: | Normal | Milestone: | |
| Component: | General | Version: | 4.0 |
| Keywords: | IBM | Cc: | Satya Minnekanti |
Description
In convertToPx function style attribute is always set to left even when editor is in RTL mode, e.g.
calculator = CKEDITOR.dom.element.createFromHtml( '<div style="position:absolute;left:-9999px;' + 'top:-9999px;margin:0px;padding:0px;border:0px;"' + '></div>', CKEDITOR.document );
Should this be replaced with 'right' for RTL mode?
Change History (9)
comment:1 Changed 9 years ago by
| Status: | new → confirmed |
|---|---|
| Version: | 4.5.10 (GitHub - master) → 4.0 |
comment:2 Changed 9 years ago by
| Milestone: | → CKEditor 4.6.2 |
|---|---|
| Priority: | Normal → Nice to have (we want to work on it) |
comment:3 Changed 9 years ago by
| Owner: | set to Tade0 |
|---|---|
| Status: | confirmed → assigned |
comment:4 Changed 9 years ago by
| Status: | assigned → review |
|---|
Interestingly this function had no test coverage, so I added them.
As mentioned earlier in principle this shouldn't be dependant on LTR-RTL, so I simply added a "1" before each value.
Changes pushed to branch:t/14721.
comment:5 Changed 9 years ago by
| Status: | review → assigned |
|---|
While reviewing I saw that it should be pretty much language direction-independent. This element is inserted directly into the body and is positioned absolutely so moving it outside left border of body will take effect on both ltr and rtl languages.
@irina can I ask you to provide a case where it fails?
comment:6 Changed 9 years ago by
| Status: | assigned → pending |
|---|
comment:7 Changed 9 years ago by
| Milestone: | CKEditor 4.6.2 |
|---|
comment:8 Changed 9 years ago by
| Priority: | Nice to have (we want to work on it) → Normal |
|---|
comment:9 Changed 9 years ago by
| Resolution: | → expired |
|---|---|
| Status: | pending → closed |

Goal of this function is to convert non-px to px but it makes sense to move it to the right in RTL mode so as to not show it by accident.
Maybe the values should be made bigger. The 9999px may be too small these days.