Closed
Bug 595650
Opened 15 years ago
Closed 15 years ago
fix border-radius computed style code to reflect new meaning of vertical % values
Categories
(Core :: CSS Parsing and Computation, defect)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
FIXED
mozilla2.0b7
| Tracking | Status | |
|---|---|---|
| blocking2.0 | --- | betaN+ |
People
(Reporter: dbaron, Assigned: dbaron)
References
Details
(Keywords: css3)
Attachments
(1 file)
|
8.32 KB,
patch
|
bzbarsky
:
review+
|
Details | Diff | Splinter Review |
In bug 471643 we changed the meaning of percents on border-radius but didn't change the computed style code to match. We need to make the appropriate changes to nsComputedDOMStyle::GetEllipseRadii.
|
Assignee | |
Updated•15 years ago
|
blocking2.0: --- → betaN+
|
|
Assignee | |
Comment 1•15 years ago
|
||
I'll just fix this in bug 595651, since most of the code I'd write for this would go away there if I fixed them separately.
|
|
Assignee | |
Comment 2•15 years ago
|
||
Actually, that's not true, because of outline-radius.
|
|
Assignee | |
Comment 3•15 years ago
|
||
Attachment #477993 -
Flags: review?(bzbarsky)
|
|
Assignee | |
Updated•15 years ago
|
Whiteboard: [needs review]
|
||
Comment 4•15 years ago
|
||
Comment on attachment 477993 [details] [diff] [review]
patch
r=me
Attachment #477993 -
Flags: review?(bzbarsky) → review+
|
|
Assignee | |
Updated•15 years ago
|
Whiteboard: [needs review] → [waiting to land until after 4.0b7 freeze]
|
|
Assignee | |
Comment 5•15 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Whiteboard: [waiting to land until after 4.0b7 freeze]
Target Milestone: --- → mozilla2.0b8
|
||
Updated•15 years ago
|
Target Milestone: mozilla2.0b8 → mozilla2.0b7
You need to log in
before you can comment on or make changes to this bug.
Description
•