319 bytes, text/html
500 bytes, text/html
670 bytes, text/html
5.85 KB, patch
Stuart Parmenter: review+
Marc Attinasi: superreview+
|Details | Diff | Splinter Review|
DESCRIPTION: The image with class eleven in the above page is displayed with a height of 600px, when it should be its natural size. The problem is that when an image with display: block is between its max-height and min-height, so neither property should do anything, it's getting its max-height as its height. STEPS TO REPRODUCE: * load attached testcase ACTUAL RESULTS: * the image is 800px tall EXPECTED RESULTS: * the image should be about 300px tall DOES NOT WORK CORRECTLY ON: * Linux, apprunner, 1999-11-09-08-M11
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.
mass moving all Kipp's pre-beta bugs to M15. Nisheeth and I will prioritize these and selectively move high-priority bugs into M13 and M14.
mine! mine mine mine! all mine! whoo-hoo!
moving all buster m15 bugs to m16.
important to fix, but no owner currently
redistributing bugs across future milestones, sorry for the spam
This bug has been marked "future" because the original netscape engineer working on this is over-burdened. If you feel this is an error, that you or another known resource will be working on this bug,or if it blocks your work in some way -- please attach your concern to the bug for reconsideration.
It appears that this now works. However, we should test it again once the height:% bug is fixed. Also, I tested it on Win2K, David originally reported the bug on Linux.
(actually, just for the record, the testcase I attached did not get around a bug in our strict DTD, it fixed a bug in the original testcase.)
Percentage widths on images are completely broken right now. Don't mark this WORKSFORME until that is fixed...
Netscape's standard compliance QA team reorganised itself once again, so taking remaining non-tables style bugs. Sorry about the spam. I tried to get this done directly at the database level, but apparently that is "not easy because of the shadow db", "plus it screws up the audit trail", so no can do...
There is still a bug here. I shall attach a testcase. It seems to only occur when the 'height' property is set to 'auto'.
odd: the testcase that Ian most recently added renders identically in todays build and in IE6. Setting to P:2, Normal, mozilla 1.0
Nothing odd about it, that would just imply IE6 is broken too ;-)
So, will this bug magically fix itself in a couple of months, when it is two years old :) That would be cool, heh heh heh.
Does that often happen to your bugs? If so, remind me to assign some to you just before they turn 2 years old ;-)
Could we get a fix for this before the end of mozilla098? It blocks implementation of a P2 feature on our PRD, part of a theme for MachV.
So pavlov broke min-height and max-height even more than it was broken when Ian reopened the bug. But I have a fix...
Comment on attachment 60906 [details] [diff] [review] patch r=pavlov
Comment on attachment 60906 [details] [diff] [review] patch sr=attinasi
dbaron landed the fix on 12/10/2001 19:21
Hixie, in attachment 50145 [details] you say that "this should be after the image, not under it". Is that correct, and if so, is that this bug or another one?
See also bug 234686 for further refinements.