Closed
Bug 620295
Opened 14 years ago
Closed 14 years ago
SVG <a> elements do not expose their href properties in JS
Categories
(Core :: SVG, defect)
Core
SVG
Tracking
()
RESOLVED
FIXED
People
(Reporter: heycam, Assigned: heycam)
Details
Attachments
(2 files, 1 obsolete file)
|
8.49 KB,
patch
|
jwatt
:
review+
|
Details | Diff | Splinter Review |
|
8.32 KB,
patch
|
roc
:
approval2.0+
|
Details | Diff | Splinter Review |
SVGAElement objects in the DOM should implement SVGURIReference and expose a .href property. Doing
javascript:alert(document.createElementNS('http://www.w3.org/2000/svg', 'a').href)
alerts "undefined", while
javascript:alert(document.createElementNS('http://www.w3.org/2000/svg', 'image').href)
alerts "[object SVGAnimatedString]".
|
Assignee | |
Updated•14 years ago
|
Assignee: nobody → cam
|
|
Assignee | |
Comment 1•14 years ago
|
||
Attachment #499210 -
Flags: review?(jwatt)
|
|
Assignee | |
Updated•14 years ago
|
Attachment #499210 -
Attachment is obsolete: true
Attachment #499210 -
Flags: review?(jwatt)
|
|
Assignee | |
Comment 2•14 years ago
|
||
|
||
Comment 3•14 years ago
|
||
Comment on attachment 499211 [details] [diff] [review]
with correct bug number in test comment this time
I'm assuming you meant to rerequest review on the new patch after it got cancelled on the obsolete patch.
So we just forgot to expose the logic for this. Oops. :/
Thanks for the thorough tests! Can you convert a-href-01.svg into a mochitest though please, since it's just testing DOM functionality. r=jwatt with that.
Attachment #499211 -
Flags: review+
|
|
||
Updated•14 years ago
|
OS: Mac OS X → All
Hardware: x86 → All
|
|
Assignee | |
Comment 4•14 years ago
|
||
Attachment #499222 -
Flags: approval2.0?
Attachment #499222 -
Flags: approval2.0? → approval2.0+
|
|
Assignee | |
Updated•14 years ago
|
Keywords: checkin-needed
|
||
Comment 5•14 years ago
|
||
You need to log in
before you can comment on or make changes to this bug.
Description
•