Closed
Bug 298545
Opened 19 years ago
Closed 19 years ago
border-style: double displayed as solid
Categories
(Core :: CSS Parsing and Computation, defect)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
FIXED
People
(Reporter: mbateman, Assigned: zbraniecki)
References
()
Details
(Keywords: regression, Whiteboard: [no l10n impact])
Attachments
(2 files)
(deleted),
text/html
|
Details | |
(deleted),
patch
|
roc
:
review+
roc
:
superreview+
asa
:
approval1.8b4+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.8b2) Gecko/20050619 Camino/0.9a1
Build Identifier: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.8b2) Gecko/20050619 Camino/0.9a1
In the page at http://www.the-undercurrent.com, for example, the border under
the links should be double. It's set to width 3px style double. But it shows up
as solid in Camino.
Reproducible: Always
Steps to Reproduce:
1. make a CSS element with border-style: double and sufficient width for this to
render
2. view the page in Camino
3. ponder the universe
Actual Results:
It renders a solid border.
Expected Results:
Render a double border.
Comment 1•19 years ago
|
||
Core bug. Checking for dupes before switching.
Comment 2•19 years ago
|
||
-> Core.
I'll create a testcase later. Also, if this is a dupe, I apologize. I couldn't
find it.
This *could* be poorly formed CSS but it does validate.
Assignee: pinkerton → dbaron
Component: Page Layout → Style System (CSS)
Product: Camino → Core
Version: unspecified → Trunk
Comment 3•19 years ago
|
||
This seems only be a problem when border-width is defined. Otherwise,
border-style: double shows perfectly as demonstrated in this test case.
Updated•19 years ago
|
Status: UNCONFIRMED → NEW
Ever confirmed: true
is this a recent regression? On June 08 there went a fix in for bug 1781.
Comment 5•19 years ago
|
||
(In reply to comment #4)
> is this a recent regression? On June 08 there went a fix in for bug 1781.
Yes it did. This works in the 6/8 build of Camino but not in the 6/9 build.
Comment 6•19 years ago
|
||
Broken on OS/2 & Linux current trunk too. Testcase upper p is thick solid in both.
OS: MacOS X → All
Hardware: Macintosh → All
Assignee | ||
Comment 8•19 years ago
|
||
ah, damn - it's such a trivial bug.
This patches fixes the testcase, URL, my internal testcases and
http://yano.jogger.pl/comment.php?eid=130987 .
Assignee | ||
Updated•19 years ago
|
Attachment #188701 -
Flags: superreview?(roc)
Attachment #188701 -
Flags: review?(roc)
Assignee | ||
Updated•19 years ago
|
Updated•19 years ago
|
Whiteboard: [no l10n impact]
Attachment #188701 -
Flags: superreview?(roc)
Attachment #188701 -
Flags: superreview+
Attachment #188701 -
Flags: review?(roc)
Attachment #188701 -
Flags: review+
Assignee | ||
Comment 9•19 years ago
|
||
Comment on attachment 188701 [details] [diff] [review]
patch
this trivial patch fixes regression with small widths in double borders.
Attachment #188701 -
Flags: approval1.8b4?
Updated•19 years ago
|
Attachment #188701 -
Flags: approval1.8b4? → approval1.8b4+
Comment 10•19 years ago
|
||
*** Bug 300890 has been marked as a duplicate of this bug. ***
Assignee | ||
Comment 11•19 years ago
|
||
*** Bug 301171 has been marked as a duplicate of this bug. ***
Comment 13•19 years ago
|
||
Checking in nsCSSRendering.cpp;
/cvsroot/mozilla/layout/base/nsCSSRendering.cpp,v <-- nsCSSRendering.cpp
new revision: 3.262; previous revision: 3.261
done
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Comment 14•18 years ago
|
||
The problem seems to be still present in Gran Paradiso alpha 4 on Mac Os X
Bottom and right borders of a button with border-style:double and border-width:3px are shown as solid. If you set a width and a height to the button double borders are ok.
Assignee | ||
Comment 15•18 years ago
|
||
I don't have Mac OS X :(
I'll try to confirm that and reproduce it on other platforms.
Comment 16•18 years ago
|
||
(In reply to comment #15)
> I don't have Mac OS X :(
>
> I'll try to confirm that and reproduce it on other platforms.
>
Sorry for my bad usage of BugZilla: I've already opened a new bug (I've been suggested that it's better than commenting a fixed bug) and discovered that this problem is solved in the latest nigthlies
https://bugzilla.mozilla.org/show_bug.cgi?id=379807
You need to log in
before you can comment on or make changes to this bug.
Description
•