Closed
Bug 119301
Opened 23 years ago
Closed 23 years ago
Implement nsIDOMXULMenuListElement on autocomplete widget
Categories
(SeaMonkey :: Autocomplete, defect, P3)
SeaMonkey
Autocomplete
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla0.9.9
People
(Reporter: aaronlev, Assigned: aaronlev)
References
Details
(Keywords: access, Whiteboard: seeking r=)
Attachments
(2 files)
4.05 KB,
patch
|
hewitt
:
superreview+
|
Details | Diff | Splinter Review |
4.04 KB,
patch
|
jag+mozilla
:
review+
jag+mozilla
:
superreview+
|
Details | Diff | Splinter Review |
The autocomplete widget is like an editable menulist. It should support the nsIDOMXULMenuListElement interface
Assignee | ||
Comment 1•23 years ago
|
||
Should this be a XUL 1.0 blocker?
.
Component: XP Toolkit/Widgets: XUL → XP Apps: Autocomplete
QA Contact: jrgm → blaker
Assignee | ||
Comment 3•23 years ago
|
||
Joe, I think we can open a bug to finish the rest later. Is this worthy?
Assignee | ||
Comment 4•23 years ago
|
||
-> I'll do the first part, what's needed for accessibility
Assignee: hewitt → aaronl
Priority: -- → P3
Whiteboard: seeking r=, sr=
Target Milestone: --- → mozilla0.9.9
Assignee | ||
Updated•23 years ago
|
Status: NEW → ASSIGNED
Comment 5•23 years ago
|
||
Comment on attachment 65211 [details] [diff] [review] This implements enough for accessibility Just change return this.hasAttribute('open'); to return this.getAttribute("open") == "true"; and then sr=hewitt
Attachment #65211 -
Flags: superreview+
Assignee | ||
Comment 6•23 years ago
|
||
Okay, hewitt's correction is fixed on my local tree.
Whiteboard: seeking r=, sr= → seeking r=
Assignee | ||
Comment 7•23 years ago
|
||
Comment 8•23 years ago
|
||
Comment on attachment 65583 [details] [diff] [review] patch with jag's fixes r=jag, carrying over sr=hewitt
Attachment #65583 -
Flags: superreview+
Attachment #65583 -
Flags: review+
Assignee | ||
Comment 9•23 years ago
|
||
fixed
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Updated•20 years ago
|
Product: Core → Mozilla Application Suite
You need to log in
before you can comment on or make changes to this bug.
Description
•