Closed
Bug 2051
Opened 27 years ago
Closed 27 years ago
Text flows outside of table cells at this url
Categories
(Core :: Layout: Tables, defect, P3)
Tracking
()
VERIFIED
WORKSFORME
People
(Reporter: andreww, Assigned: buster)
References
()
Details
This url shows how text is escaping from the table cells and
over td borders.
|
||
Comment 1•27 years ago
|
||
Here's a simpler test case that reproduces the behaviour.
--------------------------------------------------
<html><head></head><body>
<pre>
The width of the inner table in the table below does not get 'propagated'
to the outer, enclosing table (??) when a <FORM> is inside the inner
table. The strings 'XXX' in the HTML code are used to turn tags and
attributes on and off, in ways that workaround the problem.
Note also that the FORM is being displayed as an inline element, when it
should be block (Bugzilla #1094), but this bug (#2070) is independent of
#1094 (I think).</pre>
<TABLE BORDER=1 CELLPADDING=2 CELLSPACING=0 WIDTH=500>
<!-- "fix" 1: declare TD with same width as inner table -->
<!-- <TR><TD XXX WIDTH=300> -->
<TR><TD XXXWIDTH=300>
<!-- "fix" 2: have inner table use auto-width -->
<!-- <TABLE BORDER=1 CELLPADDING=2 CELLSPACING=0 XXXWIDTH=300> -->
<TABLE BORDER=1 CELLPADDING=2 CELLSPACING=0 XXX WIDTH=300>
<TR><TD>
blah blah blah blah blah blah blah
blah blah blah blah blah blah blah
<!-- "fix" 3: "remove" the <FORM> -->
<!-- <FORMXXX METHOD=GET ACTION="WHATEVER"> -->
<FORM XXX METHOD=GET ACTION="WHATEVER">
<select name="range">
<option value="something">123456789_12345</select>
</FORM>
</TD></TR>
</TABLE>
</td><td>
ASDF ASDF ASDF ASDF ASDF ASDF ASDF
ASDF ASDF ASDF ASDF ASDF ASDF ASDF
</td></tr>
</TABLE>
</body></html>
--------------------------------------------------
|
|
||
Comment 2•27 years ago
|
||
Sorry, small typo: should have said: "... this bug (#2051) is independent ... "
First thing I notice about this page is the fonts are wrong. They're too
big. This leads to the form elements as well as the text being too big.
Perhaps the maxElementSize for the content is being measured at one font but
rendered in another?
|
|
||
Updated•27 years ago
|
Assignee: peterl → kipp
|
|
||
Comment 4•27 years ago
|
||
This page now asserts in nsInlineReflow::CanPlaceFrame. This appears to be the
same freed style context bug we discussed for Demo5.
It seems the original bug was caused by style contexts getting switched during
reflow. That explains the discrepancy between measured sizes.
Feel free to hand this bug off once you add the debug code we discussed to catch
these kinds of errors.
An update from peter:
Yes. I added two asserts, one to SetStyleContext, the other to
ReResolveStyleContext (back on 2/4 it was mentioned in a CVS update).
I had to immediately comment out the one in ReResolveStyleContext because it was
getting tripped all over the place. (I seem to recall it was
first-line/letter code tripping it?)
You should uncomment the one in ReResolveStyleContext and then fix that problem.
When I looked at it last, it seemed that we probably really need to be
re-resolving during reflow there. If that's the case, then you need to add some
code to the reflow states to reset the cached style data when that
happens...
|
|
||
Comment 7•27 years ago
|
||
per leger, assigning QA contacts to all open bugs without QA contacts according
to list at http://bugzilla.mozilla.org/describecomponents.cgi?product=Browser
|
||
Comment 8•27 years ago
|
||
I cannot reproduce any of the problems mentioned on this page. I would hazard
a guess that these problems were all fixed during the great rewrites of the
inline and first-* code.
I suggest this can be marked WORKSFORME unless someone sees a reason for this
bug to stay open.
[Note. The <FONT> related layout issues on the bignosebird.com page are covered
by bug 6233.]
|
|
||
Updated•27 years ago
|
Priority: P2 → P3
Whiteboard: WORKSFORME?
Target Milestone: M15
|
|
||
Comment 9•27 years ago
|
||
|
|
||
Updated•27 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 27 years ago
Resolution: --- → WORKSFORME
Whiteboard: WORKSFORME?
|
|
||
Comment 10•27 years ago
|
||
I cannot reproduce this bug. Marking WORKSFORME. Please reopen if the page
still has a layout problem.
|
|
||
Updated•27 years ago
|
Status: RESOLVED → VERIFIED
|
|
||
Comment 11•27 years ago
|
||
Using 5/24 Apprunner, verified WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•