82768 - HR with style border: 2px #FFFFFF groove; becomes dots

Status: RESOLVED WORKSFORME

(Core :: Layout, defect)

Description

[Tim Powell]

With Mozilla/5.0 (Windows; U; WinNT4.0; en-US; rv:0.9+) Gecko/20010525,
specifying a 2px groove in the style for the HR causes 2 dots to appear at each
end of the expected groove and nothing in between. It should draw a groove.

This is a regression that occurred sometime right before 0.9. 0.8.1 did not
exhibit this problem.

Comment 1

[Tim Powell]

Attachment: testcase with hr and groove

Comment 2

[Tim Powell]

This code works with Netscape 4.x, Netscape 6.0, IE4, and IE5.

Comment 3

[doctor__j]

I think that's a dupe of bug 81776.

Comment 4

[Tim Powell]

They certainly look like similar bugs. One difference I see is that bug 81776 
is seeing this behavior in table cells. In this one it's caused by a div and 
styles.

Comment 5

[TommyBee (No E-mail)]

Seeing this on Mac OS 9.1 / Build 2001053005 as well.  Platform / OS -> All?

Comment 6

[Skewer]

The problem appears to be a failure to draw the top and bottom borders of the
HR. The space is provided in the layout in my testcase, but the borders are
either not drawn or only drawn one pixel thick.

I'm seeing these results in my testcase using 2001070604 Win98:

1. Expected behavior.
2. 2px small dots on left and right edges of HR.
3. HR takes up correct amount of space, but top and bottom borders are not
displayed.
4. HR takes up correct amount of space, but top and bottom borders are only 1px
thick, instead of the expected 50px.5. Same as #4.
6. HR takes up correct amount of space, but top and borders are not shown, and
left and right borders take up too much vertical space, overlapping text.

Comment 7

[Skewer]

Attachment: HR/CSS1 testcase

Comment 8

[Tim Powell]

Proposing for 1.0. This blocks css1.

Comment 9

[karnaze (gone)]

Reassigning to attinasi.

Comment 10

[Marc Attinasi]

Our style rule for HR could be changed to force the -moz-bg-inset border. That
would mean that we ignore author-defined styling of HR, which is probably better
than what we do now!

hr {
  display: block;
  border: 1px -moz-bg-inset !important;
  margin: 0 auto 0 auto;
}

NOTE: HR is a true pain - I recommend that we take this easy approach because HR
is like a house of cards...

Comment 11

[karnaze (gone)]

->m1.1

Comment 12

[Chris Creal]

Attachment: example page with disputed code

<hr width="50%" size="2" style="color:orange; border:dotted orange 2px"/>

the previous code is not rendered properly under Mozilla 1.4 (appears as two
dots in sizes 3 and below).

this bug IS fixed in 1.5 beta, BUT a different bug arised. Now sizes of 4 and
below look exactly the same.

Comment 13

[Boris Zbarsky [:bzbarsky]]

This bug got fixed by the hrframe removal.