Opened 10 years ago
Closed 6 years ago
#12905 closed enhancement (patchwelcome)
CheckBox, RadioButton: Control over base text direction (textDir)
| Reported by: | tomer | Owned by: | |
|---|---|---|---|
| Priority: | high | Milestone: | 1.13 |
| Component: | Dijit - Form | Version: | 1.6.1 |
| Keywords: | Cc: | [email protected]…, [email protected]…, [email protected]… | |
| Blocked By: | Blocking: |
Description (last modified by )
Attachments (2)
Change History (12)
Changed 10 years ago by
| Attachment: | patch201105091118_Dojo-textDirCheckBox.txt added |
|---|
comment:1 Changed 10 years ago by
| Description: | modified (diff) |
|---|---|
| Summary: | Control over base text direction (textDir) for dijit.form.CheckBox and dijit.form.RadioButton → CheckBox, RadioButton: Control over base text direction (textDir) |
| Type: | defect → enhancement |
comment:2 Changed 10 years ago by
| Description: | modified (diff) |
|---|
Changed 10 years ago by
| Attachment: | checkboxTextDir.patch added |
|---|
updated patch. but, i'm not really interested in having checkbox adjust the textdir of the label unless the label is part of the checkbox.
comment:3 Changed 10 years ago by
| Blocked By: | 11611 added |
|---|
comment:4 Changed 10 years ago by
| Blocked By: | 11611 removed |
|---|
(In #11611) The label in ToggleButton? is different than the LABEL tag - probably just poorly named. Patches are welcome. The generated LABEL element would need to be text-direction aware.
comment:5 Changed 10 years ago by
| Milestone: | 1.8 → tbd |
|---|
comment:6 Changed 9 years ago by
| Milestone: | tbd → 1.9 |
|---|
comment:7 Changed 9 years ago by
| Milestone: | 1.9 → future |
|---|
Oh, this is the ticket that depends on #11611.
Doug - yes, the label in ToggleButton is different than the LABEL tag typically used with checkboxes and radio buttons, but I think you are missing the point of #11611, which is for an app to be able to instantiate a checkbox+label in the same way that it can currently instantiate a ToggleButton, like new CheckBox({label: "hi"}).
comment:8 Changed 8 years ago by
| Owner: | Douglas Hays deleted |
|---|---|
| Status: | new → assigned |
comment:9 Changed 8 years ago by
| Status: | assigned → open |
|---|
comment:10 Changed 6 years ago by
| Milestone: | future → 1.12 |
|---|---|
| Resolution: | → patchwelcome |
| Status: | open → closed |
Given that no one has shown interest in creating a patch in the past 4+ years, I'm closing this as patchwelcome.
original patch from Maria