Closed
Bug 847600
Opened 11 years ago
Closed 11 years ago
Remove SVGEvent
Categories
(Core :: DOM: Events, defect)
Tracking
()
RESOLVED
FIXED
mozilla22
People
(Reporter: smaug, Assigned: smaug)
References
Details
(Keywords: dev-doc-needed, site-compat)
Attachments
(2 files)
15.48 KB,
patch
|
jwatt
:
review+
|
Details | Diff | Splinter Review |
15.77 KB,
patch
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Updated•11 years ago
|
Summary: Paris bindings for SVGEvent and SVGZoomEvent → Paris binding for SVGEvent
Assignee | ||
Comment 1•11 years ago
|
||
It is not clear to me why we need SVGEvent interface at all. Could we remove it?
Assignee | ||
Comment 2•11 years ago
|
||
Ping. I'd like to avoid implementing useless events in Webidl.
Comment 3•11 years ago
|
||
The SVGEvent interface exists in SVG 1.0: http://www.w3.org/TR/SVG10/idl.html http://www.w3.org/TR/SVG10/script.html but no longer exists in SVG 1.1: http://www.w3.org/TR/SVG11/idl.html http://www.w3.org/TR/SVG11/script.html Seems like it can just go away. (Note the message types SVGLoad, SVGUnload, SVGAbort, SVGError, SVGResize and SVGScroll still exist, but we don't need the SVGEvent interface for them.)
Assignee | ||
Comment 4•11 years ago
|
||
Ok, thanks!
Assignee | ||
Comment 5•11 years ago
|
||
Assignee: nobody → bugs
Attachment #723111 -
Flags: review?(jwatt)
Assignee | ||
Comment 6•11 years ago
|
||
https://tbpl.mozilla.org/?tree=Try&rev=84eab1736339
Updated•11 years ago
|
Attachment #723111 -
Flags: review?(jwatt) → review+
Assignee | ||
Updated•11 years ago
|
Summary: Paris binding for SVGEvent → Remove SVGEVent
Assignee | ||
Updated•11 years ago
|
Summary: Remove SVGEVent → Remove SVGEvent
Assignee | ||
Comment 7•11 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/d8648c78e3ad
Comment 8•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/d8648c78e3ad
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla22
Comment 9•11 years ago
|
||
I've added this bug to the compatibility doc. Please correct the info if wrong. https://developer.mozilla.org/en-US/docs/Site_Compatibility_for_Firefox_22
Keywords: dev-doc-complete
Comment 10•11 years ago
|
||
https://developer.mozilla.org/en-US/docs/DOM/Mozilla_event_reference too
Updated•11 years ago
|
Keywords: dev-doc-complete → dev-doc-needed
Updated•11 years ago
|
Keywords: site-compat
You need to log in
before you can comment on or make changes to this bug.
Description
•