isVisible should use predicate in selectors

diff --git a/gwtquery-core/src/main/java/com/google/gwt/query/client/impl/DocumentStyleImpl.java b/gwtquery-core/src/main/java/com/google/gwt/query/client/impl/DocumentStyleImpl.java
index 880278cbce6b6554ecb498926b8be82d3908ed65..ac932e9002776294b9f73d63108a46d0ac5d2e3d 100644 (file)
--- a/gwtquery-core/src/main/java/com/google/gwt/query/client/impl/DocumentStyleImpl.java
+++ b/gwtquery-core/src/main/java/com/google/gwt/query/client/impl/DocumentStyleImpl.java
@@ -16,6 +16,7 @@
 package com.google.gwt.query.client.impl;
 
 import static com.google.gwt.query.client.GQuery.$;
+
 import com.google.gwt.dom.client.Document;
 import com.google.gwt.dom.client.Element;
 import com.google.gwt.dom.client.Node;
@@ -211,7 +212,7 @@ public class DocumentStyleImpl {
    * Return whether the element is visible
    */
   public boolean isVisible(Element e) {
-    return !"none".equalsIgnoreCase(curCSS(e, "display", true));
+    return SelectorEngine.filters.get("visible").f(e, 0);
   }
 
   public double num(String val) {

diff --git a/gwtquery-core/src/main/java/com/google/gwt/query/client/impl/SelectorEngine.java b/gwtquery-core/src/main/java/com/google/gwt/query/client/impl/SelectorEngine.java
index beedb287d12e5ab160b44a65504a78884ac8cda0..0d565d0e09c613cd38cae2114ce2eefbb932ee48 100644 (file)
--- a/gwtquery-core/src/main/java/com/google/gwt/query/client/impl/SelectorEngine.java
+++ b/gwtquery-core/src/main/java/com/google/gwt/query/client/impl/SelectorEngine.java
@@ -107,7 +107,8 @@ public class SelectorEngine implements HasSelector {
     filters = JsMap.create();
     filters.put("visible", new Predicate(){
       public boolean f(Element e, int index) {
-        return (e.getOffsetWidth() + e.getOffsetHeight()) > 0 && styleImpl.isVisible(e);
+        return (e.getOffsetWidth() + e.getOffsetHeight()) > 0 &&
+          !"none".equalsIgnoreCase(styleImpl.curCSS(e, "display", true));
       }
     });
     filters.put("hidden", new Predicate() {