The default bug view has changed. See this FAQ.

fix dynamic matrix transforms

RESOLVED FIXED

Status

()

Core
SVG
RESOLVED FIXED
13 years ago
12 years ago

People

(Reporter: Alex Fritze, Assigned: Alex Fritze)

Tracking

({fixed1.8})

Trunk
x86
All
fixed1.8
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

13 years ago
Dynamic updates of transforms in code like the following are currently not
working. Upcoming patch fixes that.

<svg>
  <circle transform="matrix(1 0 0 1 0 0)"/>
</svg>

circle_element.transform.baseVal.getItem(0).matrix.a = 10;

Expected Result: new transform matrix [10 0 0 1 0 0] should be applied to circle.
Actual Result: matrix gets the new value, but circle is not redrawn with new
transform.
(Assignee)

Comment 1

13 years ago
Created attachment 158181 [details] [diff] [review]
fix
(Assignee)

Updated

13 years ago
Attachment #158181 - Flags: review?(tor)
r=+ from me. Not sure how to add that to the patch without removing the request
for review from tor.
Comment on attachment 158181 [details] [diff] [review]
fix

r=jonathan.watt@strath.ac.uk
As discussed with tor
Attachment #158181 - Flags: review?(tor) → review+
Alex, can you check in your patch so we can close this bug please?
Created attachment 187549 [details] [diff] [review]
fix updated to tip

Updated Alex's patch to tip. Alex, do you mind if I check this in for you?
Attachment #158181 - Attachment is obsolete: true
Attachment #187549 - Flags: approval1.8b3?

Updated

12 years ago
Attachment #187549 - Flags: approval1.8b3? → approval1.8b3+
Uhm, was the patch checked in? What's the status of this bug?
Comment on attachment 187549 [details] [diff] [review]
fix updated to tip

Requesting approval for branch. Alex says he doesn't mind if one of us checks
this in for him. This is SVG only and low risk IMO. It fixes a conformance
issue, so I think it's important we include this for Firefox 1.5.
Attachment #187549 - Flags: approval1.8b5?

Updated

12 years ago
Attachment #187549 - Flags: approval1.8b5? → approval1.8b5+
checked in on trunk and branch
Status: NEW → RESOLVED
Last Resolved: 12 years ago
Keywords: fixed1.8
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.