Bug 14312 – Removing an attribute has no effect on SVG

• Status: RESOLVED
• Resolution: FIXED
• Component: SVG

[Bruce Rindahl]

The DOM methods removeAttribute and removeAttributeNS are apparently not implemented.  I can add an attribute using .setAttributeNS(null,"transform","translate(0,100)") .  It cannot be removed using
.removeAttributeNS(null,"transform").  Thsishould be allowed per the specs.

[Bruce Rindahl]

Created attachment 15185 [details]
SVG file showing behavior

Clicking on the green rectangle shifts the blue rectangle downward.  Clicking on the red rectangle should shift the blue rectangle back up but it doesn't.

[Alexey Proskuryakov]

Actually, the attribute is successfully removed, you can check that by entering the string below in the address bar:

javascript:alert(document.getElementById("blue").getAttributeNS(null,"transform"))

However, this has no effect on rendering for some reason (even if I resize the window for re-layout).

[Bruce Rindahl]

(In reply to comment #2)
I can confirm that.  Something else must be needed to refresh.  Setting the transform to "scale(1)" does make it work.
Enter:
javascript:document.getElementById("blue").setAttributeNS(null,"transform","scale(1)") 

[Rob Buis]

Created attachment 15684 [details]
First attempt

This should do it. However it is done on trunk, I am sure I can find some victim to apply it to feature branch, or it takes a while longer until I am back on that branch, but I am heading back there for sure.
Cheers,

Rob.

[Nikolas Zimmermann]

Comment on attachment 15684 [details]
First attempt

Please also land in feature-branch, my tree is blocked by patches, too.

[Rob Buis]

Landed in r24640 on feature branch.