Ticket #5237 (closed Bug: fixed)

Opened 5 years ago

Last modified 4 years ago

English text in dialogs' title is flipped when using RTL language

Reported by: Saare Owned by: Saare
Priority: Normal Milestone: CKEditor 3.4.2
Component: UI : Language Version: 3.1
Keywords: IE Cc:

Description

To reproduce

  • Go to the demo and open the multi language interface.
  • Choose RTL language (i.e Hebrew)
  • Click on the 'Help' button for example.

Notice that the product name in the dialog title is flipped.

Attachments

5237.patch (1.6 KB) - added by Dekel 4 years ago.
5237_2.patch (905 bytes) - added by garry.yao 4 years ago.
5237_3.patch (868 bytes) - added by Saare 4 years ago.

Change History

comment:1 follow-up: ↓ 10 Changed 5 years ago by fredck

  • Keywords Confirmed IE added
  • Component changed from General to UI : Language
  • Milestone set to CKEditor 3.3

Confirmed with IE only.

We had issues with CSS in IE and because of this we were forced to make some "weird" things with dialog titles (weird solutions for weird browsers!). Because of this, ltr words are reverted.

We have solved this problem in the About dialog. In fact you can see that it works well for the Arabic language.

To make it work with Hebrew, it's enough to change the following entry in the about section of the language file:

dlgTitle : 'אודות rotidEKC',

But ok, this fixes the About dialog. Do you see any other dialog that needs fixes?

comment:2 follow-up: ↓ 3 Changed 5 years ago by Saare

Umm, SCAYT plugin for example.
I have recently translated everything and you applied it to the 3.2 version, so after this release i'll have a better look on that one.
Anyway, i think it should be fixed in the CSS for further compatibility

comment:3 in reply to: ↑ 2 Changed 5 years ago by fredck

Replying to Saare:

Anyway, i think it should be fixed in the CSS for further compatibility

I think so also... but really I remember we've spent a lot of time on this issue. If I recall it well it was breaking the dialog dragging in the title area with IE.

comment:4 Changed 5 years ago by Saare

OK, in Hebrew the problem is in all the SCAYT dialog (btw is there a way to translate the options inside of them?), Div container and About dialog.

comment:5 follow-up: ↓ 6 Changed 5 years ago by Saare

Oh, and the new 'Accessibility instructions' dialog.

comment:6 in reply to: ↑ 5 Changed 5 years ago by fredck

Replying to Saare:

Oh, and the new 'Accessibility instructions' dialog.

Isn't it translatable? I mean, do you leave it in English for the Hebrew language?

comment:7 Changed 5 years ago by Saare

Oops.. yes, i'll also translate this plugin with all the other language phrases that are to be translated, targeted to the 3.3 version release.
Anyway, if the problem is with the dragging, maybe if you cover the dialog title with a transparent DIV it will work OK.

Changed 4 years ago by Dekel

comment:8 Changed 4 years ago by Dekel

This problem should be resolved using CSS Only! patch attached.

comment:9 Changed 4 years ago by garry.yao

  • Keywords Pending added; Confirmed removed

I'm curious which browser do you have this reproduced? WFM in both IE7 and IE8.

comment:10 in reply to: ↑ 1 Changed 4 years ago by Saare

IE8. If you check this bug with Arabic in About dialog you will not observe this issue, as Fred wrote.

Changed 4 years ago by garry.yao

comment:11 Changed 4 years ago by garry.yao

  • Keywords Confirmed Review? added; Pending removed
  • Status changed from new to assigned
  • Owner set to garry.yao

but really I remember we've spent a lot of time on this issue. If I recall it well it was breaking >the dialog dragging in the title area with IE.

WFM in all IEs, I would revert the original hacks unless it's proved to be necessary.

comment:12 Changed 4 years ago by fredck

  • Keywords Review+ added; Review? removed

Much probably, the old hack is not anymore necessary because of the recent dialog structure changes.

comment:13 Changed 4 years ago by garry.yao

  • Status changed from assigned to closed
  • Resolution set to fixed

Fixed with [5454].

comment:14 Changed 4 years ago by Saare

  • Keywords Confirmed Review+ removed
  • Status changed from closed to reopened
  • Resolution fixed deleted
  • Milestone changed from CKEditor 3.3 to CKEditor 3.4.2

The same fix must be ported to other skins as well.

Changed 4 years ago by Saare

comment:15 Changed 4 years ago by Saare

  • Status changed from reopened to review
  • Owner changed from garry.yao to Saare

comment:16 Changed 4 years ago by tobiasz.cudnik

  • Status changed from review to review_passed

comment:17 Changed 4 years ago by Saare

  • Status changed from review_passed to closed
  • Resolution set to fixed

Fixed with [5897].

Note: See TracTickets for help on using tickets.
© 2003 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy