114706 - resizing window only triggers events if horizontally resized

Status: RESOLVED FIXED

(Core :: Layout: Tables, defect, P2)

Description

[Dodger]

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (Windows; U; Win98; en-US; rv:0.9.6) Gecko/20011120
BuildID:    2001112009

When resizing a window, TABLEs with height set and specified as a percentage
should redraw with the window to maintain a constant percentage.

Instead, when the page is resized horizontally, the table resizes but if only
the vertical aspect of the window is changed, the table does not redraw.

Reproducible: Always
Steps to Reproduce:
1.load page www.perl5cgi.com (or org or net)
2.change horizontal aspect of window
3.change vertical aspect without changing horizontal aspect (i.e. drag up and
down on the bottom resize handle of the window).

Actual Results:  1.page loaded
2.table redrew
3.nothing -- table did not redraw (thus leaving a large orange expanse at the
bottom when enlarged, or unnecessary scrollbars when shrunk)

Expected Results:  1.page loaded
2.table redrew
3.table should have redrawn

Support for table height tags is a more recent feature.
This feature requires the ability to detect a change in the vertical aspect of
the page. It appears that while the support for the tag property has been added,
redrawing support has not. The fact that a table properly resizes when the
horizontal aspect is changed seems to be a byproduct of the requirement that
percentage WIDTH properties require a redraw on resize, but would not need code
to detect a vertical resize.

Comment 1

[Boris Zbarsky [:bzbarsky]]

confirming.  making the page shorter reflows correctly, but making it taller
does live an orange stripe at the bottom.

Comment 2

[karnaze (gone)]

Attachment: reduced test case

Comment 3

[Greg Valure]

Attachment: further reduced testcase

It doesn't even need nested tables.  What it does need is the TD to have a
height property.  Having only the height in table or putting it in TR doesn't
cause the problem.

Comment 4

[Dodger]

The original page I noted this on -- www.perl5cgi.com -- does not contain height
tags in the TRs or TDs at all. It only has 100% height tags on the outer and
second tables (the innermost table has no formatting applied to it and does not
appear to cause any issues as it fits to its contents).

In my Mozilla the reducing the vertical aspect doesn't work, either. The effect
of reduced test case #1 is as follows:
    The outermost table reflows properly. This means the space between the top
and bottom borders of the outer table and the top and bottom borders of the
inner table increase as the page is enlarged vertically.
    The innermost table does not reflow unless the horizontal aspect is changed.
Thus, the page gains a scrollbar and does not fit on the screen.

Comment 5

[karnaze (gone)]

Attachment: patch to fix the bug

Comment 6

[Alexandru Savulov]

Comment on attachment 61958 [details] [diff] [review]
patch to fix the bug

r= alexsavulov

Comment 7

[Marc Attinasi]

Comment on attachment 61958 [details] [diff] [review]
patch to fix the bug

sr=attinasi

Comment 8

[karnaze (gone)]

->m099

Comment 9

[karnaze (gone)]

Attachment: patch to fix the bug

The previous patch is obsolete, since percent height caluclations have changed
a lot since then.

Comment 10

[kinmoz]

Comment on attachment 88335 [details] [diff] [review]
patch to fix the bug

sr=[e-mail]

Comment 11

[Alexandru Savulov]

Comment on attachment 88335 [details] [diff] [review]
patch to fix the bug

r= alexsavulov

Comment 12

[karnaze (gone)]

The patch is in the trunk.

Comment 13

[Daniel Wang]

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