Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#16017 closed enhancement (wontfix)

Complex Expressions (URLs, emails,..etc) with Bidi text are not ordered correctly when direction is RTL

Reported by: Alia Naguib Owned by: Alia Naguib
Priority: undecided Milestone: tbd
Component: Dijit - Form Version: 1.6.0
Keywords: Cc: HKatory
Blocked By: Blocking:

Description

Locale: English

Browsers: IE8/Windows 7 & FF10/Windows 7

Procedure:

1- Set the direction to RTL (use dir="rtl")

2- Create a set of radio buttons

3- Create a set of check boxes

4- Set the labels of the radio buttons and check buttons with English and any bidi language.

Ex: - English (123) عربي 123

  • c:/Desktop/الحافظة الأولى‎/الحافظة الثانية‎/الملف‎.txt
  • First Choice: All English (123)
  • الاول‎.الثاني‎@السرفر‎.com

Expected Result:

1- Text maintains the same order even when the direction is set to right to left (RTL)

Actual Result:

1- The text is not ordered correctly. Brackets are misplaced and Arabic text is in reversed order

Ex: - c:/Desktop/الحافظة الأولى/الحافظة الثانية/الملف.txt

  • ( All English (123

Attachments (1)

CheckBox.js.patch (2.5 KB) - added by Douglas Hays 7 years ago.
patch from Hossam Katory (IBM, CCLA)

Download all attachments as: .zip

Change History (5)

comment:1 Changed 7 years ago by Douglas Hays

Owner: changed from Douglas Hays to Alia Naguib
Status: newpending

I'm not sure I understand. Isn't the label under the control of the web page author and not the CheckBox widget? So you would do something like:
<label for="cb1"><BDO dir="LTR">c:/Desktop/الحافظة الأولى‎/الحافظة الثانية‎/الملف‎.txt</BDO></label>

Changed 7 years ago by Douglas Hays

Attachment: CheckBox.js.patch added

patch from Hossam Katory (IBM, CCLA)

comment:2 Changed 7 years ago by Alia Naguib

Status: pendingnew

Yes, but it is a hassle for the author to modify each and every widget. Simply the "dir" attribute will be set for the whole form or checkbox group and the labels should be adjusted automatically without having to modify each one. Also, there is the case where the direction is inherited from other parent tags.

comment:3 Changed 7 years ago by Douglas Hays

Cc: HKatory added
Resolution: wontfix
Status: newclosed
Type: defectenhancement

The LABEL tag is not part of a widget but is native HTML and not under the control of any widget. This is an enhancement request that will cause code-bloat for everyone and only benefit a few labels with mixed-content.
bill, do you want this code added?

comment:4 Changed 7 years ago by bill

It comes back to #11611. It's not dijit's job to handle the LABEL tag at all, unless the label tag is part of the Checkbox widget. So let's keep this closed and just use that ticket.

Note: See TracTickets for help on using tickets.