131233 - rounded borders and backgrounds not working for tall elements

Status: VERIFIED FIXED

(Core :: CSS Parsing and Computation, defect, P5)

Description

[Tam]

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (X11; U; Linux i586; en-US; rv:0.9.8) Gecko/20020214
BuildID:    0.9.8

Hi, the bug is: When you view this page in full screen, there is noch problem,
in some other windowsizes there is. You can see the border and the blue
background ABOVE the text, but the <div> tag is around it.The IE for instance
displays it correctly.



Reproducible: Always
Steps to Reproduce:
1. Open Page in Mozilla
2. maximize the window
3. resize the window to a smaller one
4. if it does not show the error resize again because it seems to be a special
window size.

Actual Results:  You can see the border and the blue background ABOVE the text

Expected Results:  Show the border whitch is above the text, just AROUND the
text as done correctly in some resolution.

at 1152x864 in full screen mode no error occures.

Code example:

<div class="rechts">
<?PHP
include ("$topic.php");
?>
</div>

in the $topic included file it is:

<?PHP include ("bofh/$story.inc"); ?>

the sec parameter is just for the menu ob the left.

Comment 1

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

I suspect this may be a duplicate of my ConditionRect bug, except that the
height seems a little too small for that to be the issue.  It seems like there's
an issue with the rounded-border background drawing code as well.  Although this
could be some other issue with the rounded-border code.

FWIW, the window widths where I see the problem are very narrow ones.

Comment 2

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Don't set the target milestone, please.  That's for the bug owner only.  (Also,
setting a target in the past might lead to a bug being ignored.)

Comment 3

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

There were some PRInt16 values storing twip-valued numbers.  Fix to be attached.

Comment 4

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Attachment: patch

Comment 5

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

They might have been pixel valued, actually.  But either way -- we should be
able to handle rounded borders on documents taller than 32767 pixels.

Comment 6

[Marc Attinasi]

Comment on attachment 74390 [details] [diff] [review]
patch

sr=attinasi

Comment 7

[Hixie (not reading bugmail)]

Attachment: Self explanatory testcase: standard compliant version

Comment 8

[Hixie (not reading bugmail)]

Attachment: Self explanatory testcase: -moz-border-radius version

Comment 9

[dcone (gone)]

Comment on attachment 74390 [details] [diff] [review]
patch

r=dcone

Comment 10

[Asa Dotzler [:asa]]

Comment on attachment 74390 [details] [diff] [review]
patch

a=asa (on behalf of drivers) for checkin to the 1.0 trunk

Comment 11

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

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

Comment 12

[Hixie (not reading bugmail)]

dbaron: the testcases show this problem also for some extreme non cases of
non-rounded borders. Does your patch fix that too?

Comment 13

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Extreme cases of non-rounded borders would be covered under my bug with
ConditionRect in the summary.  (I'm assuming you're talking about the bug that,
when you just hit the criterion for causing it, will show up at the ends of the
page but not the middle.)

Comment 14

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Fix checked in 2002-03-16 07:25 PST.

Comment 15

[Hixie (not reading bugmail)]

dbaron: No, I'm referring to the effect seen on attachment 74414 [details] when the width
of the window is just enough for one word. The effect seen is the same as with
the border-radius case: the border goes up instead of down.

Comment 16

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Yes, but if you scroll down to the middle of the document, you'll see the border
appear again.  (You might have to fiddle with the font size, but that's what I
see.)  And that's how I know it's a ConditionRect problem (bug 115526).

Comment 17

[Hixie (not reading bugmail)]

So it does. Cool.

VERIFIED.

Comment 18

[Kai Lahmann (is there, where MNG is)]

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