Closed Bug 55475 Opened 24 years ago Closed 21 years ago

Horizontal rule <hr> tags with noshade render oddly

Categories

(Core :: Layout, defect)

defect
Not set
normal

Tracking

()

RESOLVED WORKSFORME
Future

People

(Reporter: john, Unassigned)

References

()

Details

(Keywords: testcase)

Attachments

(1 file, 1 obsolete file)

On http://hadrian.ods.org/hr/ (a test case), the <hr noshade> element has a strange, small, disjoined blob on the end of an otherwise decently rendered horizontal rule. Linux nightly build 2000100509. Also, the style="color: red" directive doesn't work.
html element
Assignee: asa → clayton
Component: Browser-General → HTML Element
QA Contact: doronr → lorca
Due to dead DNS, test case (just a single <hr noshade>) is at http://212.56.123.232/hr/
The calculations for the rounded endcaps on noshade HR's is off. Whether you have a gap between the center section seems to depend on whether the HR has an even or odd width. Also, if you make really big HR's (size=70 say), and magnify them, you'll see the endcaps aren't aligned properly with the center section. I say dump the rounded endcaps personally. Or fix the calculations for them, I don't care. See nsHRFrame.cpp. This was also discussed in passing in the resolution of the "HR's are broken by -moz-bg-inset" bug recently fixed. Confirming (I hadn't gotten around to reporting this myself). FreeBSD 4.1 20001004xx
Status: UNCONFIRMED → NEW
Ever confirmed: true
Ack, was putting in some info to confirm, Lost my changes. Oh, well. What I saw on commercial branch bits: Mac (10-05-13): ***/****** ***/*** ***/*** Linux (10-06-10): ***/*** ***/*** * ***/*** * Win NT (10-05-08): ***/*** ****/**** ***/*** So Win seems to be fine, but Mac and Linux as way off.
OS: Linux → All
Hardware: PC → All
Dividing up claytons bugs to triage
Assignee: clayton → rods
buster, I seem to remember you working on hr rendering - reassigning
Assignee: rods → buster
the rounded corners are what Nav4 does, so Kipp added them to HRs. But IE doesn't do them, and they don't effect layout at all, they're just a painting phenomena. Since we don't do them right (and neither does Nav4!), I'd suggest we pull the code that does them. It's 100% trivial. Patch attached momentarily. This isn't a showstopper, but it cleans up some sloppiness that's a bit embarassing. Time to fix: 5 minutes. Risk to fix: 0 (does not change layout at all, uses a well-tested code path in all cases, rather than just some cases.) Benefit: layout looks better on Mac and Linux.
Status: NEW → ASSIGNED
Attached patch trivial fix, remove buggy arc-drawing code (obsolete) (deleted) — Splinter Review
per private conversation with jar, milestone -> Future. This will not make NS6 RTM.
Target Milestone: --- → Future
Low risk per buster's comment-can we put this into the trunk?
Keywords: nsbeta1, testcase
Reassigning QA Contact for all open and unverified bugs previously under Lorca's care to Gerardo as per phone conversation this morning.
QA Contact: lorca → gerardok
qa contact updated.
QA Contact: gerardok → bsharma
This patch still looks good to me. jst, roc+moz, can you give [s]r to this one-liner?
Woah. I have a better fix in bug 38370 that doesn't get rid of the arcs, and has no blobs. I would recommend _not_ checking this in, and instead fixing the bugs that block the checking in of the fix to bug 38370.
Depends on: 38370
*** Bug 97987 has been marked as a duplicate of this bug. ***
SPAM. HTML Element component is deprecated, changing to Layout component. See bug 88132 for details.
Come on Bugzilla, you can do it...
Component: HTML Element → Layout
An <hr noshade> tag used to (not so old Mozillas, NS4, etc) render as a black line. It's rendering as a gray line now (2001090605-macosX-trunk).
QA Contact: bsharma → moied
Build reassigning Buster's bugs to Marc.
Assignee: buster → attinasi
Status: ASSIGNED → NEW
Blocks: 104166
*** Bug 106606 has been marked as a duplicate of this bug. ***
Attachment #16780 - Attachment is obsolete: true
Assignee: attinasi → other
Priority: P3 → --
Came across this bug whilst searching for dupes for bug 216084 (it isn't a dupe). Realised this is probably fixed. The old test page isn't accessible any more but I've created a test case using the info in the bug. That appears to render fine with the noshade attribute specified - there are no "disjoined blobs" anywhere in the latest nightly (2003081303). Marking as works for me. (Note that the styling of <hr>s is a whole separate issue...
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → WORKSFORME
Probably fixed by bug 38370?
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: