Changeset 17906 for lang/javascript/vimperator-plugins/trunk

Timestamp:

08/19/08 21:58:19 (3 months ago)

Author:

anekos

Message:

isScrollableを修正

Files:

• lang/javascript/vimperator-plugins/trunk/scroll_div.js (modified) (5 diffs)

lang/javascript/vimperator-plugins/trunk/scroll_div.js

@@ -3 +3 @@
   let re = /auto|scroll/i;
 
-  // FIXME!!!!!!!!  Firebug では overflow になってるのに false になったりする。
   function isScrollable (e, doc) {
-    let s = doc.defaultView.getComputedStyle(e, null);
-    for each (let n in ['overflow', 'overflow-x', 'overflow-y']) {
-      //if (s[n] && s[n].match(/.+/)) liberator.log(n + ':' + s[n]);
-      if (s[n] && s[n].match(re))
-        return true;
+    try {
+      let s = doc.defaultView.getComputedStyle(e, '');
+      if (e.scrollHeight <= e.clientHeight)
+        return;
+      for each (let n in ['overflow', 'overflowY', 'overflowX']) {
+        if (s[n] && s[n].match(re))
+          return true;
+      }
+    } catch (e) {
+      liberator.log(e);
     }
   }
@@ -44 +48 @@
     }
 
-    liberator.log(result.length);
+    liberator.log('scrollableElements: ' + result.length);
     return result;
   }
@@ -62 +66 @@
     let es = scrollableElements();
     let idx = content.document.__div_scroller_index || 0;
-    liberator.log("idx: " + idx);
     return es[idx];
   }
@@ -71 +74 @@
     if (elem)
       elem.scrollTop += dy;
-    //for each (let elem in scrollableElements())
+    //for each (let elem in scrollableElements()) {
+    //  liberator.log(elem.tagName);
+    //  liberator.log(elem.id);
     //  elem.scrollTop += dy;
+    //}
   }
 
@@ -89 +95 @@
   );
 
+  scroll(100);