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

Mongolian vowel separator causing exceptions. #495

Closed
GirlBossRush opened this issue May 11, 2014 · 4 comments
Closed

Mongolian vowel separator causing exceptions. #495

GirlBossRush opened this issue May 11, 2014 · 4 comments

Comments

@GirlBossRush
Copy link

Not sure why or how this necessary. polymer.js includes a Mongolian vowel separator, causing the following error:

Script contained non-latin characters that were forced to latin. Some characters may be wrong. 
b {impl: script, parentNode_: undefined, firstChild_: undefined, lastChild_: undefined, nextSibling_: undefined…}
 platform.js?body=1:15
Uncaught SyntaxError: Unexpected token ILLEGAL data:text/javascrip…


http://www.fileformat.info/info/unicode/char/180e/index.htm

@dfreedm
Copy link
Member

dfreedm commented May 12, 2014

That character is part of an "isWhitespace" check in the polymer-expressions library: https://github.com/Polymer/polymer-expressions/blob/master/third_party/esprima/esprima.js#L118

@GirlBossRush
Copy link
Author

Can this error be fixed?

@dfreedm
Copy link
Member

dfreedm commented May 12, 2014

I think we can work around it by using the "ascii_only" compression option with uglify.

As a side note, I would recommend you set your server or pages to use UTF-8 encoding to avoid problems like this in the future.

dfreedm added a commit to googlearchive/platform-dev that referenced this issue May 12, 2014
@dfreedm
Copy link
Member

dfreedm commented May 12, 2014

Fixed with googlearchive/polymer-dev@08a5ba2

Next release (probably today) will have a unicode escaped build.

@dfreedm dfreedm closed this as completed May 12, 2014
nevir pushed a commit to nevir/polymer that referenced this issue Oct 31, 2014
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

No branches or pull requests

2 participants