From 51da8c1f6a51448cd7f9573b59ae664d054a9fd3 Mon Sep 17 00:00:00 2001
From: gfawcett <gfawcett@6d9bc8c9-1ec2-4278-b937-99fde70a366f>
Date: Mon, 4 May 2009 01:33:07 +0000
Subject: [PATCH] Added explanation-text for Resequence Items

Actually, this had been an earlier feature, but a typo had disabled
it. This version is a bit tighter anyway.

If you click on Resequence Items, a short explanatory message will
appear below the link.


git-svn-id: svn://svn.open-ils.org/ILS-Contrib/servres/trunk@447 6d9bc8c9-1ec2-4278-b937-99fde70a366f
---
 conifer/TODO                              | 13 +++++++------
 conifer/static/menublocks.js              |  5 +++--
 conifer/templates/components/course.xhtml |  4 ++--
 3 files changed, 12 insertions(+), 10 deletions(-)

diff --git a/conifer/TODO b/conifer/TODO
index ce2fa15..0c60e0d 100644
--- a/conifer/TODO
+++ b/conifer/TODO
@@ -4,9 +4,6 @@ IMPORTANT:
 
 * factor out hardcoded references to the EG server.
 
-* People should be able to register themselves into open courses.
-  That is, actually become a member of them.
-
 * Notify circulation of wanted items. What should it do?
 
 * set up a proper issue-tracker?
@@ -42,10 +39,8 @@ FROM KGS:
 
 > Course details -- seems a little terse -- but may be what folks use
 
-> Sequencing -- easy! Save -- maybe make a  button?  [where's my narrative?]
-
 > Notify circulation of wanted items -- FAIL
-
+ 
 > Why is staff a member of a course? is that a requirement? (proxy
 > instructor makes more sense; but how to accomplish that?).
 
@@ -69,3 +64,9 @@ RECENTLY DONE:
 
 * save-sequence fails on IE6.
 
+* People should be able to register themselves into open courses.
+  That is, actually become a member of them.
+
+> Sequencing -- easy! Save -- maybe make a  button?  [where's my narrative?]
+  [I didn't make it a button, but I did add the narrative back.]
+
diff --git a/conifer/static/menublocks.js b/conifer/static/menublocks.js
index 836da7f..a00f57c 100644
--- a/conifer/static/menublocks.js
+++ b/conifer/static/menublocks.js
@@ -59,12 +59,13 @@ function doResequence() {
     if (!resequencing)  {
 	$('.itemtree:nth(0)').sortable({axis:'y'});
 	$('.itemtree:nth(0) > .an_item').addClass('sort_item');
-	$('#resequence_panel').after($('#ropanelmessage'));
+	//$('#resequence_panel').after($('#ropanelmessage'));
+	$('#ropanelmessage').show();
 	$('#resequence_panel a').text($('#i18n-save-order').text());
 	resequencing = true;
     } else {
 	$('.an_item').removeClass('sort_item');
-	$('#ropanelmessage').remove();
+	$('#ropanelmessage').hide();
 	$('#resequence_panel a').text('...');
 	$('.itemtree').sortable('destroy');
 	resequencing = false;
diff --git a/conifer/templates/components/course.xhtml b/conifer/templates/components/course.xhtml
index 0ff52da..bd9ce9c 100644
--- a/conifer/templates/components/course.xhtml
+++ b/conifer/templates/components/course.xhtml
@@ -107,13 +107,13 @@ searchtext = _('search this course...')
    <div py:def="item_resequence_panel()" py:strip="True">
     <div id="resequence_panel" class="little_action_panel">
       <a href="javascript:doResequence();">Resequence items</a>
+      <div id="ropanelmessage" style="clear: right; display: none;"
+	   class="little_action_panel">Drag the items around. Then click Save Sequence, above.</div>
     </div>
     <div style="display: none;">
       <span id="i18n-save-order">Save Sequence</span>
       <span id="i18n-resequence-items">Resequence Items</span>
       <span id="i18n-new-order-saved">The new sequence has been saved.</span>
-      <div id="eropanelmessage" style="clear: right; width: 10em;"
-	   class="little_action_panel">Drag the items around. Then click Save Sequence, above.</div>
     </div>
     </div>
 
-- 
2.11.0