Closed
Bug 309641
Opened 20 years ago
Closed 20 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•20 years ago
|
||
Assignee: nobody → neil.parkwaycc.co.uk
Status: NEW → ASSIGNED
Attachment #197331 -
Flags: superreview?(mscott)
Attachment #197331 -
Flags: review?(mnyromyr)
| Reporter | ||
Comment 2•20 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•20 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•20 years ago
|
||
Fix checked in to the trunk.
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
| Assignee | ||
Comment 5•20 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•20 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
•