reproducible debian: breakout package page navigation into seperate template

Decoupling the mainbody and left navigation for package pages allows us to
dry up some of the logic and css between package pages and all other pages that
have a left navigation.

Signed-off-by: Holger Levsen <holger@layer-acht.org>

3 files changed, 46 insertions, 48 deletions

diff --git a/mustache-templates/reproducible/basic_page.mustache b/mustache-templates/reproducible/basic_page.mustache
index f5fe0383..5ba1f8db 100644
--- a/mustache-templates/reproducible/basic_page.mustache
+++ b/mustache-templates/reproducible/basic_page.mustache
@@ -16,11 +16,11 @@
       {{{navigation_html}}}
     </header>
     {{/navigation_html}}
-    {{#navigation_html}}<div class="mainbody">{{/navigation_html}}
+    <div {{#navigation_html}}class="mainbody"{{/navigation_html}}>
       {{#main_header}}
       <h2>{{main_header}}</h2>
       {{/main_header}}
       {{{main_html}}}
-    {{#navigation_html}}</div>{{/navigation_html}}
+    </div>
   </body>
 </html>
diff --git a/mustache-templates/reproducible/package_navigation.mustache b/mustache-templates/reproducible/package_navigation.mustache
new file mode 100644
index 00000000..0d579b7d
--- /dev/null
+++ b/mustache-templates/reproducible/package_navigation.mustache
@@ -0,0 +1,44 @@
+<h2 class="package-name">{{package}}</h2>
+<ul class="menu">
+  <h4>Test Details
+    <a href="/debian/index_notify.html" target="_parent">
+      <span class="notification" title="Notifications for this package are enabled. Every reproducibility related status change will be emailed to the maintainers">{{notify_maintainer}}</span>
+    </a>
+  </h4>
+  {{#notes_uri}}
+  <li><a href="{{notes_uri}}" title="Additional commentary on this packages unreproduciblity" target="main">notes</a></li>
+  {{/notes_uri}}
+  {{^notes_uri}}
+  {{^reproducible}}
+  <li>No notes exist for this package. <a href="https://anonscm.debian.org/git/reproducible/notes.git/tree/README">Add some.</a></li>
+  {{/reproducible}}
+  {{#reproducible}}
+  <li>Good work! This package looks reproducible :)</li>
+  {{/reproducible}}
+  {{/notes_uri}}
+  <li>
+    <a href="{{history}}" target="main">test history</a>
+  </li>
+  <h4>Suite and Architecture</h4>
+  {{{suitearch_section_html}}}
+  <h4>General Package Details</h4>
+  <li>
+    <a href="https://tracker.debian.org/{{package}}" title="Go to: Debian Package Tracking Site">PTS</a>
+    <a href="https://bugs.debian.org/src:{{package}}" title="Go to: Debian Bug Tracking Site">BTS</a>
+  </li>
+  <li>
+    <a href="https://sources.debian.net/src/{{package}}/{{version}}/" title="Go to: the package source!">sources</a>
+    <ul class="children">
+      <li>
+        <div><a href="https://sources.debian.net/src/{{package}}/{{version}}/debian">debian/</a></div>
+        <div><a href="https://sources.debian.net/src/{{package}}/{{version}}/debian/changelog">debian/changelog</a></div>
+        <div><a href="https://sources.debian.net/src/{{package}}/{{version}}/debian/control">debian/control</a></div>
+        <div><a href="https://sources.debian.net/src/{{package}}/{{version}}/debian/rules">debian/rules</a></div>
+      </li>
+    </ul>
+  </li>
+</ul>
+<ul>
+  <li><a href="{{dashboard_url}}">Debian dashboard</a></li>
+</ul>
+{{{project_links_html}}}
diff --git a/mustache-templates/reproducible/package_page.mustache b/mustache-templates/reproducible/package_page.mustache
index 9a78300a..38ca65f8 100644
--- a/mustache-templates/reproducible/package_page.mustache
+++ b/mustache-templates/reproducible/package_page.mustache
@@ -1,49 +1,3 @@
-<header class="head">
-  <h2 class="package-name">{{package}}</h2>
-  <ul class="menu">
-    <h4>Test Details
-      <a href="/debian/index_notify.html" target="_parent">
-        <span class="notification" title="Notifications for this package are enabled. Every reproducibility related status change will be emailed to the maintainers">{{notify_maintainer}}</span>
-      </a>
-    </h4>
-    {{#notes_uri}}
-    <li><a href="{{notes_uri}}" title="Additional commentary on this packages unreproduciblity" target="main">notes</a></li>
-    {{/notes_uri}}
-    {{^notes_uri}}
-    {{^reproducible}}
-    <li>No notes exist for this package. <a href="https://anonscm.debian.org/git/reproducible/notes.git/tree/README">Add some.</a></li>
-    {{/reproducible}}
-    {{#reproducible}}
-    <li>Good work! This package looks reproducible :)</li>
-    {{/reproducible}}
-    {{/notes_uri}}
-    <li>
-      <a href="{{history}}" target="main">test history</a>
-    </li>
-    <h4>Suite and Architecture</h4>
-    {{{suitearch_section_html}}}
-    <h4>General Package Details</h4>
-    <li>
-      <a href="https://tracker.debian.org/{{package}}" title="Go to: Debian Package Tracking Site">PTS</a>
-      <a href="https://bugs.debian.org/src:{{package}}" title="Go to: Debian Bug Tracking Site">BTS</a>
-    </li>
-    <li>
-      <a href="https://sources.debian.net/src/{{package}}/{{version}}/" title="Go to: the package source!">sources</a>
-      <ul class="children">
-        <li>
-          <div><a href="https://sources.debian.net/src/{{package}}/{{version}}/debian">debian/</a></div>
-          <div><a href="https://sources.debian.net/src/{{package}}/{{version}}/debian/changelog">debian/changelog</a></div>
-          <div><a href="https://sources.debian.net/src/{{package}}/{{version}}/debian/control">debian/control</a></div>
-          <div><a href="https://sources.debian.net/src/{{package}}/{{version}}/debian/rules">debian/rules</a></div>
-        </li>
-      </ul>
-    </li>
-  </ul>
-  <ul>
-    <li><a href="{{dashboard_url}}">Debian dashboard</a></li>
-  </ul>
-{{{project_links_html}}}
-</header>
 <iframe id="main" name="main" tabindex="1" src="{{default_view}}">
   <p>
     Your browser does not support iframes.