From 02e66e48a06800a940ed2d91fc7e3105f2908097 Mon Sep 17 00:00:00 2001
From: Francis Lachapelle <flachapelle@inverse.ca>
Date: Tue, 29 Jun 2010 23:41:24 +0000
Subject: [PATCH] See ChangeLog.

Monotone-Parent: 5b2d034015171bcc6958b829b7479e9040fcef98
Monotone-Revision: cdff5ba452cb3c31c7ea08e23f764471023265a2

Monotone-Author: flachapelle@inverse.ca
Monotone-Date: 2010-06-29T23:41:24
Monotone-Branch: ca.inverse.sogo
---
 ChangeLog                         |  5 +++++
 UI/WebServerResources/MailerUI.js | 16 ++++++++--------
 2 files changed, 13 insertions(+), 8 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 281acf036..819ff5fab 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2010-06-30  Francis Lachapelle  <flachapelle@inverse.ca>
+
+	* UI/WebServerResources/MailerUI.js (onDocumentKeydown): improved
+	scrolling when using the arrows keys.
+
 2010-06-29  Francis Lachapelle  <flachapelle@inverse.ca>
 
 	* UI/WebServerResources/UIxCalendarProperties.js
diff --git a/UI/WebServerResources/MailerUI.js b/UI/WebServerResources/MailerUI.js
index ac23c021f..7bf01f642 100644
--- a/UI/WebServerResources/MailerUI.js
+++ b/UI/WebServerResources/MailerUI.js
@@ -317,18 +317,18 @@ function onDocumentKeydown(event) {
                     row.up().deselectAll();
 					
                     // Adjust the scollbar
-                    var viewPort = $("mailboxContent");
+                    var viewPort = $("mailboxList");
                     var divDimensions = viewPort.getDimensions();
+                    var centerOffset = divDimensions.height/2;
                     var rowScrollOffset = nextRow.cumulativeScrollOffset();
-                    var rowPosition = nextRow.positionedOffset();
-                    var divBottom = divDimensions.height + rowScrollOffset.top;
-                    var rowBottom = rowPosition.top + nextRow.getHeight();
+                     var divBottom = divDimensions.height + rowScrollOffset.top;
+                    var rowBottom = nextRow.offsetTop + nextRow.getHeight();
 
                     if (divBottom < rowBottom)
-                        viewPort.scrollTop += rowBottom - divBottom;
-                    else if (rowScrollOffset.top > rowPosition.top)
-                        viewPort.scrollTop -= rowScrollOffset.top - rowPosition.top;
-					
+                        viewPort.scrollTop += rowBottom - divBottom + centerOffset;
+                     else if (viewPort.scrollTop > nextRow.offsetTop)
+                        viewPort.scrollTop -= rowScrollOffset.top - nextRow.offsetTop + centerOffset;
+		    
                     // Select and load the next message
                     nextRow.selectElement();
                     loadMessage(Mailer.currentMessages[Mailer.currentMailbox]);