From 18416c5773ee1921cb9ab46884fae5eb191271f7 Mon Sep 17 00:00:00 2001
From: manolobevia <manolobevia@440791e1-c2bf-4c1e-ad7c-caf3f8b0ebe8>
Date: Thu, 14 Jul 2011 22:15:17 +0000
Subject: [PATCH] NIHVIVO-2232:  Moved  internal class display section above
 the select list choosing classes in a class group.

---
 webapp/web/css/menupage/testmenupage.css      |  4 ++-
 .../edit/forms/testMenuManagement.ftl         | 31 ++++++++++---------
 2 files changed, 20 insertions(+), 15 deletions(-)

diff --git a/webapp/web/css/menupage/testmenupage.css b/webapp/web/css/menupage/testmenupage.css
index 4e6e235f2..1a1107aa4 100644
--- a/webapp/web/css/menupage/testmenupage.css
+++ b/webapp/web/css/menupage/testmenupage.css
@@ -37,7 +37,6 @@ ul#selectedClasses {
     margin-top: 20px;
 }
 #internal-class {
-    margin-top: 20px;
     margin-bottom: 10px;
     padding: 0.8em 1em;
     background: #f4f4f4;
@@ -53,4 +52,7 @@ ul#selectedClasses {
 }
 ul#selectedClasses li {
     margin-left: .4em;
+}
+#selectClassesMessage {
+   padding-top: 15px;
 }
\ No newline at end of file
diff --git a/webapp/web/templates/freemarker/edit/forms/testMenuManagement.ftl b/webapp/web/templates/freemarker/edit/forms/testMenuManagement.ftl
index 3b820d779..792f72452 100644
--- a/webapp/web/templates/freemarker/edit/forms/testMenuManagement.ftl
+++ b/webapp/web/templates/freemarker/edit/forms/testMenuManagement.ftl
@@ -61,6 +61,23 @@
         <p id="selectClassesMessage" name="selectClassesMessage" ${existingClassGroupStyle}>Select content to display</p>
         <section id="classesInSelectedGroup" name="classesInSelectedGroup" ${existingClassGroupStyle}>
             <ul id="selectedClasses" name="selectedClasses" role="menu">
+                
+                 <section id="internal-class" role="region">
+                        <#if internalClass?has_content>
+                            <#assign enableInternalClass = '' />
+                            <#assign disableClass = 'class="inline"' />
+                        <#else>
+                            <#assign enableInternalClass = '<p class="note">To enable this option, you must first select an <a href="${urls.base}/processInstitutionalInternalClass">institutional internal class</a> for your instance</p>' />
+                            <#assign disableClass = 'class="disable inline" disabled="disabled"' />
+                        </#if>
+
+                        <input type="checkbox" ${disableClass} name="display-internalClass" value="${internalClassUri}" id="display-internalClass" <#if pageInternalOnly?has_content>checked</#if> role="input" />
+                        <label ${disableClass} class="inline" for="display-internalClass">Only display <em>${associatedPage}</em> within my institution</label>
+
+                        ${enableInternalClass}
+                    </section>
+                
+                
                 <#--Adding a default class for "ALL" in case all classes selected-->
                 <li class="ui-state-default" role="menuitem">
                     <input type="checkbox" name="allSelected" id="allSelected" value="all" <#if isClassGroupPage = true || includeAllClasses = true>checked</#if> />
@@ -83,20 +100,6 @@
             </ul>
         </section>
         
-        <section id="internal-class" role="region">
-            <#if internalClass?has_content>
-                <#assign enableInternalClass = '' />
-                <#assign disableClass = 'class="inline"' />
-            <#else>
-                <#assign enableInternalClass = '<p class="note">To enable this option, you must first select an <a href="${urls.base}/processInstitutionalInternalClass">institutional internal class</a> for your instance</p>' />
-                <#assign disableClass = 'class="disable inline" disabled="disabled"' />
-            </#if>
-        
-            <input type="checkbox" ${disableClass} name="display-internalClass" value="${internalClassUri}" id="display-internalClass" <#if pageInternalOnly?has_content>checked</#if> role="input" />
-            <label ${disableClass} class="inline" for="display-internalClass">Only display <em>${associatedPage}</em> within my institution</label>
-        
-            ${enableInternalClass}
-        </section>
         <input type="submit" name="submit-${menuAction}" value="Save changes" class="submit" role="input" /> or <a class="cancel" href="${cancelUrl}">Cancel</a>
 
         <p class="requiredHint">* required fields</p>