72181 - Tables do not reflow correctly on text resize

Status: VERIFIED FIXED

(Core :: Layout: Tables, defect)

Description

[AM]

Version:
Mozilla 0.8.1
Mozilla/5.0 (X11; U; Linux 2.2.16 i686; en-US; 0.8.1) Gecko/20010314
Download of new-cache build.

One of the nicer features of mozilla is the ^C+ ^C- zooming
in and out of a page. This should lead to a reflow otherwise
things get overlayed.

Example:
go to www.nyt.com
do ^C+ a couple of times to zoom in. The page is undreadable
as different columns overprint.
Feature is now useless.

Comment 1

[Fabian Guisset]

reassigning to the new cache people. 

Comment 2

[Boris Zbarsky [:bzbarsky]]

Not a new cache issue.  I see this in the 2001-03-14-08 nightly.

Comment 3

[Boris Zbarsky [:bzbarsky]]

More specifically, this broke sometime between the 2001-03-12-21 and
2001-03-13-08 nightlys (works in 12-21, not in 13-08)

Looks like a result of karnaze's checkins of "table reflow coalescing" (around
12-22).  Over to HTML tables

Comment 4

[Boris Zbarsky [:bzbarsky]]

Attachment: Simple testcase. Hit Ctrl-+ a few times....

Comment 5

[karnaze (gone)]

I didn't know about this feature and can't get it to work in NS6. Either it 
didn't work in NS6, or I don't know how to invoke it, in which case please help 
me. Is it - control C then +? The table reflow coalescing occured on 3/13. Did 
it break between 3/12 and 3/13 or on 3/8. I can't tell from the comment.

Comment 6

[karnaze (gone)]

Never mind about needing help. It is control +.

Comment 7

[Boris Zbarsky [:bzbarsky]]

This feature was not in Netscape 6, if I recall correctly.

This broke between the evening of 3/12 and the morning of 3/13, hence my guess
that the table reflow coalescing is responsible.

Comment 8

[karnaze (gone)]

Moving to m0.9

Comment 9

[karnaze (gone)]

Attachment: patch to fix the bug

Comment 10

[karnaze (gone)]

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

Comment 11

[Owen Williams]

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

Comment 12

[Amarendra Hanumanula]

QA contact update

Comment 13

[Chris Siebenmann]

 [e-mail]'s patch works great for me, although other
changes today rendered it invalid and non-applying (and, of course,
the problem is still there without it).

Comment 14

[Marc Attinasi]

sr=attinasi

Comment 15

[karnaze (gone)]

Attachment: better patch (ignore previous one)

Comment 16

[karnaze (gone)]

The patch is in.

Comment 17

[Boris Zbarsky [:bzbarsky]]

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

Comment 18

[Chris Petersen]

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

Comment 19

[Amarendra Hanumanula]

 This bug was fixed Marking verified

Comment 20

[:Gavin Sharp [email: [e-mail]]]

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