Using var instead let in order to support old browsers

[matheussouza9]

My applications needs to run on Firefox 43 (unfortunaly I can't update it) and it doesn't support let identifier. So, this simple pull request changes it to var.

[coveralls]

Coverage increased (+0.02%) to 90.199% when pulling 16d1c43 on matheussouza9:master into da2a028 on danvk:master.

[danvk]

How are you sourcing dygraphs? The distribution version should use var.

…

On Wed, Jan 30, 2019 at 9:43 AM matheussouza9 ***@***.***> wrote: My applications needs to run on Firefox 43 (unfortunaly I can't update it) and it doesn't support let identifier. So, this simple pull request changes it to var. ------------------------------ You can view, comment on, or merge this pull request online at: #952 Commit Summary - Using var instead let in order to support old browsers File Changes - *M* src/dygraph.js <https://github.com/danvk/dygraphs/pull/952/files#diff-0> (2) Patch Links: - https://github.com/danvk/dygraphs/pull/952.patch - https://github.com/danvk/dygraphs/pull/952.diff — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#952>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAF__TtASAxW20VFCNk5aKYjyBGPzqufks5vIa-AgaJpZM4aaWKG> .

[matheussouza9]

Dygraphs installed from npm. I'm using with Angular 7.
import Dygraph from 'dygraphs'
Importing it like this, it loads dygraph.js from src instead dygraph.min.js from dist.
To solve the problem, I've added the path to the minified file in angular.json scripts section like
"scripts": [ "node_modules/dygraphs/dist/dygraph.min.js" ]
and "importing" (only declaring) in .ts files like this:
declare let Dygraph: any;

[danvk]

There's also an index.es5.js file that you could try importing instead.