From 238707504abfe03102a12c284710b680b72f363b Mon Sep 17 00:00:00 2001
From: Chris McDonough <chrism@plope.com>
Date: Sun, 15 Apr 2012 03:32:13 -0400
Subject: [PATCH] - Make batching in contents view use bootstrap batching UI
 elements.

---
 TODO.txt                             | 11 +++++------
 substanced/sdi/templates/contents.pt | 18 +++++++++---------
 2 files changed, 14 insertions(+), 15 deletions(-)

diff --git a/TODO.txt b/TODO.txt
index 4802866f..faf4c074 100644
--- a/TODO.txt
+++ b/TODO.txt
@@ -5,8 +5,6 @@ UI-Only
 
 - Allow folder contents UI ("Contents" tab) sorting by name and by type.
 
-- Make batching in contents view use bootstrap batching UI elements.
-
 UI-Plus-Guts
 ------------
 
@@ -23,7 +21,7 @@ UI-Plus-Guts
 - Allow cut, copy, paste, rename, (import/export?) from contents view.
 
 - Add Undo tab to Site (or per folder?) which will allow admins to undo recent
-  transactions.
+  transactions.  Implied: add transaction notes to all transactions.
 
 - Allow multiple property sheets on single content object.
 
@@ -40,7 +38,8 @@ Guts-Only
 
 - Create a scaffold.
 
-- Path index CQE syntax.
+- Define path index CQE syntax (currently path indexes can only be queried
+  using query instance syntax).
 
 - Substance D ACLs contain object identifiers which refer to principals.
   Currently, when a principal is deleted, ACLs which refer to the principal
@@ -49,8 +48,8 @@ Guts-Only
   that are grantable to future objects.  As a result, some other principal
   may wind up with an object id that used to belong to a deleted principal,
   and he may inappropriately be granted access due to stale ACLs that
-  possesses that object id.  Some object that is not a principal might wind
-  up with that object id, and undefined things will happen.
+  possesses that object id.  Or some other object that is not a principal
+  might wind up with that object id, and undefined things will happen.
 
   Here's what would be nice: when an ACL is changed or added, using the
   object map, form relationships between the principal objects it names and
diff --git a/substanced/sdi/templates/contents.pt b/substanced/sdi/templates/contents.pt
index 6cc443c3..151dfb78 100644
--- a/substanced/sdi/templates/contents.pt
+++ b/substanced/sdi/templates/contents.pt
@@ -16,15 +16,15 @@
 
 <fieldset>
 
-      <tr id="pagination-buttons" tal:condition="batchinfo['required']">
-        <td colspan="4">
-          <a tal:omit-tag="not batchinfo['first_url']" href="${batchinfo['first_url']}">first</a>
-          <a tal:omit-tag="not batchinfo['prev_url']" href="${batchinfo['prev_url']}">prev</a>
-          <span id="batch-num-disp" >&nbsp;batch ${batchinfo['num']+1} of ${batchinfo['last']+1}&nbsp;</span>
-          <a tal:omit-tag="not batchinfo['next_url']" href="${batchinfo['next_url']}">next</a>
-          <a tal:omit-tag="not batchinfo['last_url']" href="${batchinfo['last_url']}">last</a>
-        </td>
-      </tr>
+      <div class="pagination" tal:condition="batchinfo['required']">
+        <ul>
+          <li tal:attributes="class not batchinfo['first_url'] and 'disabled'" ><a href="${batchinfo['first_url']}">&laquo;</a></li>
+          <li tal:attributes="class not batchinfo['prev_url'] and 'disabled'"><a href="${batchinfo['prev_url']}">&larr;</a></li>
+          <li class="disabled"><a href="#">page ${batchinfo['num']+1} of ${batchinfo['last']+1}</a></li>
+          <li tal:attributes="class not batchinfo['next_url'] and 'disabled'"><a  href="${batchinfo['next_url']}">&rarr;</a></li>
+          <li tal:attributes="class not batchinfo['last_url'] and 'disabled'" ><a href="${batchinfo['last_url']}">&raquo;</a></li>
+        </ul>
+      </div>
 
   <h2 tal:condition="not batchinfo['batch']">No items</h2>