Skip to content
Permalink
Browse files

Core: cleaned up $.ui.hasScroll().

  • Loading branch information
scottgonzalez committed Jul 11, 2008
1 parent bb0cac9 commit 9b7ab1621d0b0b0045a785b0f458d142bd98559a
Showing with 11 additions and 3 deletions.
  1. +11 −3 ui/ui.core.js
@@ -53,9 +53,17 @@ $.ui = {
$(el).attr('unselectable', 'off').css('MozUserSelect', '');
},
hasScroll: function(e, a) {
var scroll = /top/.test(a||"top") ? 'scrollTop' : 'scrollLeft', has = false;
if (e[scroll] > 0) return true; e[scroll] = 1;
has = e[scroll] > 0 ? true : false; e[scroll] = 0;
var scroll = (a && a == 'left') ? 'scrollLeft' : 'scrollTop',
has = false;

if (e[scroll] > 0) { return true; }

// TODO: determine which cases actually cause this to happen
// if the element doesn't have the scroll set, see if it's possible to
// set the scroll
e[scroll] = 1;
has = (e[scroll] > 0);
e[scroll] = 0;
return has;
}
};

0 comments on commit 9b7ab16

Please sign in to comment.
You can’t perform that action at this time.