Opened 15 years ago

Closed 15 years ago

Last modified 15 years ago

#5728 closed Bug (fixed)

Text field & Upload Button in Upload Tab of Image Properties dialog are not shown Properly in Arabic

Reported by: Satya Minnekanti Owned by: Tobiasz Cudnik
Priority: Normal Milestone: CKEditor 3.4
Component: General Version: 3.2
Keywords: IBM Review+ Cc: Damian, joek

Description

To reproduce the defect

  1. open http://nightly.ckeditor.com/5508/_samples/ui_languages.html
  1. Click on Image Toolbar icon dialog and Click on Upload Tab on Image Properties dialog
  1. Take a Screen shot of it.
  1. Change the Language to Arabic and Click on Image Toolbar icon again and Click on Upload Tab on Image Properties dialog

Expected Result:

The Fields on the upload Tab of Image Properties should be shown as Exact Mirror Image of the same Fields in English Language.

Actual Result:

The Upload Text Field and Upload buttons on the upload Tab of Image Properties dialog are not shown as Mirror Image of the same Fields in English Language.

I am attaching the Screen shots of the Upload Tab of Image dialog in both English and Arabic

Find this issue on GitHub

Change History (10)

Changed 15 years ago by Satya Minnekanti

Changed 15 years ago by Satya Minnekanti

comment:1 Changed 15 years ago by Frederico Caldeira Knabben

We'll check it, but this field is rendered by the browser (<input type="file">), so it may work properly only on Arabic browsers.

comment:2 Changed 15 years ago by Damian

Is this not just a matter of adding dir="rtl" to both the file input and the enclosing form?

comment:3 Changed 15 years ago by Tobiasz Cudnik

Owner: set to Tobiasz Cudnik

Changed 15 years ago by Tobiasz Cudnik

Attachment: 5728_1.patch added

comment:4 Changed 15 years ago by Tobiasz Cudnik

Keywords: Review? added
Status: newassigned

Adding dir="rtl" to form makes file field reverted. I've added it additionally to iframe's HTML element along with lang attribute.

comment:5 Changed 15 years ago by Garry Yao

Keywords: Review+ added; Review? removed

Just wonder the necessity of "dir" on form element, please confirm before commit.

comment:6 Changed 15 years ago by Tobiasz Cudnik

Resolution: fixed
Status: assignedclosed

Fixed with [5673].

comment:7 Changed 15 years ago by Tobiasz Cudnik

Committed into correct branch with [5696].

Find this issue on GitHub
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