Closed
Bug 62185
Opened 24 years ago
Closed 22 years ago
menulists not selecting first item by default
Categories
(Core :: XUL, defect, P3)
Core
XUL
Tracking
()
RESOLVED
DUPLICATE
of bug 118038
Future
People
(Reporter: bryner, Assigned: hyatt)
References
(Blocks 1 open bug)
Details
(Keywords: regression)
Attachments
(1 file)
671 bytes,
patch
|
Details | Diff | Splinter Review |
In the linux filepicker, the menulist for the filter is blank when the dialog initially appears. What should be happening is that the first item is selected. This seems to have broken with the XBL menulist checkins, although I'm not sure which one in particular.
Reporter | ||
Updated•24 years ago
|
Keywords: nsbeta1,
regression
Reporter | ||
Comment 2•24 years ago
|
||
Yes, the filter list is initially empty. Items are added to the menulist using appendChild. Shouldn't the menulist select the first child as soon as it's added?
Reporter | ||
Comment 3•24 years ago
|
||
Reporter | ||
Comment 5•24 years ago
|
||
Checked in a fix for the filepicker. Per hyatt, reassigning to him to cover the generic case of selecting the first item automatically when the menulist is initially empty. Also un-nominate for beta1, since we have an easy workaround.
Assignee: bryner → hyatt
Severity: normal → minor
Keywords: nsbeta1
OS: Linux → All
Hardware: PC → All
Target Milestone: --- → Future
Comment 6•24 years ago
|
||
The menulist actually does do this on binding attached, but I guess we'll need to add additional logic to handle the case when it starts empty and items are programmatically appended.
Assignee | ||
Comment 7•24 years ago
|
||
Later. :) I don't want to put any mutation event listeners in yet if we can help it. :)
Status: NEW → ASSIGNED
Comment 8•22 years ago
|
||
I think this is fixed by Neil's fix to bug 118038. Smells like a dup to me. *** This bug has been marked as a duplicate of 118038 ***
Status: ASSIGNED → RESOLVED
Closed: 22 years ago
Resolution: --- → DUPLICATE
Component: XP Toolkit/Widgets: Menus → XUL
QA Contact: jrgmorrison → xptoolkit.widgets
You need to log in
before you can comment on or make changes to this bug.
Description
•