Closed Bug 3916 Opened 26 years ago Closed 24 years ago

{compat} [IMG] Make IMG height=% relative to viewport

Categories

(Core :: Layout, defect, P3)

Product:
Core
Component:
Layout
Type:
defect

Tracking

()

Status:
VERIFIED DUPLICATE of bug 33443

People

(Reporter: elig, Assigned: buster)

References

(
URL
)

Details

(Keywords: testcase)

[Hey, Pam! I know this isn't ImageLib, but figured you'd know who gets it. Thanks!] * TITLE/SUMMARY IMG tag ignores HEIGHT attribute (when specified as percentage) * STEPS TO REPRODUCE 0) Launch Apprunner 1) View http://slip/projects/marvin/html/img_height_percent.html * RESULT - What happened All images, irregardless of percentage size, appear the same size. However, the sister test cases work fine: http://slip/projects/marvin/html/img_width_percent.html http://slip/projects/marvin/html/img_height_percent.html (excepting those covered by 2097) - What was expected Images to scale properly. * REGRESSION - Occurs On Mac OS Apprunner (3.17.99 optimized build) Win32 Apprunner (3.17.99 optimized build [NT 4, Service Pack 3]) Linux Apprunner (3.17.99 optimized build) - Doesn't Occur On Communicator 4.51 RTM (Mac OS) * CONFIGURATIONS TESTED - [Mac] Power Mac 8500/120 (233 Mhz 604e), 64 MB RAM (VM on; 1 MB of VM used), 1024x768 (Thousands of Colors), Mac OS 8.5.1 - [Win32] Vectra VL (233 Mhz P2), 96 MB RAM, 800x600 (True Color), NT 4.0 SP3. - [Linux] Vectra VL (266 Mhz P2), 96 MB RAM.
Status: NEW → ASSIGNED
yep. probably layout. I'll double check and reassign as needed. thx, pn
Target Milestone: M6
[Still taking place; checked 4.13.99 Mac OS M4 candidate build.] Pam, would you like to reassign this one to the appropriate party, as you mentioned in your earlier comment? Thanks!
Assignee: pnunn → kipp
Status: ASSIGNED → NEW
kipp: Is this yours? (percentage w&h on image tag) -pnunn
Status: NEW → RESOLVED
Closed: 26 years ago
Resolution: --- → INVALID
See CSS2 spec, section 10.5. In particular, percentage heights' are computed relative to the containing block. If the containing block (the body in your test) has no explict height then the value is replaced with "auto".
Status: RESOLVED → REOPENED
Re-opening to ask a question... Kipp, just to confirm, this is specifically when the IMG tag is used in HTML, rather than within a style sheet, right? Do I understand correctly that both Internet Explorer 5 and Navigator 4.x's handling of this tag is incorrect? (they both scale the image on the test document.) Thanks!
Status: REOPENED → ASSIGNED
Resolution: INVALID → ---
Severity: normal → major
Priority: P3 → P5
Summary: IMG tag ignores HEIGHT attribute (when a percentage) → {compat} IMG tag ignores HEIGHT attribute (when a percentage)
Target Milestone: M6 → M9
its a compatability bug; we handle things ala css which is not 100% compatible with nav4. thanks for digging furthur!
Severity: major → minor
Component: ImageLib → Layout
Priority: P5 → P4
Summary: {compat} IMG tag ignores HEIGHT attribute (when a percentage) → {compat} Make IMG % heights relative to viewport
Recap: In compat mode, the HEIGHT attribute of the IMG element should be taken as being relative to the viewport height, if the value is given as a percentage (and this shouldn't affect the CSS interpretation, of course).
Summary: {compat} Make IMG % heights relative to viewport → {compat} Make IMG height=% relative to viewport
Whiteboard: [TESTCASE]
Once this is fixed, check that http://www.thing.net/~flux/ looks the same as it does in legacy browsers.
Severity: minor → major
Priority: P4 → P5
[As a courtesy FYI, as of the 1999092408 Mac OS, Win32 and Linux builds, this bug is still present, and has not received a "Mystery Fix" from other code changes.]
Updating to default Layout Assignee...kipp no longer with us :-(
Why are you re-reassing layout bugs? Do NOT touch layout bugs. The bugs are assigned to Kipp so they can stay neatly organized until we have a new owner for the block/inline code.
Summary: {compat} Make IMG height=% relative to viewport → {compat} [IMG] Make IMG height=% relative to viewport
Bulk moving [testcase] code to new testcase keyword. Sorry for the spam!
Keywords: testcase
Whiteboard: [TESTCASE]
mine! mine mine mine! all mine! whoo-hoo!
Assignee: kipp → buster
*** Bug 36668 has been marked as a duplicate of this bug. ***
Eli, is bug 33443 a dupe of this one or related to it?
It is indeed. I'm marking this a dup of 33443 since 33443 is assigned to waqar who has fewer bugs than buster. *** This bug has been marked as a duplicate of 33443 ***
Status: NEW → RESOLVED
Closed: 26 years ago → 24 years ago
Resolution: --- → DUPLICATE
Verified that this is a duplicate of bug 33443 and marking as such.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.