diff options
| author | onnoj <me@onnoj.net> | 2014-01-02 11:57:21 +0100 |
|---|---|---|
| committer | onnoj <me@onnoj.net> | 2014-01-02 11:57:21 +0100 |
| commit | 76ba593a1cbd2f55c1bb8966929405b4a47a3185 (patch) | |
| tree | f371d780ad9f1d9bc5b11e8cea9c3f83c37d4c23 | |
| parent | e132e7a2b54564ffb710c4fc5a8f29fc0431f847 (diff) | |
Fixed undefined check.
Forgot I had to explicitly check the type of variable type to see if it's undefined.
| -rw-r--r-- | src/library_browser.js | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/library_browser.js b/src/library_browser.js index bf8444bd..029598f3 100644 --- a/src/library_browser.js +++ b/src/library_browser.js @@ -482,12 +482,12 @@ mergeInto(LibraryManager.library, { //Neither .scrollX or .pageXOffset are defined in a spec, but //we prefer .scrollX because it is currently in a spec draft. //(see: http://www.w3.org/TR/2013/WD-cssom-view-20131217/) - var scrollX = ((window.scrollX !== undefined) ? window.scrollX : window.pageXOffset); - var scrollY = ((window.scrollY !== undefined) ? window.scrollY : window.pageYOffset); + var scrollX = ((typeof window.scrollX !== 'undefined') ? window.scrollX : window.pageXOffset); + var scrollY = ((typeof window.scrollY !== 'undefined') ? window.scrollY : window.pageYOffset); #if ASSERTIONS //If this assert lands, it's likely because the browser doesn't support scrollX or pageXOffset //and we have no viable fallback. - assert((scrollX !== undefined) && (scrollY !== undefined), 'Unable to retrieve scroll position, mouse positions likely broken.'); + assert((typeof scrollX !== 'undefined') && (typeof scrollY !== 'undefined'), 'Unable to retrieve scroll position, mouse positions likely broken.'); #endif if (event.type == 'touchstart' || event.type == 'touchend' || |