Opened 9 years ago

Last modified 8 years ago

#13473 confirmed Bug

[FF] Help icon is not mirrored in RTL CKEditor

Reported by: Heba Aboulmagd Owned by:
Priority: Normal Milestone:
Component: Core : BiDi Version:
Keywords: Cc:

Description

  • Set the language of the browser to Arabic
  • open CKEditor

Expected Result: The CKEditor is mirrored and help icon is mirrored

Actual Result: The CKEditor is mirrored but the help icon is not mirrored

Attachments (1)

help icon.png (125.0 KB) - added by Heba Aboulmagd 9 years ago.
Help icon is not mirrored in RTL (Arabic) CKEditor

Download all attachments as: .zip

Change History (6)

Changed 9 years ago by Heba Aboulmagd

Attachment: help icon.png added

Help icon is not mirrored in RTL (Arabic) CKEditor

comment:1 Changed 9 years ago by Jakub Ś

Status: newconfirmed

Problem can be reproduced from CKEditor 3.0 however there is or might be a slight problem with applying this.

According to this link (plus many other resources) - the question mark (as well as comma) in written in Hebrew in LTR fashion - ?.

In other languages like Arabic (sample keyboard), Uyghur, Persian and Kurdish the question mark is written in RTL fashion - ؟.

As you can see this isn't just a matter of creating RTL icon. Ok, it should be created and used with RTL languages but there should be some special exception made for Hebrew.

comment:2 Changed 9 years ago by Heba Aboulmagd

The problem is fixed for Arabic language by adding the css class below

[lang|="ar"].cke_rtl .cke_buttonabout_icon {

-moz-transform: scaleX(-1); -o-transform: scaleX(-1); -webkit-transform: scaleX(-1); transform: scaleX(-1); filter: FlipH; -ms-filter: "FlipH";

}

this class will apply on Arabic languages and we can add other languages like Uyghur, Persian and Kurdish

However I am facing a problem with checking in the code and I am working on fixing it

comment:3 Changed 9 years ago by Piotrek Koszuliński

I've just closed https://github.com/ckeditor/ckeditor-dev/pull/196 because this isn't a code that we can accept in the core for the reasons that I listed in my comment. Taking into account also the problems mentioned in comment:1 I don't think that this can be fixed in core.

Version 0, edited 9 years ago by Piotrek Koszuliński (next)

comment:4 Changed 9 years ago by Jakub Ś

Perhaps there is a solution.

  1. We should definitely introduce about icon for RTL languages. This would solve the problem for Aravic, Uyghur, Persian and Kurdish languages.
  1. Such solution would however introduce problem for Hebrew language which uses LTR question mark. This where @heeba solution may come into play (we are transforming RTL icon into LTR for Hebrew only):
    [lang|="he"].cke_rtl .cke_button__about_icon_rtl
    {
        -moz-transform: scaleX(-1);
        -o-transform: scaleX(-1);
        -webkit-transform: scaleX(-1);
        transform: scaleX(-1);
        filter: FlipH;
        -ms-filter: "FlipH";
    }
    

comment:5 Changed 8 years ago by Jakub Ś

Similar issue #14320.

Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy