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

Already on GitHub? Sign in to your account

Default Theme + Microsoft's JScript Compiler #64

Closed
yngndrw opened this Issue Jul 28, 2012 · 1 comment

Comments

3 participants

yngndrw commented Jul 28, 2012

Hi,

When using the MVC JavaScript/CSS Bundler I get the following error:

run-time warning JS1010: Expected identifier: default
run-time error JS1137: 'default' is a new reserved word and should not be used as an identifier: default

The problem is that when using $.noty.themes.default, default is a keyword which it doesn't like. Chrome seems perfectly happy with its use however.

The fix is to simply re-name it. I think "noty" would be a suitable name for the default theme.

This also seems to affect IE.

Thanks,

Or write $.noty.themes['default'] = {...

Also there is problem with trailing commas in google closure-compiler

@needim needim closed this Aug 28, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment