mirror of
https://github.com/mozilla/gecko-dev.git
synced 2024-10-24 02:35:41 +00:00
bug 158245 patch by nick@technisys.com.ar r=db48x sr=bz a=scc
Page info should list the alt for images in the media-tab in the tree.
This commit is contained in:
parent
5235d910e8
commit
0e9486b08e
@ -570,7 +570,7 @@ function makeMediaTab()
|
||||
var theBundle = document.getElementById("pageinfobundle");
|
||||
var imageTree = document.getElementById("imagetree");
|
||||
|
||||
var imageView = new pageInfoTreeView(["image-number","image-address","image-type"], COPYCOL_IMAGE_ADDRESS);
|
||||
var imageView = new pageInfoTreeView(["image-number","image-address","image-type","image-alt"], COPYCOL_IMAGE_ADDRESS);
|
||||
imageTree.treeBoxObject.view = imageView;
|
||||
|
||||
imageList = grabAllMedia(theWindow, theDocument);
|
||||
@ -581,6 +581,7 @@ function makeMediaTab()
|
||||
var mediaEmbed = theBundle.getString("mediaEmbed");
|
||||
var mediaLink = theBundle.getString("mediaLink");
|
||||
var mediaInput = theBundle.getString("mediaInput");
|
||||
var notSet = theBundle.getString("notset");
|
||||
|
||||
var row = null;
|
||||
var length = imageList.length;
|
||||
@ -592,10 +593,10 @@ function makeMediaTab()
|
||||
switch (elem.nodeName.toLowerCase())
|
||||
{
|
||||
case "img":
|
||||
imageView.addRow([++imageIndex, elem.src, mediaImg]);
|
||||
imageView.addRow([++imageIndex, elem.src, mediaImg, (elem.hasAttribute("alt")) ? elem.alt : notSet ]);
|
||||
break;
|
||||
case "input":
|
||||
imageView.addRow([++imageIndex, elem.src, mediaInput]);
|
||||
imageView.addRow([++imageIndex, elem.src, mediaInput, (elem.hasAttribute("alt")) ? elem.alt : notSet ]);
|
||||
break;
|
||||
case "applet":
|
||||
imageView.addRow([++imageIndex, elem.code || elem.object, mediaApplet]);
|
||||
@ -713,11 +714,30 @@ function makePreview(item)
|
||||
var theBundle = document.getElementById("pageinfobundle");
|
||||
var unknown = theBundle.getString("unknown");
|
||||
var notSet = theBundle.getString("notset");
|
||||
var emptyString = theBundle.getString("emptystring");
|
||||
|
||||
var url = ("src" in item && item.src) || ("code" in item && item.code) || ("data" in item && item.data) || ("href" in item && item.href) || unknown; // it better have at least one of those...
|
||||
document.getElementById("imageurltext").value = url;
|
||||
document.getElementById("imagetitletext").value = item.title || notSet;
|
||||
document.getElementById("imagealttext").value = ("alt" in item && item.alt) || getValueText(item) || notSet;
|
||||
|
||||
var altText = null;
|
||||
if (item.hasAttribute("alt") && ("alt" in item))
|
||||
altText = item.alt;
|
||||
else if (item.hasChildNodes())
|
||||
altText = getValueText(item);
|
||||
if (altText == null)
|
||||
altText = notSet;
|
||||
var textbox=document.getElementById("imagealttext");
|
||||
// IMO all text that is not really the value text should go in italics
|
||||
// What if somebody has <img alt="Not specified">? =)
|
||||
// We can't use textbox.style because of bug 7639
|
||||
if (altText=="") {
|
||||
textbox.value = emptyString;
|
||||
textbox.setAttribute("style","font-style:italic");
|
||||
} else {
|
||||
textbox.value = altText;
|
||||
textbox.setAttribute("style","font-style:inherit");
|
||||
}
|
||||
document.getElementById("imagelongdesctext").value = ("longDesc" in item && item.longDesc) || notSet;
|
||||
|
||||
// find out the mime type
|
||||
|
@ -236,7 +236,7 @@
|
||||
<vbox>
|
||||
<tree id="imagetree" class="inset fixedsize" onselect="onImageSelect();">
|
||||
<treecols>
|
||||
<treecol sortSeparators="true" persist="hidden width" flex="1"
|
||||
<treecol sortSeparators="true" hidden="true" persist="hidden width" flex="1"
|
||||
width="1" id="image-number" label="&mediaNo;"/>
|
||||
<splitter class="tree-splitter"/>
|
||||
<treecol sortSeparators="true" persist="hidden width" flex="10"
|
||||
@ -244,6 +244,9 @@
|
||||
<splitter class="tree-splitter"/>
|
||||
<treecol sortSeparators="true" persist="hidden width" flex="2"
|
||||
width="2" id="image-type" label="&mediaType;"/>
|
||||
<splitter class="tree-splitter"/>
|
||||
<treecol sortSeparators="true" hidden="true" persist="hidden width" flex="4"
|
||||
width="4" id="image-alt" label="&mediaAltHeader;"/>
|
||||
</treecols>
|
||||
<treechildren flex="1"/>
|
||||
</tree>
|
||||
|
@ -88,6 +88,7 @@
|
||||
<!ENTITY mediaURL "URL:">
|
||||
<!ENTITY mediaBase "Base URL:">
|
||||
<!ENTITY mediaAlt "Alternate Text:">
|
||||
<!ENTITY mediaAltHeader "Alternate Text">
|
||||
<!ENTITY mediaNo "No.">
|
||||
<!ENTITY mediaName "Name">
|
||||
<!ENTITY mediaAddress "Address">
|
||||
|
@ -29,6 +29,7 @@ pageTitle=%S:
|
||||
unknown=Unknown
|
||||
default=Default
|
||||
notset=Not Specified
|
||||
emptystring=Empty String
|
||||
|
||||
generalNotCached=(not cached)
|
||||
generalNoExpiration=(no expiration set)
|
||||
|
Loading…
Reference in New Issue
Block a user