Closed Bug 297008 Opened 20 years ago Closed 20 years ago

radial gradients: fx,fy not set correctly when only cx,cy given

Categories

(Core :: SVG, defect)

x86
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: alex, Assigned: scootermorris)

References

()

Details

Attachments

(1 file)

According to the SVG Specs, fx and fy attribs on a radial gradient should
coincide with cx, cy when not specified. Currently in Mozilla SVG they are set
to 50%.

Testcase at http://www.croczilla.com/~alex/gradient-bug.svg
Here is a patch that does the right thing.  The code was there to do it, but it
was in the wrong place :-(
Assignee: general → scootermorris
Status: NEW → ASSIGNED
Attachment #185812 - Flags: review?(alex)
Comment on attachment 185812 [details] [diff] [review]
Patch to comply with the spec

r=afri
Attachment #185812 - Flags: review?(alex) → review+
Attachment #185812 - Flags: approval1.8b3?
Comment on attachment 185812 [details] [diff] [review]
Patch to comply with the spec

a=tor
Attachment #185812 - Flags: approval1.8b3? → approval1.8b3+
Checked in:

Checking in nsSVGGradientFrame.cpp;
/cvsroot/mozilla/layout/svg/base/src/nsSVGGradientFrame.cpp,v  <-- 
nsSVGGradientFrame.cpp
new revision: 1.13; previous revision: 1.12
done
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: