Opened 9 years ago
Closed 8 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 8 years ago by
Milestone: | → CKEditor 4.6.2 |
---|---|
Priority: | Normal → Nice to have (we want to work on it) |
comment:3 Changed 8 years ago by
Owner: | set to Tade0 |
---|---|
Status: | confirmed → assigned |
comment:4 Changed 8 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 8 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 8 years ago by
Status: | assigned → pending |
---|
comment:7 Changed 8 years ago by
Milestone: | CKEditor 4.6.2 |
---|
comment:8 Changed 8 years ago by
Priority: | Nice to have (we want to work on it) → Normal |
---|
comment:9 Changed 8 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.