Open Bug 335167 Opened 18 years ago Updated 2 years ago

'Save Page As...' should be 'Save Image As...' when viewing an image (or 'Save As...')


(Firefox :: Menus, defect)





(Reporter: gidsgoldberg, Unassigned)



(Whiteboard: [Australis:P-])


(1 file)

When viewing an image the File menu reads 'Save Page As...' it should read 'Save Image As...'

IE gets around this by using 'Save As...' for everything, which would be easier to implement but not as descriptive.

The equivalant for Seamonkey is Bug 171515
Axel, please check it out. Bug 352245 is related to this one in l10n ko.
Ever confirmed: true
The sibling of this Firefox bug, Seamonkey bug 171515, also notes the same problem for text documents PDF files, etc.
One solution (mentioned in Bug 352245), is to change it to "Save As..." instead of having to detect types and things with "Save <type> As..." and all its' localised equivalents.
Summary: File --> 'Save Page As...' should be 'Save Image As...' when viewing an image → 'Save Page As...' should be 'Save Image As...' when viewing an image (or 'Save As...')
I would like to fix this bug
(In reply to Nguyen Ngoc Trung from comment #5)
> I would like to fix this bug

If you want to work on this bug, I recommend that you look into the documentation to become familiar with Mozilla development:

If you have specific questions about writing this patch, you can ask here.
Attached patch Proposed patchSplinter Review
I have changed the label "Save Page As..." to "Save As..."
Comment on attachment 613797 [details] [diff] [review]
Proposed patch

Thanks for the patch! Sorry it took us so long to notice :(

When changing strings, it's necessary to also change the entity name so that localizers can detect the change. So that means you would also need to change "savePageCmd.label" to e.g. "saveCmd.label", and update other references to it in browser-appmenu, browser-context, etc.

But before moving further with patching, let's see whether UX folks have an opinion on this bug.
Attachment #613797 - Flags: ui-review?(ux-review)
Blocks: 978380
Whiteboard: [Australis:P-]
Assignee: nobody → trunga0
This should apply to video and audio files, since there are dedicated strings for those too.

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

Assignee: trunga0 → nobody
Flags: needinfo?(jaws)
Flags: needinfo?(jaws)
Severity: minor → S4
You need to log in before you can comment on or make changes to this bug.