diff options
author | Paul Bakaus <paul.bakaus@googlemail.com> | 2008-07-03 14:01:24 +0000 |
---|---|---|
committer | Paul Bakaus <paul.bakaus@googlemail.com> | 2008-07-03 14:01:24 +0000 |
commit | cbda2f35ae18b10d2ca1904af9b2d156dc909b2b (patch) | |
tree | 36230c0de7658626fa0d9b9cd80831bd57448ac6 /ui/ui.droppable.js | |
parent | 9c2e67cff190f9f82db3b3bd25381c0a2a7c3933 (diff) | |
download | jquery-ui-cbda2f35ae18b10d2ca1904af9b2d156dc909b2b.tar.gz jquery-ui-cbda2f35ae18b10d2ca1904af9b2d156dc909b2b.zip |
droppable: a small performance improvement by not using :visible, and deferring to a simpler syntax of checking for diplay none (fixed bool)
Diffstat (limited to 'ui/ui.droppable.js')
-rw-r--r-- | ui/ui.droppable.js | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ui/ui.droppable.js b/ui/ui.droppable.js index 5f208afa3..6d3e159fa 100644 --- a/ui/ui.droppable.js +++ b/ui/ui.droppable.js @@ -179,7 +179,7 @@ $.ui.ddmanager = { for (var i = 0; i < m.length; i++) { if(m[i].options.disabled || (t && !m[i].options.accept.call(m[i].element,(t.currentItem || t.element)))) continue; - m[i].visible = m[i].element.css("display") == "none"; if(!m[i].visible) continue; //If the element is not visible, continue + m[i].visible = m[i].element.css("display") != "none"; if(!m[i].visible) continue; //If the element is not visible, continue m[i].offset = m[i].element.offset(); m[i].proportions = { width: m[i].element[0].offsetWidth, height: m[i].element[0].offsetHeight }; |