Closed
Bug 298186
Opened 19 years ago
Closed 19 years ago
richlistbox - richlistitem's disabled property is not available except by using getAttribute
Categories
(Toolkit :: UI Widgets, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: robert.strong.bugs, Assigned: doronr)
References
Details
(Keywords: regression)
Attachments
(1 file)
629 bytes,
patch
|
mconnor
:
first-review+
chofmann
:
approval-aviary1.1a2+
|
Details | Diff | Splinter Review |
The extension manager code retrieves the disabled property directly (e.g. elem.disabled instead of elem.getAttribute("disabled")). With the landing of bug 285584 it appears that this is no longer available except by using getAttribute. Changing the following in richlistbox.xml resolves this though I am unsure if that is the way you would prefer to fix this issue. - <binding id="richlistitem"> + <binding id="richlistitem" extends="chrome://global/content/bindings/general.xml#basecontrol">
Assignee | ||
Comment 1•19 years ago
|
||
Rob - next time, don't be shy and post a patch :)
Attachment #186927 -
Flags: first-review?(mconnor)
Reporter | ||
Comment 2•19 years ago
|
||
I wasn't sure whether you wanted tabIndex which is a gimme from general.xml with this approach for all richlistitems so i deferred to you . ;-)
Updated•19 years ago
|
Attachment #186927 -
Flags: first-review?(mconnor) → first-review+
Assignee | ||
Updated•19 years ago
|
Attachment #186927 -
Flags: approval-aviary1.1a2?
Comment 3•19 years ago
|
||
Comment on attachment 186927 [details] [diff] [review] le patch a=chofmann
Attachment #186927 -
Flags: approval-aviary1.1a2? → approval-aviary1.1a2+
Assignee | ||
Comment 4•19 years ago
|
||
checked in
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 5•19 years ago
|
||
*** Bug 298403 has been marked as a duplicate of this bug. ***
You need to log in
before you can comment on or make changes to this bug.
Description
•