Closed
Bug 55475
Opened 24 years ago
Closed 22 years ago
Horizontal rule <hr> tags with noshade render oddly
Categories
(Core :: Layout, defect)
Core
Layout
Tracking
()
RESOLVED
WORKSFORME
Future
People
(Reporter: john, Unassigned)
References
()
Details
(Keywords: testcase)
Attachments
(1 file, 1 obsolete file)
177 bytes,
text/html
|
Details |
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.
Comment 1•24 years ago
|
||
html element
Assignee: asa → clayton
Component: Browser-General → HTML Element
QA Contact: doronr → lorca
Reporter | ||
Comment 2•24 years ago
|
||
Due to dead DNS, test case (just a single <hr noshade>) is at
http://212.56.123.232/hr/
Comment 3•24 years ago
|
||
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
Comment 6•24 years ago
|
||
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
per private conversation with jar, milestone -> Future. This will not make NS6 RTM.
Target Milestone: --- → Future
Comment 10•24 years ago
|
||
Low risk per buster's comment-can we put this into the trunk?
Comment 11•24 years ago
|
||
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
Comment 13•24 years ago
|
||
This patch still looks good to me.
jst, roc+moz, can you give [s]r to this one-liner?
sr=roc+moz
Comment 15•24 years ago
|
||
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
Comment 16•23 years ago
|
||
*** 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).
Comment 20•23 years ago
|
||
Build reassigning Buster's bugs to Marc.
Assignee: buster → attinasi
Status: ASSIGNED → NEW
Comment 21•23 years ago
|
||
*** Bug 106606 has been marked as a duplicate of this bug. ***
Updated•23 years ago
|
Attachment #16780 -
Attachment is obsolete: true
Comment 22•22 years ago
|
||
Assignee: attinasi → other
Priority: P3 → --
Comment 23•22 years ago
|
||
Comment 24•22 years ago
|
||
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: 22 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.
Description
•