Fix indentation

Also move flashing into layout.tt.

src/root/build.tt

@@ -63,10 +63,6 @@
 [% END %]
 
 
-[% IF flashMsg %]
-<p class="btn-info btn-large">[% flashMsg %]</p>
-[% END %]
-
         <ul class="nav nav-tabs">
                 <li class="active"><a href="#tabs-summary" data-toggle="tab">Summary</a></li>
                 <li><a href="#tabs-information" data-toggle="tab">Details</a></li>

src/root/layout.tt

@@ -104,6 +104,16 @@
 
     <div class="container">
 
+      [% IF flashMsg %]
+        <br />
+        <p class="btn-info btn-large">[% flashMsg %]</p>
+      [% END %]
+
+      [% IF errorMsg %]
+        <br />
+        <p class="btn-warning btn-large">Error: [% errorMsg %]</p>
+      [% END %]
+
       [% IF !hideHeader %]
         <div class="page-header">
           <h1><small>[% HTML.escape(title) %]</small></h1>

src/root/login.tt

@@ -1,10 +1,6 @@
 [% WRAPPER layout.tt title="Sign in" %]
 [% PROCESS common.tt %]
 
-[% IF errorMsg %]
-<p class="btn-warning btn-large">Error: [% errorMsg %]</p>
-[% END %]
-
 [% IF c.user_exists %]
 <p class="btn-info btn-large">
 You are already logged in as <tt>[% c.user.username %]</tt>.

src/root/queue.tt

@@ -3,10 +3,6 @@
 
 <p><a class="btn" href="[% c.uri_for('/status') %]">Running build steps</a></p>
 
-[% IF flashMsg %]
-<p class="btn-info btn-large">[% flashMsg %]</p>
-[% END %]
-
 [% IF queue.size == 0 %]
 
   <p>The queue is empty.</p>

src/root/topbar.tt

@@ -4,10 +4,6 @@
   </li>
 [% END %]
 
-[% BLOCK makeLink %]
-[% INCLUDE makeLinkWrapped content="" %]
-[% END %]
-
 [% BLOCK makeSubMenu %]
   <li class="dropdown">
       <a class="dropdown-toggle" href="#" data-toggle="dropdown">[% title %]<b class="caret"></b></a>
@@ -17,7 +13,6 @@
   </li>
 [% END %]
 
-
 [% BLOCK makeLinkWrapped %]
   <li [% IF curUri == uri %]class="active"[% END %]>
     <a href="[% uri %]">[% title %]</a>
@@ -29,8 +24,7 @@
 [% INCLUDE makeLinkWrapped content="" %]
 [% END %]
 
-
+<ul class="nav" id="top-menu">
-  <ul class="nav" id="top-menu">
   [% INCLUDE makeLink
      uri = c.uri_for(c.controller('Root').action_for('queue'))
      title = "Queue ("_ nrRunningBuilds _"/"_ nrQueuedBuilds _")" %]
@@ -38,8 +32,6 @@
      uri = c.uri_for(c.controller('Root').action_for('all'))
      title = "All builds" %]
 
-[% IF project || admin %]
-
   [% IF project %]
     [% WRAPPER makeSubMenu title=("Project: " _ project.name) collapsed=(jobset || job)  %]
       [% INCLUDE menuItem uri = c.uri_for(c.controller('Project').action_for('view'), [project.name]) title = "Overview" %]
@@ -62,7 +54,6 @@
     [% END %]
   [% END %]
 
-
   [% IF jobset %]
     [% WRAPPER makeSubMenu title=("Jobset: " _ jobset.name) collapsed=job %]
       [% INCLUDE makeLink
@@ -93,12 +84,9 @@
           [% INCLUDE menuItem uri = c.uri_for('/jobset' project.name jobset.name 'hide') title = "Hide" %]
         [% END %]
       [% END %]
-
-
     [% END %]
   [% END %]
 
-
   [% IF job %]
     [% WRAPPER makeSubMenu title=("Job: " _ job.name) %]
       [% INCLUDE makeLink
@@ -113,15 +101,10 @@
       [% INCLUDE makeLink
         uri = c.uri_for(c.controller('Job').action_for('errors'), [project.name, jobset.name, job.name])
         title = "Errors" %]
-    <li>[% INCLUDE maybeLink uri = c.uri_for(c.controller('Admin').action_for('users')) content = "Manage users" class = "" %]</li>
-    <li>[% INCLUDE maybeLink uri = c.uri_for(c.controller('Admin').action_for('news')) content = "Manage news" class = "" %]</li>
-
     [% END %]
   [% END %]
 
-[% END %]
+  [% IF build %]
-
-[% IF build %]
     [% WRAPPER makeSubMenu title=("Build: " _ build.id) %]
       [% INCLUDE makeLink
         uri = c.uri_for('/build' build.id)
@@ -131,8 +114,8 @@
         [% INCLUDE makeLink
           uri = c.uri_for('/build' build.id 'clone')
           title = "Clone build" %]
-          [% IF !available %]
+        [% IF available %]
-          [% ELSIF build.keep %]
+          [% IF build.keep %]
             [% INCLUDE makeLink
                uri = c.uri_for('/build' build.id 'keep' 0)
                title = "Unkeep build" %]
@@ -141,20 +124,22 @@
                uri = c.uri_for('/build' build.id 'keep' 1)
                title = "Keep build" %]
           [% END %]
+        [% END %]
+        [% IF build.finished %]
           [% INCLUDE makeLink
             uri = c.uri_for('/build' build.id 'restart')
             title = "Restart build" %]
+        [% END %]
+        [% IF !build.finished %]
           [% INCLUDE makeLink
             uri = c.uri_for('/build' build.id 'cancel')
             title = "Cancel build" %]
         [% END %]
-
+      [% END %]
+    [% END %]
   [% END %]
 
-[% END %]
+  [% IF c.user_exists && c.check_user_roles('admin') %]
-
-[% IF c.user_exists %]
-  [% IF c.check_user_roles('admin') %]
     [% WRAPPER makeSubMenu title="Admin" %]
       [% INCLUDE makeLink
         uri = c.uri_for(c.controller('Admin').action_for('index'))
@@ -179,26 +164,18 @@
         uri = c.uri_for(c.controller('Admin').action_for('clear_queue_non_current'))
         content = "Clear all non-running old builds from queue"
         confirmmsg = "Are you sure you want to clear the queue?"
-        class = ""
+        class = "" %]
-      %]
       <li class="divider"></li>
       [% IF c.check_user_roles('admin') %]
         [% INCLUDE menuItem uri = c.uri_for(c.controller('Project').action_for('create')) title = "Create new project" %]
       [% END %]
-
     [% END %]
-[% END %]
-
   [% END %]
 
   [% IF c.user_exists %]
-        [% INCLUDE makeLink
+    [% INCLUDE makeLink uri = c.uri_for(c.controller('Root').action_for('logout')) title = "Sign out" %]
-          uri = c.uri_for(c.controller('Root').action_for('logout'))
-          title = "Sign out" %]
   [% ELSE %]
-        [% INCLUDE makeLink
+    [% INCLUDE makeLink uri = c.uri_for(c.controller('Root').action_for('login')) title = "Sign in" %]
-          uri = c.uri_for(c.controller('Root').action_for('login'))
-          title = "Sign in" %]
   [% END %]
 
-  </ul>
+</ul>