121631 - Page Info should show ALT attributes for images in Media tab

Status: RESOLVED FIXED

(SeaMonkey :: Page Info, defect)

Description

[Dan Tobias]

One more nit to pick with the otherwise excellent new Page Info feature (as of
build 2002012403):

The ALT attribute for an image ought to be one of the fields shown, as it was in
the old Page Info feature.  The new one shows the TITLE and LONGDESC (still,
alas, underutilized by Web developers), but omits the ALT (which is more often
used).

Comment 1

[Warner Young]

Agreed.  Confirming.

Comment 2

[Christopher Hoess (gone)]

->db48x (Page Info)

Comment 3

[sairuh (rarely reading bugmail)]

mass moving open bugs pertaining to page info to [e-mail] as qa contact.

to find all bugspam pertaining to this, set your search string to
"BigBlueDestinyIsHere".

Comment 4

[Daniel Brooks [:db48x]]

it's definatly doable. let me look at see if I can remeber why I didn't include
it in the first place.

Comment 5

[Daniel Brooks [:db48x]]

Attachment: add alt text

Comment 6

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 66504 [details] [diff] [review]
add alt text

r=bzbarsky

Comment 7

[Daniel Brooks [:db48x]]

Attachment: correctly finds alt text for objects

Comment 8

[Daniel Brooks [:db48x]]

Attachment: correctly finds alt text for objects

Comment 9

[Daniel Brooks [:db48x]]

Comment on attachment 66509 [details] [diff] [review]
correctly finds alt text for objects

blah. I hit submit too many times or something

Comment 10

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 66510 [details] [diff] [review]
correctly finds alt text for objects

Right.	Good point on objects.

r=bzbarsky

Comment 11

[Daniel Brooks [:db48x]]

Attachment: fix strict js warning

Comment 12

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 66561 [details] [diff] [review]
fix strict js warning

Um. Rigth.  I shouldn't have missed that....

Comment 13

[Daniel Brooks [:db48x]]

jag, could you look at this for sr= please?

Comment 14

[Adam Hauner]

*** Bug 123637 has been marked as a duplicate of this bug. ***

Comment 15

[Nicolás Lichtmaier]

Uhm.. I've always used Mozilla's "page info" to check for ALT text in images and
add it the missing places. Sorry for the spam, but this seems to have been
forgotten (as is it have been already done and ready to be checked in for a
month now).

Comment 16

[jag (Peter Annema)]

Comment on attachment 66561 [details] [diff] [review]
fix strict js warning

sr=jag

Comment 17

[Boris Zbarsky [:bzbarsky]]

drivers say, "not critical for 0.9.9, ask for 1.0 approval once 0.9.9 is out"

Comment 18

[Gilles Durys]

*** Bug 129277 has been marked as a duplicate of this bug. ***

Comment 19

[Asa Dotzler [:asa]]

Comment on attachment 66561 [details] [diff] [review]
fix strict js warning

a=asa (on behalf of drivers) for checkin to the 1.0 trunk

Comment 20

[Boris Zbarsky [:bzbarsky]]

checked in on the trunk

Comment 21

[Hakon]

Well it would be nice to also have the ability to list ALT attributes in a
column in the list, by clicking on the "column button". As you could do in the
old PageInfo. Currently the only options available are "no.", "Address" and "Type".

Comment 22

[Daniel Brooks [:db48x]]

There isn't room for an alt tag of the proper length in the outliner. Rember,
the contents of the alt tag are supposed to be whatever text is neccessary to
replace the image, so it when used correctly it'll be a sentace or two at the
least. Much better suited to the preview area than a column in the outliner.

Comment 23

[Nicolás Lichtmaier]

It was there before, and it was useful. I used it to quickly ensure that pages
has all ALT texts. For me, the current view-info is lesss useful than the old.. =/.
You could take the first N characters and add '...', and put the whole value in
a tooltip.

Comment 24

[Hakon]

Yes but the "Address" content is often too long as well.
I didn't believe ALT properties were usually longer than a "Advertisement from
'Company'", as opposed to the LONGDESC, but I guess you're right about how it
_should_ be used.
Nevertheless, it's a fact that many users need to use this as a makeshift at
webplaces that use the ALT property wrong. And so it would far more convenient
to be able to scroll the outliner in order to find where there _are_ ALT
properties, instead of having to scroll each single entry in order to find the
right images.

Comment 25

[Nicolás Lichtmaier]

I have filed bug 158245 and posted a patch that adds the "alt" attribute of
images to the <tree> in the media tab.