From d0ba90c1e4658ff7b22462ad841042161005b4ef Mon Sep 17 00:00:00 2001
From: Qiang Xue <qiang.xue@gmail.com>
Date: Thu, 30 May 2013 15:05:42 -0400
Subject: [PATCH] better auto scrolling.

---
 framework/yii/assets/yii.activeForm.js | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/framework/yii/assets/yii.activeForm.js b/framework/yii/assets/yii.activeForm.js
index fb4c16a..1a2e58d 100644
--- a/framework/yii/assets/yii.activeForm.js
+++ b/framework/yii/assets/yii.activeForm.js
@@ -143,7 +143,11 @@
 					});
 					updateSummary($form, messages);
 					if (errors.length) {
-						$(window).scrollTop($form.find(errors.join(',')).first().offset().top);
+						var top = $form.find(errors.join(',')).first().offset().top;
+						var wtop = $(window).scrollTop();
+						if (top < wtop || top > wtop + $(window).height) {
+							$(window).scrollTop(top);
+						}
 					} else {
 						data.validated = true;
 						var $button = data.submitObject || $form.find(':submit:first');