From c542acd28d0a391b697881c7e0bbc63040a3596b Mon Sep 17 00:00:00 2001
From: tof <>
Date: Fri, 24 Aug 2007 13:34:25 +0000
Subject: Fixed #696

---
 HISTORY                                             |  1 +
 .../Javascripts/source/prado/controls/controls.js   | 21 ++++++++++++---------
 2 files changed, 13 insertions(+), 9 deletions(-)

diff --git a/HISTORY b/HISTORY
index fe25768e..e3b57a89 100644
--- a/HISTORY
+++ b/HISTORY
@@ -7,6 +7,7 @@ BUG: Ticket#656 - TDatePicker does not return correct value when in callback mod
 BUG: Ticket#662 - Ensure TForm to properly encode the ampersand in action URL (Qiang)
 BUG: Ticket#666 - TActiveRecord::deleteAll() method always requires a criteria or null parameter (Qiang)
 BUG: Ticket#670 - TDatePicker: Year Issue (Christophe)
+BUG: Ticket#696 - TTabPanel: JS error when one of the tabs is invisible (Qiang,Christophe)
 ENH: Ticket#577 - Added image button support for TPager (Qiang)
 ENH: Ticket#623 - TMemCache to support multiple servers (Carl)
 ENH: Ticket#667 - Added TFeedService.ContentType property (Qiang)
diff --git a/framework/Web/Javascripts/source/prado/controls/controls.js b/framework/Web/Javascripts/source/prado/controls/controls.js
index e0f9218c..e024234c 100644
--- a/framework/Web/Javascripts/source/prado/controls/controls.js
+++ b/framework/Web/Javascripts/source/prado/controls/controls.js
@@ -322,16 +322,19 @@ Prado.WebUI.TTabPanel.prototype =
 		for(var i = 0; i<length; i++)
 		{
 			var item = this.views[i];
-			if(item == viewID)
+			if ($(item))
 			{
-				$(item+'_0').className=this.activeCssClass;
-				$(item).show();
-				this.hiddenField.value=i;
-			}
-			else
-			{
-				$(item+'_0').className=this.normalCssClass;
-				$(item).hide();
+				if(item == viewID)
+				{
+					$(item+'_0').className=this.activeCssClass;
+					$(item).show();
+					this.hiddenField.value=i;
+				}
+				else
+				{
+					$(item+'_0').className=this.normalCssClass;
+					$(item).hide();
+				}
 			}
 		}
 	}
-- 
cgit v1.2.3