Bug 14312 - Removing an attribute has no effect on SVG
Summary: Removing an attribute has no effect on SVG
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: SVG (show other bugs)
Version: 523.x (Safari 3)
Hardware: PC OS X 10.4
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-06-22 12:11 PDT by Bruce Rindahl
Modified: 2007-07-26 00:43 PDT (History)
0 users

See Also:


Attachments
SVG file showing behavior (1.13 KB, image/svg+xml)
2007-06-22 12:15 PDT, Bruce Rindahl
no flags Details
First attempt (8.80 KB, patch)
2007-07-25 12:56 PDT, Rob Buis
zimmermann: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Bruce Rindahl 2007-06-22 12:11:21 PDT
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.
Comment 1 Bruce Rindahl 2007-06-22 12:15:20 PDT
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.
Comment 2 Alexey Proskuryakov 2007-07-06 05:22:36 PDT
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).
Comment 3 Bruce Rindahl 2007-07-06 10:46:38 PDT
(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)") 
Comment 4 Rob Buis 2007-07-25 12:56:40 PDT
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.
Comment 5 Nikolas Zimmermann 2007-07-25 13:37:57 PDT
Comment on attachment 15684 [details]
First attempt

Please also land in feature-branch, my tree is blocked by patches, too.
Comment 6 Rob Buis 2007-07-26 00:43:13 PDT
Landed in r24640 on feature branch.