cleanup for list_view in base skin

9 files changed, 127 insertions, 49 deletions

diff --git a/share/skin/base/layout/action/link.tt b/share/skin/base/layout/action/link.tt
index 41cee96..bd5013c 100644
--- a/share/skin/base/layout/action/link.tt
+++ b/share/skin/base/layout/action/link.tt
@@ -1,7 +1,5 @@
 =for layout widget
 
-<span class="action_link">
-  <a href="[% uri %]">[% label %]</a>
-</span>
+<a href="[% uri %]">[% label %]</a>
 
 =cut
diff --git a/share/skin/base/layout/collection/grid.tt b/share/skin/base/layout/collection/grid.tt
index bfde10d..4042652 100644
--- a/share/skin/base/layout/collection/grid.tt
+++ b/share/skin/base/layout/collection/grid.tt
@@ -1,26 +1,20 @@
 =for layout widget
 
-<table>
-  [% header %]
-  [% body   %]
-  [% footer %]
-</table>
+[% header %]
+[% body   %]
+[% footer %]
 
 =for layout header
 
-<thead>
-  [% header_row %]
-</thead>
+[% header_row %]
 
 =for layout header_row
 
-<tr>
-  [% header_cells %]
-</tr>
+[% header_cells %]
 
 =for layout header_cell
 
-<th> [% header_cell_contents %] </th>
+[% header_cell_contents %]
 
 =for layout header_cell_contents
 
@@ -28,10 +22,6 @@
 
 =for layout body
 
-<tbody>
-
-  [% members %]
-
-</tbody>
+[% members %]
 
 =cut
diff --git a/share/skin/base/layout/collection/grid/member.tt b/share/skin/base/layout/collection/grid/member.tt
index 9cf24e2..377f97c 100644
--- a/share/skin/base/layout/collection/grid/member.tt
+++ b/share/skin/base/layout/collection/grid/member.tt
@@ -1,11 +1,9 @@
 =for layout widget
 
-<tr>
-  [% field_list %]
-</tr>
+[% field_list %]
 
 =for layout field
 
-<td>[% call_next %]</td>
+[% call_next %]
 
 =cut
diff --git a/share/skin/base/layout/collection/grid/member/with_actions.tt b/share/skin/base/layout/collection/grid/member/with_actions.tt
index a0bca88..e7ba750 100644
--- a/share/skin/base/layout/collection/grid/member/with_actions.tt
+++ b/share/skin/base/layout/collection/grid/member/with_actions.tt
@@ -7,6 +7,6 @@
 
 =for layout action
 
-<td>[% call_next %]</td>
+[% call_next %]
 
 =cut
diff --git a/share/skin/base/layout/list_view.tt b/share/skin/base/layout/list_view.tt
index e1cf3a4..8a0197d 100644
--- a/share/skin/base/layout/list_view.tt
+++ b/share/skin/base/layout/list_view.tt
@@ -12,7 +12,7 @@
 
 =for layout header_action_cell
 
-<th colspan="[% col_count %]"> Actions </th>
+Actions
 
 =for layout header_cell_contents
 
@@ -20,42 +20,34 @@
 
 =for layout actions
 
-<div class="list_view_actions">
-<ul>
-  [% call_next %]
-</ul>
-</div>
+[% call_next %]
 
 =for layout action
 
-<li>[% call_next %]</li>
+[% call_next %]
 
 =for layout pager
 
-<div class="list_view_pager">
-<ul>
-  [% first_page %]
-  [% previous_page %]
-  [% page_list %]
-  [% next_page %]
-  [% last_page %]
-</ul>
-</div>
+[% first_page %]
+[% previous_page %]
+[% page_list %]
+[% next_page %]
+[% last_page %]
 
 =for layout numbered_page_this_page
 
-<li> [% page_number %] </li>
+[% page_number %]
 
 =for layout numbered_page
 
-<li> <a href="[% page_uri %]">[% page_number %]</a> </li>
+<a href="[% page_uri %]">[% page_number %]</a>
 
 =for layout named_page
 
-<li> <a href="[% page_uri %]">[% page_name %]</a> </li>
+<a href="[% page_uri %]">[% page_name %]</a>
 
 =for layout named_page_no_page
 
-<li> [% page_name %] </li>
+[% page_name %]
 
 =cut
diff --git a/share/skin/default/layout/collection/grid.tt b/share/skin/default/layout/collection/grid.tt
new file mode 100644
index 0000000..ecfc7d6
--- /dev/null
+++ b/share/skin/default/layout/collection/grid.tt
@@ -0,0 +1,33 @@
+=extends NEXT
+
+=for layout widget
+
+<table>
+  [% call_next %]
+</table>
+
+=for layout header
+
+<thead>
+  [% call_next %]
+</thead>
+
+=for layout header_row
+
+<tr>
+  [% call_next %]
+</tr>
+
+=for layout header_cell
+
+<th> [% call_next %] </th>
+
+=for layout body
+
+<tbody>
+
+  [% call_next %]
+
+</tbody>
+
+=cut
diff --git a/share/skin/default/layout/collection/grid/member.tt b/share/skin/default/layout/collection/grid/member.tt
new file mode 100644
index 0000000..f0583d7
--- /dev/null
+++ b/share/skin/default/layout/collection/grid/member.tt
@@ -0,0 +1,13 @@
+=extends NEXT
+
+=for layout widget
+
+<tr>
+  [% call_next %]
+</tr>
+
+=for layout field
+
+<td>[% call_next %]</td>
+
+=cut
diff --git a/share/skin/default/layout/collection/grid/member/with_actions.tt b/share/skin/default/layout/collection/grid/member/with_actions.tt
new file mode 100644
index 0000000..a0bca88
--- /dev/null
+++ b/share/skin/default/layout/collection/grid/member/with_actions.tt
@@ -0,0 +1,12 @@
+=extends collection/grid/member
+
+=for layout field_list
+
+[% call_next %]
+[% actions %]
+
+=for layout action
+
+<td>[% call_next %]</td>
+
+=cut
diff --git a/share/skin/default/layout/list_view.tt b/share/skin/default/layout/list_view.tt
index 78ab34f..f1a72b1 100644
--- a/share/skin/default/layout/list_view.tt
+++ b/share/skin/default/layout/list_view.tt
@@ -1,15 +1,57 @@
-=extends NEXT
+=extends collection/grid
+
+=for layout widget
+
+[% pager_fragment %]
+
+[% call_next %]
+
+[% pager_fragment %]
+
+[% actions %]
+
+=for layout header_action_cell
+
+<th colspan="[% col_count %]"> Actions </th>
+
+=for layout header_cell_contents
+
+<a href="[% order_uri %]">[% call_next %]</a>
+
+=for layout actions
+
+<ul class="collection_actions">
+  [% call_next %]
+</ul>
+
+=for layout action
+
+<li>[% call_next %]</li>
 
 =for layout pager
 
-<div class="pager">
-<ul>
+<ul class="pager">
   [% first_page %]
   [% previous_page %]
   [% page_list %]
   [% next_page %]
   [% last_page %]
 </ul>
-</div>
+
+=for layout numbered_page_this_page
+
+<li> [% page_number %] </li>
+
+=for layout numbered_page
+
+<li> <a href="[% page_uri %]">[% page_number %]</a> </li>
+
+=for layout named_page
+
+<li> <a href="[% page_uri %]">[% page_name %]</a> </li>
+
+=for layout named_page_no_page
+
+<li> [% page_name %] </li>
 
 =cut