Opened 7 years ago

Closed 7 years ago

Last modified 6 years ago

#15945 closed defect (invalid)

Programmatically created dijit.InlineEditbox is not mirrored correctly

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

Description

Locale: English Browsers: IE8/Windows 7, FF10/Windows 7

Procedure:

1- Set the direction of the programmatically created inlineEditBox to right to left (use dir="rtl")

2- Click on the text to do some editing

Actual Result:

  • Once editing is activated, the text is automatically left aligned

Expected Result:

  • Since RTL direction is specified, then whether in edit mode or not the text should remain right aligned

Attachments (3)

InlineEditBoxDefect.jpg (42.4 KB) - added by Alia Naguib 7 years ago.
InlineEditBox.js.patch (444 bytes) - added by Douglas Hays 7 years ago.
patch from Hossam Katory (IBM, CCLA)
test_InlineEditBox.html (13.3 KB) - added by Alia Naguib 7 years ago.

Download all attachments as: .zip

Change History (7)

Changed 7 years ago by Alia Naguib

Attachment: InlineEditBoxDefect.jpg added

comment:1 Changed 7 years ago by bill

Component: GeneralDijit
Owner: set to Alia Naguib
Status: newpending

comment:2 Changed 7 years ago by trac-o-bot

Resolution: invalid
Status: pendingclosed

Because we get so many tickets, we often need to return them to the initial reporter for more information. If that person does not reply within 14 days, the ticket will automatically be closed, and that has happened in this case. If you still are interested in pursuing this issue, feel free to add a comment with the requested information and we will be happy to reopen the ticket if it is still valid. Thanks!

Changed 7 years ago by Douglas Hays

Attachment: InlineEditBox.js.patch added

patch from Hossam Katory (IBM, CCLA)

comment:3 Changed 7 years ago by Alia Naguib

The direction is set on the widget level

Changed 7 years ago by Alia Naguib

Attachment: test_InlineEditBox.html added

comment:4 Changed 6 years ago by bill

That test case doesn't even create an InlineEditBox programmatically, does it?

Judging from Hossam's patch file, this is actually an enhancement request for InlineEditBox to inherit the direction setting from the srcNodeRef, as opposed to needing to specify the direction to the constructor. But there's no reason to do that for InlineEditBox when it isn't done for other widgets, like TextBox.

Note: See TracTickets for help on using tickets.