142182 - Right-Click on Image has conflicting option shortcuts

Status: VERIFIED WORKSFORME

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

Description

[carl mueller]

I hope this hasn't been posted yet. You guys are doing a great job, Mozilla
0.9.9 is fantastic.

Anyway, in the most extreme example of this bug, if you right-click on an image
link, the A shortcut key maps to these three right-click menu options:

Save Page As
Save Link As
Save Image

if its not an image link, but just an image, you still have a collision with
Save Page As and Save Image.

Thus, Right-Clicking on an image link and pressing 'A' on the keyboard activates
the save page as, when I usually want to Save Link As or Save Image. 

I is currently used in the Right-Click menu by Get Page Info and Send Link
L is currently used in select all

Just to spout my personal views on the UI design, the right click menu should
reserve its keyboard shortcuts to actions that are specifically performed with
off-click: saving links/images/specific sections of a page.

Select All and Page Info are easily activated commands using ALT-E-A and ALT-V-I
regardless of what you're doing in a page. Thus, using L and I on them given the
frequency that right-click menus would be used to Save Link and Save Image would
seem wasteful.

Dunno what to do about Send Link, but I'd think that action happens less often
than saving a link/destination.

Comment 1

[Boris Zbarsky [:bzbarsky]]

Context menus have been rewritten since 0.9.9.  In particular, the shortcut keys
for 

Save Link Target As
Save Image As

are "r" and "v" respectively.  "Save Page As" is no longer in the "Linked image"
context menu.  Neither are "Select All" or "View page info".  "Send Link" is
also gone from context menus now.

Marking this worksforme, since I think that covers all the issues raised in this
bug... If I've missed something, please let me know.

Comment 2

[sairuh (rarely reading bugmail)]

mass-verifying WorksForMe bugs.

reopen only if this bug is still a problem with a *recent trunk build*.

mail search string for bugspam: AchilleaMillefolium