Opened 9 years ago
Last modified 9 years ago
#14319 confirmed Bug
BIDI: When locale is set to Arabic, Font Size combo displays Arabic-European digits
Reported by: | edithk | Owned by: | |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | General | Version: | 4.0 |
Keywords: | Cc: | tomerm@… |
Description
Steps to reproduce
Change the browser local to Arabic and reload CKEditor.
Expected result
In the toolbar, in the Font Size dropdown menu:
- text is displayed in Arabic
- digits are displayed in Arabic-Indic (٨, ٩)
Actual result
The name and the tooltip of the Font size dropdown are displayed in Arabic but the size digits are displayed in Arabic-European (8, 9).
Attachments (2)
Change History (4)
comment:1 Changed 9 years ago by
Summary: | When locale is set to Arabic, Font Size combo displays Arabic-European digits → BIDI: When locale is set to Arabic, Font Size combo displays Arabic-European digits |
---|
Changed 9 years ago by
Attachment: | FontSizeCombo.png added |
---|
Changed 9 years ago by
Attachment: | Arabic_numerals.png added |
---|
comment:2 Changed 9 years ago by
Status: | new → confirmed |
---|---|
Version: | 4.5.4 → 4.0 |
This is similar issue to #13473 and #14320. Until we find cross language solution we have to use European numbers. Fortunately European numbers can be recognized world-wide so current state can be treated as sort of workaround.
In order to introduce such change we would have to consider languages like (please see attached image): European, Arabic-Indic, Eastern Arabic-Indic, Devanagari and Tamil. Each of them uses different numbers which should be defined in language file. Font size plugin would have to use language codes for style definition names. St the moment they are hardcoded in European numbers.
NOTE: If we were to introduce language specific solution we need to remember that currently style names (e.g. 8px) are used as a backup for style value vars[ styleType ] = values[ i ] = parts[ 1 ] || name;
. This could lead to problems so this is something to look out for.
screenshot to demonstrate Font Size combo