Closed Bug 66683 Opened 24 years ago Closed 2 years ago

non-navigator sidebars's save image as not showing (related to 21515)

Categories

(Firefox :: File Handling, defect)

x86
All
defect
Not set
normal

Tracking

()

RESOLVED INVALID

People

(Reporter: doronr, Unassigned)

Details

(Keywords: regression)

spun of 21515. for some reason, the feature is messed on non-navigator sidebars
(the entire menuitem is gone).
some random keywords
Status: NEW → ASSIGNED
Keywords: regression
Target Milestone: --- → mozilla0.8
What's a non-navigator sidebar?
sidebars in mail or editor usually
ok, here is what is happening

setAttribute works correctly, i can get the attribute after setting.

the menuitem however gets the accesskey and never refreshes. temp patch would be
to put in a default label.

this works outside of sidebar and in the navigator sidebar.

wierd
cc: timeless, blake
http://lxr.mozilla.org/seamonkey/source/xpfe/communicator/resources/content/nsContextMenu.js#117

that's the code, any one have any idea why this is not updating the context menu
in non-navigator sidebars?
So I'm guessing that this.onImage is false in these cases?
actually, no, as it would show nothing if it were.

on my tree, I dump a getAttribute(), and it dumps the correct string "Save image
(imagename)..." Seems that somehow, the context menu is not getting updated.
even though setAttribute does work
adding folx who might be interested in this...
Probably related to bug 61305, "in mailnews, sidebar context menu has too many 
items".
0.9 moving spam
Target Milestone: mozilla0.8 → mozilla0.9
This is showing up for me using build 2001030804.  I'm using the Mozillazine
sidebar in Composer, and when I right-click on the MZ image I see Save Image. 
But it doesn't work in Address Book.  Ugh.
--> future, like 61305
Target Milestone: mozilla0.9 → Future
spam: over to File Handling. haven't changed the current owners, but i did take
qa contact for the nonce. pls do retriage/reassign if needed.
Component: XP Apps → File Handling
QA Contact: sairuh → petersen
QA Contact: chrispetersen → file-handling
Product: Core → Firefox
Target Milestone: Future → ---
Version: Trunk → unspecified

The bug assignee didn't login in Bugzilla in the last 7 months.
:Gijs, could you have a look please?
For more information, please visit auto_nag documentation.

Assignee: doronr → nobody
Status: ASSIGNED → NEW
Flags: needinfo?(gijskruitbosch+bugs)
Status: NEW → RESOLVED
Closed: 2 years ago
Flags: needinfo?(gijskruitbosch+bugs)
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.