Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updates to enable SMIL in all versions of IE #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jonfrost
Copy link

Hi Robin,

I updated all of the JS libraries to the latest version: Modernizr, SVG Web, and FakeSMILe.
Also, added support for animation in all versions of IE including IE9 RC.
Also, added a new *-smil test page, and updated the index.html file to link to these.
I tested that the new svg-boilerplate-smil.html file works in the latest version of these browsers:
Opera 11, Firefox 4, Chrome. Also in IE9 RC and IE8.

I did not change your svg-boilerplate.html file, but you will notice that in the new svg-boilerplate-smil.html file, I actually removed the Shim because IE9 and < IE8 did not like the Shim dynamically loading the FakeSMILe and SVG Web libraries. I was reveiving a variety of errors. Hopefully you can revisit the Shim using this working configuration. I will take another look at the Shim as well when time permits.
Best, Jon

…nd FakeSMILe.

Also, added support for animation in all versions of IE including IE9 RC.
Also, added a new *-smil test page, and updated the index.html file to link to these.

You will notice that in the new svg-boilerplate-smil.html file, I actually removed the
Shim because IE9 and < IE8 did not like the Shim dynamically loading the FakeSMILe and
SVG Web libraries.  I was reveiving a variety of errors.  Hopefully you can revisit the
Shim using this working configuration.  I will take another look at the Shim as well
when time permits.
@heymatthew
Copy link

Any update on this? This patch is almost 2 years old.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants