Open Bug 335167 Opened 14 years ago Updated Last year

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

Categories

(Firefox :: Menus, defect, minor)

defect
Not set
minor

Tracking

()

People

(Reporter: gidsgoldberg, Assigned: trunga0)

References

Details

(Whiteboard: [Australis:P-])

Attachments

(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.
Status: UNCONFIRMED → NEW
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.
Duplicate of this bug: 510883
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: https://developer.mozilla.org/en-US/mozilla

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)
Whiteboard: [Australis:P-]
Assignee: nobody → trunga0
This should apply to video and audio files, since there are dedicated strings for those too.
You need to log in before you can comment on or make changes to this bug.