Closed Bug 552830 Opened 14 years ago Closed 14 years ago

Should not force rx=ry for a rect by clamping if one is omitted.

Categories

(Core :: SVG, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: longsonr, Assigned: longsonr)

References

Details

Attachments

(1 file)

If rx or ry is omitted from a rect and rx or ry need clamping the clamping should just affect one or the other as appropriate and not force rx to stay = ry

see http://dev.w3.org/cvsweb/~checkout~/SVG/profiles/1.1F2/test/svg/shapes-rect-07-f.svg?rev=1.1&content-type=application/xml

which Opera renders as expected.
Attachment #432942 - Attachment is patch: true
Attachment #432942 - Flags: review?(jwatt)
Comment on attachment 432942 [details] [diff] [review]
patch (I'll add a reftest before I land it)

I can't think of any really strong reason for favoring our strategy over Opera's, so I guess this is okay.

Note that you should also delete the sentence starting "We do this after clamping rx and...".
Attachment #432942 - Flags: review?(jwatt) → review+
pushed http://hg.mozilla.org/mozilla-central/rev/a92f5fcdaa56
Status: NEW → RESOLVED
Closed: 14 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Assignee: nobody → longsonr
Blocks: ietestcenter
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: