From: Bill Erickson <berick@esilibrary.com>
Date: Fri, 28 Sep 2012 17:58:07 +0000 (-0400)
Subject: ACQ invoice tabs scroll improvements
X-Git-Tag: sprint4-merge-nov22~3717
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=cfe2507fa9a5e67608b189a7b37aa9435550ad8c;p=working%2FEvergreen.git

ACQ invoice tabs scroll improvements

Avoid nested scrollbars on tabbed ACQ invoice UI.  This is done by
removing a ContentPane layer and setting doLayout='false' on the
TabContainer, which allows the individual tabs to control their own
height dynamically, which means no more hard-set 600px height on the
tabs.

Signed-off-by: Bill Erickson <berick@esilibrary.com>
Signed-off-by: Kathy Lussier <klussier@masslnc.org>
---

diff --git a/Open-ILS/src/templates/acq/invoice/view.tt2 b/Open-ILS/src/templates/acq/invoice/view.tt2
index 27f2916ae8..a4a60fb8e4 100644
--- a/Open-ILS/src/templates/acq/invoice/view.tt2
+++ b/Open-ILS/src/templates/acq/invoice/view.tt2
@@ -44,11 +44,13 @@
         </div>
     </div>
 
+    <!--
     <div dojoType="dijit.layout.ContentPane" layoutAlign="client">
+    -->
 
-        <div dojoType="dijit.layout.TabContainer" style="width: 96%; height: 100%;">
+        <div dojoType="dijit.layout.TabContainer" style="width: 96%; height: 100%;" doLayout='false'>
             <div dojoType="dijit.layout.ContentPane" 
-                class='oils-acq-detail-content-pane' title="Invoice" selected='true' style='height:600px'>
+                class='oils-acq-detail-content-pane' title="Invoice" selected='true'>
 
                 <script type='dojo/connect' event='onShow'>
                     // the table is left at display=none on subsequent tab views
@@ -302,7 +304,7 @@
                 </div>
             </div> <!-- tab 2 -->
         </div> <!-- end tabcontainer -->
-    </div> <!-- end contentpane -->
+    <!--</div>--> <!-- end contentpane -->
 </div>
 <div dojoType='openils.widget.ProgressDialog' jsId='progressDialog'></div>
 <div jsId='extraItemsDialog' dojoType="dijit.Dialog" title="Extra Items">