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

range.js initialization in IE 8/9 #99

Open
dosena opened this issue Aug 7, 2013 · 0 comments
Open

range.js initialization in IE 8/9 #99

dosena opened this issue Aug 7, 2013 · 0 comments

Comments

@dosena
Copy link

dosena commented Aug 7, 2013

Line 77 in the $.Range constructor is problematic if you've included jquerypp in and there's no DOM yet.

B/c this.win.document.body is null, this.range doesn't get assigned.

If this.range doesn't get assigned, the module setup (anonymous) function chokes at line 642 ( ... fn.compare = range.compareBoundaryPoints ? ...) and bails w/ an exception.

This is especially dangerous if you, like me, happen to have range.js concatenated w/ a bunch of other files (and minified) - those other files' initialization may not run b/c of the exception.

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

1 participant