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

Bootstrap shouldnt add Infinity #14

Closed
pvdz opened this issue Aug 6, 2012 · 2 comments
Closed

Bootstrap shouldnt add Infinity #14

pvdz opened this issue Aug 6, 2012 · 2 comments

Comments

@pvdz
Copy link

pvdz commented Aug 6, 2012

One of the reasons the Xero demo fails is because bootstrap is trying to set Infinity on the iframe global. Since Infinity is a read-only property (not Writable nor Configurable), this fails hard.

Since Infinity is a number value, are there cross frame problems for not setting it explicitly? Else it should probably be purged in bootstrap.js:88, just like stage is deleted.

It should probably not be added in the first place, for reasons above.

@davidaurelio
Copy link
Contributor

  • Bootstrap isn’t to blame here. It just does what the shims tell it to do.
  • There was a fix that fixed it some time ago. Can you verify you run the latest version of bonsai, please?

@pvdz
Copy link
Author

pvdz commented Aug 6, 2012

Probably due to out of date something.

@pvdz pvdz closed this as completed Aug 6, 2012
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