Last Comment Bug 309641 - Template-generated menulist popups change selection when opened via keyboard
: Template-generated menulist popups change selection when opened via keyboard
Status: RESOLVED FIXED
: fixed1.8, regression
Product: Core
Classification: Components
Component: XUL (show other bugs)
: Trunk
: x86 Windows 2000
: -- normal (vote)
: ---
Assigned To: neil@parkwaycc.co.uk
:
:
Mentors:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2005-09-22 10:25 PDT by Karsten Düsterloh
Modified: 2008-07-31 03:19 PDT (History)
2 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
Proposed patch (677 bytes, patch)
2005-09-25 06:33 PDT, neil@parkwaycc.co.uk
mnyromyr: review+
mscott: superreview+
mscott: approval1.8b5+
Details | Diff | Splinter Review

Description Karsten Düsterloh 2005-09-22 10:25:01 PDT
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.
Comment 1 neil@parkwaycc.co.uk 2005-09-25 06:33:19 PDT
Created attachment 197331 [details] [diff] [review]
Proposed patch
Comment 2 Karsten Düsterloh 2005-09-26 10:28:29 PDT
Comment on attachment 197331 [details] [diff] [review]
Proposed patch

Does as intended on Win and does no harm on Mac...
Comment 3 Scott MacGregor 2005-09-26 11:34:50 PDT
Comment on attachment 197331 [details] [diff] [review]
Proposed patch

Neil, do you think this is something we should add to the 1.8 branch?
Comment 4 neil@parkwaycc.co.uk 2005-09-27 05:03:04 PDT
Fix checked in to the trunk.
Comment 5 neil@parkwaycc.co.uk 2005-09-27 05:05:43 PDT
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.
Comment 6 Scott MacGregor 2005-09-27 08:28:17 PDT
Comment on attachment 197331 [details] [diff] [review]
Proposed patch

this has no impact on Firefox at all.

Note You need to log in before you can comment on or make changes to this bug.