78992 - HR with size="1" (height) produces size="2" effect.

Status: RESOLVED FIXED

(Core :: Layout, defect, P3)

Description

[/\/\arcio Galli]

Whatever value I try with size attribute in HR element, there is no action.

Build Gecko/20010504
win98
will attach simple test.

Comment 1

[/\/\arcio Galli]

Attachment: Simple HTML with an HR size=1 inside.

Comment 2

[Conor Lennon]

What are you expecting to happen?
The size attribute specifies the height of the HR element.
If you set the size to be 10, you'll correctly get an HR element with the
correct height.

Comment 3

[/\/\arcio Galli]

In the build I tested the size=1 produced the size=2 effect. Seems like its
working for you. Roger, do you confirm this?

Comment 4

[Roger C. Soares]

Yeah! Linux nb 2001050121 showing hr element with incorrect size (height).

Comment 5

[Keyser Sose]

Marking NEW based on confirmation.

Comment 6

[/\/\arcio Galli]

Adding aruner....cc... as an FYI watcher...

Comment 7

[reorx]

This displays properly under mozilla 0.8

Comment 8

[/\/\arcio Galli]

so marking.. regression

Comment 9

[/\/\arcio Galli]

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

Comment 10

[/\/\arcio Galli]

Changing the summary. 

Comment 11

[Heikki Toivonen (remove -bugzilla when emailing directly)]

HTML Element component is deprecated, and should be removed from Bugzilla.
Clayton is not the correct owner for these. Reassigning to layout.

Comment 12

[Ashley Bischoff (blog at handcoding.com)]

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

Comment 13

[/\/\arcio Galli]

prize=pizza (whoever fix this bug marcio will pay a pizza). 

Comment 14

[R.K.Aa.]

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

Comment 15

[karnaze (gone)]

We are making the hr larger than 1 pixel, unlike IE and Nav. -->attinasi.

Comment 16

[Håkan Waara]

Attachment: Testcase

This is a testcase with 4 <HR>s in the following order:

<hr/>
<hr size="1"/>
<hr size="8"/>
<hr size="8" NOSHADE/>

Comment 17

[Håkan Waara]

Attachment: IE 5.5 output

IE 5.5 on Windows 98's output for the testcase

Comment 18

[Håkan Waara]

Attachment: NS 4.x's output

NS 4.x's rendering of the testcase

Comment 19

[Håkan Waara]

Attachment: Mozilla 2001-11-08 output

A Mozilla 2001-11-08 rendering of the testcase.

Comment 20

[Håkan Waara]

Attachment: Mozilla patched output

The rendering of Mozilla with a patch of mine applied.

Comment 21

[Håkan Waara]

As the screenshots reveal NOSHADE is broken in both the current Mozilla and the 
old NS4.x. It does only work in IE5.5 where it renders the HR solid with 
NOSHADE on.

It also looks (according to the screenshot of my patched build) that my patch 
fixes this bug.

Comment 22

[Håkan Waara]

Attachment: Patch -u

Patch for this bug.

Comment 23

[Håkan Waara]

That patch is not correct, it has a bad, old regression.

I'll try to come up with a better fix.

Comment 24

[Hixie (not reading bugmail)]

I would recommend not fixing this. Instead, let's get my fix for bug 38370
massaged into the post-hyatt-rule-system world and then let's fix the bugs
blocking that. That should make _this_ bug disappear on its own.

Comment 25

[R.K.Aa.]

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

Comment 26

[R.K.Aa.]

bug 58447 and bug 55475 seem related.

Comment 27

[Marc Attinasi]

Moving Mozilla 1.01 bugs to 'future' milestone with priority P1

I will be pulling bugs from 'future' milestones when scheduling later work.

Comment 28

[R.K.Aa.]

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

Comment 29

[Mats Palmgren (inactive)]

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

Comment 30

[Eugene Savitsky]

Can we move this to next milestone? I advocate mozilla/NN and this was the first
bug.

Tnx.

Comment 31

[R.K.Aa.]

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

Comment 32

[R.K.Aa.]

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

Comment 33

[Boris Zbarsky [:bzbarsky]]

Fixed by bug 38370