Closed
Bug 309641
Opened 19 years ago
Closed 19 years ago
Template-generated menulist popups change selection when opened via keyboard
Categories
(Core :: XUL, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: mnyromyr, Assigned: neil)
Details
(Keywords: fixed1.8, regression)
Attachments
(1 file)
677 bytes,
patch
|
mnyromyr
:
review+
mscott
:
superreview+
mscott
:
approval1.8b5+
|
Details | Diff | Splinter Review |
If you open a template-generated menulist popup (eg. Location toolbar in MailNews and search location dropdown on MailNews' search dialog) via ALT-downarrow, the selection changes also. This is not the case for 'normal' menulists. Seen with self-built SeaMonkey Win2k trunk nightly of 2005-09-21.
Assignee | ||
Comment 1•19 years ago
|
||
Assignee: nobody → neil.parkwaycc.co.uk
Status: NEW → ASSIGNED
Attachment #197331 -
Flags: superreview?(mscott)
Attachment #197331 -
Flags: review?(mnyromyr)
Reporter | ||
Comment 2•19 years ago
|
||
Comment on attachment 197331 [details] [diff] [review] Proposed patch Does as intended on Win and does no harm on Mac...
Attachment #197331 -
Flags: review?(mnyromyr) → review+
Comment 3•19 years ago
|
||
Comment on attachment 197331 [details] [diff] [review] Proposed patch Neil, do you think this is something we should add to the 1.8 branch?
Attachment #197331 -
Flags: superreview?(mscott) → superreview+
Assignee | ||
Comment 4•19 years ago
|
||
Fix checked in to the trunk.
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 5•19 years ago
|
||
Comment on attachment 197331 [details] [diff] [review] Proposed patch >Neil, do you think this is something we should add to the 1.8 branch? Yes, it has the bug too, and the fix is trivial.
Attachment #197331 -
Flags: approval1.8b5?
Comment 6•19 years ago
|
||
Comment on attachment 197331 [details] [diff] [review] Proposed patch this has no impact on Firefox at all.
Attachment #197331 -
Flags: approval1.8b5? → approval1.8b5+
Component: XP Toolkit/Widgets: Menus → XUL
QA Contact: xptoolkit.menus → xptoolkit.widgets
You need to log in
before you can comment on or make changes to this bug.
Description
•