15882 - FORM text is overlapping an image

Status: VERIFIED FIXED

(Core :: Layout, defect, P3)

Description

[Mats Palmgren (inactive)]

FORM text is overlapping an image in the "Weather box" at http://www.cnn.com
Occurs in: apprunner 1999-10-07-14-M11 on Windows NT 4.0 SP5.
Does not occur in: Navigator 3.04, Communicator 4.7 and IE4 (same OS as above).

Comment 1

[Mats Palmgren (inactive)]

Attachment: Testcase distilled from http://www.cnn.com

Comment 2

[troy]

It looks okay with my build pulled this morning. Are you still seeing the
problem with today's build?

Comment 3

[Mats Palmgren (inactive)]

Yes, it occurs in apprunner 1999-10-08-10-M11 too.

Comment 4

[troy]

Kipp, it sounds like a block issue. However, I can't reproduce this either in
apprunner or viewer. It looks fine to me with my tree built this morning

Comment 5

[Mats Palmgren (inactive)]

Attachment: screenshot of problem at www.cnn.com

Comment 6

[troy]

Some extra info. The screen shot Mats provided that demonstrates the problem
shows one obvious difference: the font size is much larger in his case where it
doesn't display properly

Mats, are you using large fonts on your machine?

Comment 7

[Mats Palmgren (inactive)]

Yes, I am using "Large Fonts" and 1152x864 pixels on a 20 inch monitor.

Comment 8

[kipp]

Can you try todays build? I checked in a bugfix to the text-frame code that
might have caused this problem...I haven't been able to reproduce it :-(

Comment 9

[Mats Palmgren (inactive)]

Attachment: testcase #2 (for those of you with "Small Fonts")

Comment 10

[Mats Palmgren (inactive)]

Bug occurs in nightly build 1999-10-13-13-M11 too, both under "Large Fonts"
(testcase 1) and "Small Fonts" (testcase 2).

Comment 11

[kipp]

I think this bug has already been fixed. The problem was almost certainly caused
by reflow optimizations that we have. There is a known lingering problem there
(15059), but other than that we think we have them fixed..Can you check again
and see? I can't recreate it here -- nobody here has :-(

Comment 12

[Mats Palmgren (inactive)]

Tried out 1999-10-20-11-M11 using both Large and Small Fonts. Checked both
testcases and http://www.cnn.com and it looked OK in all permutations.

Comment 13

[Chris Petersen]

Fixed in the Oct 21 build (1999102110).

Comment 14

[Pulsebot]

Pushed by ffxbld-merge:
https://hg.mozilla.org/releases/mozilla-beta/rev/20e3fb4678c8
[fenix] Closes https://github.com/mozilla-mobile/fenix/issues/15882: check for account state abnormalities after account manager is ready