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

IE8 causes following script to stop #5

Open
cnsgithub opened this issue Jun 1, 2016 · 0 comments
Open

IE8 causes following script to stop #5

cnsgithub opened this issue Jun 1, 2016 · 0 comments
Labels

Comments

@cnsgithub
Copy link

cnsgithub commented Jun 1, 2016

Hi, Due to errors in IE8 further scripts are not properly processed

Starts here:
Object doesn't support this property or method , L40, C9

Not supporting IE8 is fine, but it should be checked for support ( it doesnt have canvas for instance ) and then the library should stop trying to do things in order to keep the console clean of errors and allow the scripts to continue

Thanks

Suggestion

  var document = window.document;
//insert here
  var elem = document.createElement('canvas');
    if(!!(elem.getContext && elem.getContext('2d')) == false){
        console.warn("No Canvas support stopping radialIndicator.js");
        return;
    }
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants