From eb5e8e85a45e8497ffdf5c5df78014e085eff0bf Mon Sep 17 00:00:00 2001
From: "gavin%gavinsharp.com"
 <gavin%gavinsharp.com@18797224-902f-48f8-a5cc-f745e15eee43>
Date: Thu, 15 Sep 2005 23:54:40 +0000
Subject: [PATCH] =?UTF-8?q?Bug=20305730:=20Support=20page=20up/down=20in?=
 =?UTF-8?q?=20richlistbox,=20patch=20by=20Simon=20B=C3=BCnzli=20<zeniko@gm?=
 =?UTF-8?q?ail.com>,=20r=3Dmconnor?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

git-svn-id: svn://10.0.0.236/trunk@180332 18797224-902f-48f8-a5cc-f745e15eee43
---
 .../toolkit/content/widgets/richlistbox.xml   | 66 +++++++++++++++++++
 1 file changed, 66 insertions(+)
diff --git a/mozilla/toolkit/content/widgets/richlistbox.xml b/mozilla/toolkit/content/widgets/richlistbox.xml
index 05dc9d30134..5eb75885988 100644
--- a/mozilla/toolkit/content/widgets/richlistbox.xml
+++ b/mozilla/toolkit/content/widgets/richlistbox.xml
@@ -314,6 +314,70 @@
         </body>
       </method>
 
+      <method name="_isItemVisible">
+        <parameter name="aItem"/>
+        <body>
+          <![CDATA[
+            if (!aItem)
+              return false;
+
+            var y = {};
+            this.scrollBoxObject.getPosition({}, y);
+            y.value += this.scrollBoxObject.y;
+
+            // Partially visible items are also considered visible
+            return (aItem.boxObject.y + aItem.boxObject.height >= y.value) &&
+                   (aItem.boxObject.y < y.value + this.scrollBoxObject.height);
+          ]]>
+        </body>
+      </method>
+
+      <method name="scrollOnePage">
+        <parameter name="aDirection"/> <!-- Must be -1 or 1 -->
+        <body>
+          <![CDATA[
+            if (this.getRowCount() == 0)
+              return false;
+
+            var index = this.selectedIndex;
+
+            // If nothing is selected, we start at the extreme
+            // we're moving away from
+            if (index == -1)
+              index = aDirection == -1 ? this.getRowCount() - 1 : 0;
+
+            // If the selected item is visible, we scroll by one page so that
+            // the newly selected item is at approximately the same position as
+            // the currently selected one
+            var currentItem = this.getItemAtIndex(index);
+            if (this._isItemVisible(currentItem))
+              this.scrollBoxObject.scrollBy(0, this.scrollBoxObject.height * aDirection);
+
+            // Figure out, how many items fully fit into the view port
+            // (measuring from the top of the currently selected one), and
+            // determine the index of the first one lying (partially) outside
+            var height = this.scrollBoxObject.height;
+            while (height >= 0 && index >= 0 && index < this.getRowCount()) {
+              var actualIndex = (index && aDirection == -1) ? index - 1 : index;
+              height -= this.getItemAtIndex(actualIndex).boxObject.height;
+              index += aDirection;
+            }
+            index -= aDirection;
+
+            if (this.selectedItem != index) {
+              this.selectedItem = this.getItemAtIndex(index);
+              return true;
+            }
+
+            // Move by at least one item if the view port is too small
+            if (aDirection == -1)
+              return this.goUp();
+
+            return this.goDown();
+          ]]>
+        </body>
+      </method>
+
       <method name="getItemAtIndex">
         <parameter name="aIndex"/>
         <body>
@@ -394,6 +458,8 @@
     <handlers>
       <handler event="keypress" keycode="VK_UP" action="goUp(); event.preventDefault();"/>
       <handler event="keypress" keycode="VK_DOWN" action="goDown(); event.preventDefault();"/>
+      <handler event="keypress" keycode="VK_PAGE_UP" action="scrollOnePage(-1); event.preventDefault();"/>
+      <handler event="keypress" keycode="VK_PAGE_DOWN" action="scrollOnePage(1); event.preventDefault();"/>
       <handler event="keypress" keycode="VK_HOME" action="clearSelection(); goDown(); event.preventDefault();"/>
       <handler event="keypress" keycode="VK_END" action="clearSelection(); goUp(); event.preventDefault();"/>
 
