102629 - duplicate menu access key 'w' in context menu [framed pages]

Status: VERIFIED FIXED

(Core :: DOM: UI Events & Focus Handling, defect)

Description

[sairuh (rarely reading bugmail)]

found using 2001.10.01.0x-branch comm bits. the menu access key 'w' is used for
three items: "show only this frame", "view image" and "view background image".
see below to observe this.
test A
------
1. go to http://www.iconfactory.com/
2. bring up the context menu over one of the linked images, eg, the small green
desk in the upper-left region of the central, big frame.

observe: the context menu displays the following conflicting access keys for 'w':

Show_ Only This Frame
View_ Image

test B
------
1. go to the same page, http://www.iconfactory.com/
2. bring up the context menu over a region in the central, big frame which
doesn't include an image other than the background image. eg, somewhere over the
first paragraph in the middle of the frame would suffice.

observe: the context menu displays the following conflicting access keys for 'w':

Show_ Only This Frame
View_ Background Image

Comment 1

[sairuh (rarely reading bugmail)]

nominating, adding dependency.

Comment 2

[sairuh (rarely reading bugmail)]

btw, i'm filing these bugs so that i can easily track these issues when i test
context menus --whether it's verifying the future fix, or removing it from my
list if the feature changes due to bug 75338. thx!

Comment 3

[Aaron Leventhal]

-> timeless

Comment 4

[Blake Ross]

-> me

Comment 5

[Blake Ross]

Fixed.

Comment 6

[sairuh (rarely reading bugmail)]

vrfy'd fixed --tested at http://wired.com/ on the bottom frame which has a
brackground. (iconfactory no longer has frames.) tested on linux rh7.2 and
win2k, 2002.04.09 comm bits.

Comment 7

[Macarie Neculai]

There are still duplicate access keys for frames.
Open  http://mack.ro/mozilla/test.htm.
In any frame bring up the context menu for the frame and choose "This frame".

The access key 'w' is used by:
Show_ only this Frame
Open frame in new W_indow

The access key 'f' is used by:
Bookmark this F_rame
Save F_rame as