User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8b2) Gecko/20050418 Firefox/1.0+
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8b2) Gecko/20050418 Firefox/1.0+
When using the keyboard to select bookmark context menu menuitems some menuitems
are often skipped. The closest bug to this I could find is bug 266076 but that
deals with accesskeys.
Steps to Reproduce:
1. Click Bookmarks and context menu click a bookmark
2. Use the arrow keys to select menuitems
Some enabled menuitems are skipped
Each enabled menuitem is selected in turn
This bug appears for all bookmarks and bookmark folder in the Bookmarks menu. It
appears that if the context menu is brought up from the top of the Bookmarks
Toolbar for a bookmark or bookmark folder that it is possible to select each
enabled menuitem in turn using the arrow keys on the keyboard. If the bookmark
or bookmark folder is inside of another bookmark folder in the Bookmarks Toolbar
that this bug is exhibited.
Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.8b2) Gecko/20050419
the Items are picked seemingly at random
very much depends on where the mousepointer is located
Looks to have regressed somewhere between 20050117 and 20050127. And at least
what I'm seeing isn't random, it's just every other menu item gets skipped.
I noticed it wasn't random as well just after filing. It seems as if the
keypress event is running twice for popups that are a contentmenu of a popup.
Created attachment 181370 [details]
This does not appear to be bookmarks specific though it does affect bookmarks.
Changing the summary, product, and component accordingly though I suspect the
category may still be incorrect.
Reassigning to email@example.com since this is not bookmarks specific to get it
off of Vladimir's plate.
Aaron, could this be a regression from your bug 282438? The window is
2005-02-17-07 to 2005-02-18-07, and even though I don't know what I'm talking
about, yours looks the most likely.
The regression range in comment 7 is off by a month (compare comment 2). I see
the bug in a 2005-01-19-08 build but not a 2005-01-18-07 build.
So bug 282438 is definitely not it.
Bug 167145, on the other hand, is quite a likely culprit, since it messed around
with key events...
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8b4) Gecko/20050802
For me, the navigation moves in the opposite direction of the key i'm pressing.
i.e. I press up and it moves down.
Aaron - it appears that way visually because it is skipping an item.
Created attachment 191588 [details] [diff] [review]
Ignore key presses that have already been dealt with. There can be more than one listener on the same node.
Spin off bug 303404 is a crash I found while debugging this.
Created attachment 191845 [details] [diff] [review]
Don't attach multiple listeners
*** Bug 304603 has been marked as a duplicate of this bug. ***
Neil's got a patch
Fix checked in to the trunk.
*** Bug 311259 has been marked as a duplicate of this bug. ***
Comment on attachment 191845 [details] [diff] [review]
Don't attach multiple listeners
I'm nervous about this change. We can't afford any regressions this late in the
game. What kind of testing has been done here and what kind of risk does this
neil, aaron, qa, have we verified that this doesn't break anything?
I will check this on Windows XP, but using today's Mac trunk build (Mozilla/5.0
(Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.9a1) Gecko/20051011
Firefox/1.6a1), the test case doesn't work. Using today's Mac Branch build, the
test case does work.
Just to clarify, when I mean the test case works, you can use the arrow keys to
move up and down without skipping.
(In reply to comment #20)
> I will check this on Windows XP, but using today's Mac trunk build (Mozilla/5.0
> (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.9a1) Gecko/20051011
> Firefox/1.6a1), the test case doesn't work. Using today's Mac Branch build, the
> test case does work.
This looks ok using today's Windows Trunk build (Mozilla/5.0 (Windows; U;
Windows NT 5.1; en-US; rv:1.9a1) Gecko/20051011 Firefox/1.6a1). If I run the
test case, I can arrow through the choices with no problem. I also did some
context clicking on bookmarks and didn't see any problems.
plussing since we decide to take this patch.
Neil, can you get this checked into the branch soon? If not, let us know and we
can find someone who can land it (maybe me :)).
Sorry, I overlooked the a=asa.