Bug 439133 - "Show Image" is poorly labeled, can be confused with "View Image". ui-r=faaborg, r=mano

This commit is contained in:
Dão Gottwald 2008-10-27 23:47:01 +01:00
parent f43cc80f2f
commit e785f2ee9a
3 changed files with 9 additions and 9 deletions

View File

@ -99,10 +99,10 @@
accesskey="&mediaHideControls.accesskey;"
oncommand="gContextMenu.mediaCommand('hidecontrols');"/>
<menuseparator id="context-media-sep-commands"/>
<menuitem id="context-showimage"
label="&showImageCmd.label;"
accesskey="&showImageCmd.accesskey;"
oncommand="gContextMenu.showImage();"/>
<menuitem id="context-reloadimage"
label="&reloadImageCmd.label;"
accesskey="&reloadImageCmd.accesskey;"
oncommand="gContextMenu.reloadImage();"/>
<menuitem id="context-viewimage"
label="&viewImageCmd.label;"
accesskey="&viewImageCmd.accesskey;"

View File

@ -220,8 +220,8 @@ nsContextMenu.prototype = {
.disabled = this.disableSetDesktopBackground();
}
// Show image depends on an image that's not fully loaded
this.showItem("context-showimage", (this.onImage && !this.onCompletedImage));
// Reload image depends on an image that's not fully loaded
this.showItem("context-reloadimage", (this.onImage && !this.onCompletedImage));
// View image depends on having an image that's not standalone
// (or is in a frame), or a canvas.
@ -740,7 +740,7 @@ nsContextMenu.prototype = {
BrowserPageInfo(this.target.ownerDocument);
},
showImage: function(e) {
reloadImage: function(e) {
urlSecurityCheck(this.mediaURL,
this.browser.contentPrincipal,
Ci.nsIScriptSecurityManager.DISALLOW_SCRIPT);

View File

@ -233,8 +233,8 @@
<!ENTITY viewPageInfoCmd.accesskey "I">
<!ENTITY viewFrameInfoCmd.label "View Frame Info">
<!ENTITY viewFrameInfoCmd.accesskey "I">
<!ENTITY showImageCmd.label "Show Image">
<!ENTITY showImageCmd.accesskey "S">
<!ENTITY reloadImageCmd.label "Reload Image">
<!ENTITY reloadImageCmd.accesskey "R">
<!ENTITY viewImageCmd.label "View Image">
<!ENTITY viewImageCmd.accesskey "I">
<!ENTITY viewBGImageCmd.label "View Background Image">