From 0782c0de4dff7e43fe6ebc8d30aba7196371d136 Mon Sep 17 00:00:00 2001
From: Sam Minnee <sam@silverstripe.com>
Date: Thu, 4 Feb 2010 04:47:18 +0000
Subject: [PATCH] ENHANCEMENT: Improved look and feel for report filtering
 ENHANCEMNT: Added export and print buttons to reports (from r95815)

git-svn-id: svn://svn.silverstripe.com/silverstripe/open/modules/cms/branches/2.4@98173 467b73ca-7a2a-4603-9d3b-597d59a354a9
---
 code/ReportAdmin.php            |  4 ++--
 css/cms_right.css               | 13 +++++++++++++
 javascript/LeftAndMain.js       |  5 +++++
 javascript/ReportAdmin_right.js |  7 +++++--
 4 files changed, 25 insertions(+), 4 deletions(-)

diff --git a/code/ReportAdmin.php b/code/ReportAdmin.php
index 323435b4..b72ccec2 100755
--- a/code/ReportAdmin.php
+++ b/code/ReportAdmin.php
@@ -214,6 +214,7 @@ class ReportAdmin extends LeftAndMain {
 		}
 
 		$form->setFormAction($this->Link() . '/EditForm?' . http_build_query($filteredCriteria));
+		$form->setTemplate('ReportAdminForm');
 		
 		return $form;
 	}
@@ -240,8 +241,7 @@ class ReportAdmin extends LeftAndMain {
 	}
 	
 	public function updatereport() {
-		FormResponse::load_form($this->EditForm()->renderWith('Form'));
-		FormResponse::add("$('tab-Root_Report').onclick();");
+		FormResponse::load_form($this->EditForm()->forTemplate());
 		return FormResponse::respond();
 	}
 }
diff --git a/css/cms_right.css b/css/cms_right.css
index 0ff55828..b4a20294 100644
--- a/css/cms_right.css
+++ b/css/cms_right.css
@@ -233,6 +233,19 @@
 	clear: left;
 }
 
+/**
+ * Non-tabbed scrolling area that can be used in place of tabs in LeftAndMAin
+ */
+.right #ScrollPanel {
+	background: #fff;
+	clear: left;
+	overflow: auto;
+	border: 1px solid #aaa;
+	position: relative;
+	top: 0;
+	padding: 10px;
+}
+
 /**
  * RHS Action Parameters boxes
  */
diff --git a/javascript/LeftAndMain.js b/javascript/LeftAndMain.js
index a71ca9a2..18c540bd 100644
--- a/javascript/LeftAndMain.js
+++ b/javascript/LeftAndMain.js
@@ -156,6 +156,11 @@ window.ontabschanged = function() {
 			fitToParent(_TAB_DIVS_ON_PAGE[i], 30);
 		}
 	}
+	
+	// Non-tab alternative
+	if($('ScrollPanel')) {
+		fitToParent('ScrollPanel', 0);
+	}
 }
 
 window.onresize = function(init) {
diff --git a/javascript/ReportAdmin_right.js b/javascript/ReportAdmin_right.js
index 99bf1172..31a6bbe8 100755
--- a/javascript/ReportAdmin_right.js
+++ b/javascript/ReportAdmin_right.js
@@ -11,6 +11,10 @@ Behaviour.register({
 		 */
 		prepareForm : function() {
 			ajaxActionsAtTop('Form_EditForm', 'form_actions', 'right');
+
+			// Custom code for reports section - link the search button to ajax
+			var updateReportButtonHolder = $('action_updatereport');
+			if(updateReportButtonHolder) prepareAjaxActions(updateReportButtonHolder, 'Form_EditForm');
 		},
 		
 		/**
@@ -107,7 +111,6 @@ Behaviour.register({
 			this.resetElements();
 				
 			window.ontabschanged();
-			
 		}
 	}
-});
+});
\ No newline at end of file