From 90362949a6a82ef1b7502755854b84742d9bda03 Mon Sep 17 00:00:00 2001
From: Helmut Merz <helmutm@cy55.de>
Date: Sun, 1 Jul 2012 13:09:18 +0200
Subject: [PATCH] improvements of Loopy skin; centralize definition of
 breadcrumbs macro

---
 browser/node_macros.pt       | 21 +++++++++++++++++++++
 browser/skin/lobo/body.pt    | 18 +-----------------
 browser/skin/loopy/body.pt   | 10 ++++++++++
 browser/skin/loopy/loops.css |  5 +++--
 4 files changed, 35 insertions(+), 19 deletions(-)

diff --git a/browser/node_macros.pt b/browser/node_macros.pt
index 72472e9..7f65b19 100644
--- a/browser/node_macros.pt
+++ b/browser/node_macros.pt
@@ -240,6 +240,27 @@
 </metal:actions>
 
 
+<metal:breadcrumbs define-macro="breadcrumbs">
+    <table class="breadcrumbs"
+           tal:define="crumbs view/breadcrumbs"
+           tal:condition="crumbs">
+      <tr>
+        <td style="white-space: nowrap; vertical-align: top; width: 10%"
+            i18n:translate="">You are here:</td>
+        <td>
+          <span>
+            <span style="white-space: nowrap"
+                  tal:repeat="crumb crumbs">
+              <a tal:attributes="href crumb/url"
+                  tal:content="crumb/label" />
+              <tal:delimiter
+                condition="not:repeat/crumb/end"> ></tal:delimiter></span> </span>
+        </td>
+      </tr>
+    </table>
+</metal:breadcrumbs>
+
+
 <metal:actions define-macro="view_modes">
   <ul class="view-modes"
       tal:define="viewModes view/viewModes"
diff --git a/browser/skin/lobo/body.pt b/browser/skin/lobo/body.pt
index 076c015..f1f08c6 100644
--- a/browser/skin/lobo/body.pt
+++ b/browser/skin/lobo/body.pt
@@ -27,23 +27,7 @@
     <div id="content" class="span-6"
          metal:define-macro="content">
       <metal:breadcrumbs define-slot="breadcrumbs">
-        <table class="breadcrumbs"
-               tal:define="crumbs view/breadcrumbs"
-               tal:condition="crumbs">
-          <tr>
-            <td style="white-space: nowrap; vertical-align: top; width: 10%"
-                i18n:translate="">You are here:</td>
-            <td>
-              <span>
-                <span style="white-space: nowrap"
-                      tal:repeat="crumb crumbs">
-                  <a tal:attributes="href crumb/url"
-                      tal:content="crumb/label" />
-                  <tal:delimiter
-                    condition="not:repeat/crumb/end"> ></tal:delimiter></span> </span>
-            </td>
-          </tr>
-        </table>
+        <metal:tabs use-macro="views/node_macros/breadcrumbs" />
       </metal:breadcrumbs>
       <div metal:define-slot="actions"></div>
       <div metal:define-slot="message"></div>
diff --git a/browser/skin/loopy/body.pt b/browser/skin/loopy/body.pt
index e344189..c17c3b6 100644
--- a/browser/skin/loopy/body.pt
+++ b/browser/skin/loopy/body.pt
@@ -9,6 +9,16 @@
             tal:attributes="src string:${resourceBase}logo.png" /></a>
 
 
+    <metal:breadcrumbs fill-slot="breadcrumbs">
+      <metal:tabs use-macro="views/node_macros/breadcrumbs" />
+    </metal:breadcrumbs>
+
+
+    <metal:tabs fill-slot="view_modes">
+      <metal:tabs use-macro="views/node_macros/view_modes" />
+    </metal:tabs>
+
+
     <metal:left fill-slot="portlet-left">
         <tal:portlet repeat="macro controller/macros/portlet_left">
           <metal:portlet use-macro="macro" />
diff --git a/browser/skin/loopy/loops.css b/browser/skin/loopy/loops.css
index 41bedf0..ba7e06c 100644
--- a/browser/skin/loopy/loops.css
+++ b/browser/skin/loopy/loops.css
@@ -6,8 +6,8 @@
 
 /* general */
 
-#content {width: 82%}
-#menu {width: 17%;}
+#content {width: 79%}
+#menu {width: 20%;}
 #sub-section {width: 0}
 
 
@@ -53,6 +53,7 @@ table {
 
 .breadcrumbs td {
     padding-left: 0;
+    padding-top: 12px;
 }
 
 .description {