Issue 616 in svgweb: document.getElementById() errors in IE9 Standards Mode for Non-Existent ID
svg...@googlecode.com
Owner: ----
Labels: Type-Defect Priority-Low
New issue 616 by joshua.p...@gmail.com: document.getElementById() errors in
IE9 Standards Mode for Non-Existent ID
http://code.google.com/p/svgweb/issues/detail?id=616
What steps will reproduce the problem?
1. Open any document which includes svgweb and is in IE9 standards mode
(mode can be turned on in toolbar of IE9's built in dev tools)
2. in the console, enter document.getElementById('somejunk_adsfasd')
What is the expected output? What do you see instead?
null, but get an error
What version of the product are you using? On what operating system,
browser, and version of Flash?
IE9, svgweb LA
Please provide any additional information below. Reduced test cases are
always appreciated!
svg...@googlecode.com
Comment #1 on issue 616 by emmanuel...@gmail.com: document.getElementById()
http://code.google.com/p/svgweb/issues/detail?id=616
I think this is a highly critical bug. svgweb brokes the most of javascript
apps with IE9.
Code to reprocuce, with IE9 in standard mode:
{{{
<html>
<head>
<script src="js/svgweb/src/svg.js" data-path="js/svgweb/src"
data-debug="false"></script>
<script type="text/javascript">
go = function() { alert($('Idontexist')); }
window.onsvgload = go ;
</script>
</head>
<body >
</body>
</html>
}}}
svg...@googlecode.com
Comment #2 on issue 616 by emmanuel...@gmail.com: document.getElementById()
http://code.google.com/p/svgweb/issues/detail?id=616
I think this is a highly critical bug. svgweb brokes the most of javascript
apps with IE9.
Code to reprocuce, with IE9 in standard mode:
{{{
<html>
<head>
<script src="js/svgweb/src/svg.js" data-path="js/svgweb/src"
data-debug="false"></script>
<script type="text/javascript">
go = function() { alert(document.getElementById('Idontexist')); }
svg...@googlecode.com
Comment #3 on issue 616 by joshua.p...@gmail.com: document.getElementById()
http://code.google.com/p/svgweb/issues/detail?id=616
I modified the compressed and uncompressed version to make this error go
away. I didn't document it too well, but you can run a diff.
Works for me!
Files attached.
Attachments:
svg.js 105 KB
svg-uncompressed.js 354 KB
svg...@googlecode.com
Comment #4 on issue 616 by emmanuel...@gmail.com: document.getElementById()
http://code.google.com/p/svgweb/issues/detail?id=616
I have tried your patch, unfortunately it doesn't fix the code example
above. The script still dies at getElementById call ( instead of
popup "null" ).
The error console shows (sorry it's in french ):
Journal : Error while firing onload: L’objet ne gère pas la propriété ou la
méthode « setProperty » svg-uncompressed.js, Ligne 443 Caractère 5
svg...@googlecode.com
Comment #5 on issue 616 by emmanuel...@gmail.com: document.getElementById()
http://code.google.com/p/svgweb/issues/detail?id=616
Oh, but the compiled version seems to solve the issue...
svg...@googlecode.com
Comment #6 on issue 616 by joshua.p...@gmail.com: document.getElementById()
http://code.google.com/p/svgweb/issues/detail?id=616
I believe the compiled patch also fixes issue 617. It's funny how deep the
ID problem was. Maybe the maintainers are listening to this thread and can
gain more insight form my hack-fix than I did.
svg...@googlecode.com
Comment #7 on issue 616 by emmanuel...@gmail.com: document.getElementById()
http://code.google.com/p/svgweb/issues/detail?id=616
I would be very curious to have the clear patch and some explanations !
I've tried to debug myself but I must admit it is hard for me. Thanks for
saving my app at short term . I hope mainteners will solve this problem for
a more long term consideration.
svg...@googlecode.com
Comment #8 on issue 616 by kajmagnu...@gmail.com: document.getElementById()
http://code.google.com/p/svgweb/issues/detail?id=616
@Joshua: I downloaded your patched svg.js,
and it solved this issue and issue 617 as well. Thanks!
(svg.js linked in your comment
http://code.google.com/p/svgweb/issues/detail?id=616#c3 )
svg...@googlecode.com
Comment #9 on issue 616 by alx...@gmail.com: document.getElementById()
http://code.google.com/p/svgweb/issues/detail?id=616
More anecdotal evidence, Joshua's patch solved issue 617 for me as well.
svg...@googlecode.com
Comment #10 on issue 616 by s...@antistatique.biz:
http://code.google.com/p/svgweb/issues/detail?id=616
version seem to works. Thanks a lot.
svg...@googlecode.com
Comment #11 on issue 616 by s...@antistatique.biz:
http://code.google.com/p/svgweb/issues/detail?id=616
modification..
svg...@googlecode.com
Comment #12 on issue 616 by s...@antistatique.biz:
http://code.google.com/p/svgweb/issues/detail?id=616
I get this diff and it should be quiet easy to rebuild the modification
http://www.diffchecker.com/5Ck5gv91
svg...@googlecode.com
Comment #13 on issue 616 by s...@antistatique.biz:
http://code.google.com/p/svgweb/issues/detail?id=616
Attachments:
svg-uncompressed.js 354 KB
svg...@googlecode.com
Comment #14 on issue 616 by AdamRac...@gmail.com: document.getElementById()
http://code.google.com/p/svgweb/issues/detail?id=616
other issues in the works? The last one was from 2011.
Is this project dead?
--
You received this message because this project is configured to send all
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings