r878 - trunk/ui

r878 - trunk/ui


Author: scott.gonzalez
Date: Fri Nov 7 05:40:01 2008
New Revision: 878
Modified:
trunk/ui/ui.core.js
Log:
Core: Rewrote isVisible(). New implementation inspired by
Remy's :reallyvisible selector.
Modified: trunk/ui/ui.core.js
==============================================================================
--- trunk/ui/ui.core.js    (original)
+++ trunk/ui/ui.core.js    Fri Nov 7 05:40:01 2008
@@ -168,21 +168,10 @@
    
    // TODO: add support for object, area
    tabbable: function(a, i, m) {
-
+        
        var nodeName = a.nodeName.toLowerCase();
        function isVisible(element) {
-            function checkStyles(element) {
-                var style = element.style;
-                return (style.display != 'none' && style.visibility != 'hidden');
-            }
-            
-            var visible = checkStyles(element);
-            
-            (visible && $.each($.dir(element, 'parentNode'), function() {
-                return (visible = checkStyles(this));
-            }));
-            
-            return visible;
+            return !$(element).parents().andSelf().filter(':hidden').length;
        }
        
        return (