323459 - IMG height 100% does not work anymore in XHTML 1.0 Transitional

Status: RESOLVED INVALID

(Firefox :: General, defect)

Description

[Thierry Danard]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8) Gecko/20051111 Firefox/1.5
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8) Gecko/20051111 Firefox/1.5

The value of the height attribute in images does not work when a XHTML 1.0 Transitional doctype is specified. When this doctype is removed, it works. This problem DID NOT OCCUR with Firefox 1.0.x

I have attached the files (when it works and when it doesn't) to this bug so that you can see by yourself. 

Reproducible: Always

Steps to Reproduce:
1. extract the attached zip file
2. view file "showmethisfirefoxbug.htm"
3. view file "showmewhenthisworks.htm"

Actual Results:  
The images on the side of "Useful Links" are not 100%

Expected Results:  
They should be 100%, just like Firefox 1.0.x did

IE doesn't have this problem.

Comment 1

[Thierry Danard]

Attachment: Test case

Comment 2

[Martin Schröder [:mschroeder]]

Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8) Gecko/20060114 Firefox/1.5 ID:2006011403

Can confirm this.

Comment 3

[Steve England [:stevee]]

I'm pretty sure this is a dupe of something, and is invalid. Perhaps see bug 292759; it might be relevent.

Comment 4

[Thierry Danard]

(In reply to comment #3)
> I'm pretty sure this is a dupe of something, and is invalid. Perhaps see bug
> 292759; it might be relevent.

The test case shows that the behavior of Firefox was changed with Firefox 1.5, contredicting the meaning of the "Transitional" keyword in the "XHTML 1.0 Transitional" standard.

What would be the "XHTML 1.0 Transitional" way of achieving the intended "legacy" result ?

Comment 5

[Ria Klaassen (not reading all bugmail)]

The behaviour changed on 30-Aug-2005.
http://bonsai.mozilla.org/cvsquery.cgi?treeid=default&module=PhoenixTinderbox&branch=HEAD&branchtype=match&dir=&file=&filetype=match&who=&whotype=match&sortby=Date&hours=2&date=explicit&mindate=2005-08-30+06%3A00%3A00&maxdate=2005-08-30+10%3A00%3A00&cvsroot=%2Fcvsroot

So that can only be bug 305975.

Comment 6

[Thierry Danard]

I have downloaded Mozilla Deer Park Alpha 2. The problem has not been resolved.

Comment 7

[Boris Zbarsky [:bzbarsky]]

The behavior in Firefox 1.5 is correct per the CSS spec.

Comment 8

[Thierry Danard]

The bug is not fixed. 

Comment 9

[Boris Zbarsky [:bzbarsky]]

I didn't mark it fixed.  I marked it invalid, because it's not a bug -- it's the right behavior.  Don't reopen it again, please.

Comment 10

[Thierry Danard]

I disagree with that statement. I'll leave that bug closed to avoid a status war. The correct behavior is not explicitely defined in the W3C standard. Since the document uses XHTML 1.0 Transitional, it should keep the legacy behavior that Firefox 1.0 already respects. For a constructive discussion, please document the W3C standards that justified that change so that I can stand corrected.

Comment 11

[Boris Zbarsky [:bzbarsky]]

> The correct behavior is not explicitely defined in the W3C standard.

Actually, it is as long as we're treating the "height" attribute of <img> as a presentational attribute (per CSS 2.1).  Given this one assumption, this gives a specified value of "100%" for the 'height' property of the <img>.  But the computed value for the parent (the <td>) is "auto", so per CSS 2.1 the computed height of the <img> is "auto".  That is, the behavior must be identical to not having the height property set at all.

If you think that the "height" attribute for an image should not be a CSS presentational attribute, that would be a valid change request, but probably wontfix, given that Gecko is a CSS renderer that mostly handles HTML via a UA stylesheet and CSS.

Note also that as far as comment 0's "IE does not have this problem" thing goes, IE's treatment of CSS percentage heights is generally pretty broken.