From 07decd691574889ab346f40afd47085ceb9b3b5d Mon Sep 17 00:00:00 2001
From: Eelco Dolstra <edolstra@gmail.com>
Date: Tue, 1 Nov 2016 13:13:59 +0100
Subject: [PATCH] Also hide disabled projects on the overview page by default

---
 src/lib/Hydra/Controller/Root.pm |  2 +-
 src/root/overview.tt             | 28 ++++++++++++++++++++++++++--
 src/root/project.tt              |  4 ++--
 3 files changed, 29 insertions(+), 5 deletions(-)

diff --git a/src/lib/Hydra/Controller/Root.pm b/src/lib/Hydra/Controller/Root.pm
index 8de67d1b..4aee4626 100644
--- a/src/lib/Hydra/Controller/Root.pm
+++ b/src/lib/Hydra/Controller/Root.pm
@@ -91,7 +91,7 @@ sub deserialize :ActionClass('Deserialize') { }
 sub index :Path :Args(0) {
     my ($self, $c) = @_;
     $c->stash->{template} = 'overview.tt';
-    $c->stash->{projects} = [$c->model('DB::Projects')->search(isAdmin($c) ? {} : {hidden => 0}, {order_by => 'name'})];
+    $c->stash->{projects} = [$c->model('DB::Projects')->search({}, {order_by => 'name'})];
     $c->stash->{newsItems} = [$c->model('DB::NewsItems')->search({}, { order_by => ['createtime DESC'], rows => 5 })];
     $self->status_ok($c,
         entity => $c->stash->{projects}
diff --git a/src/root/overview.tt b/src/root/overview.tt
index ebc4cea7..0e1a90c1 100644
--- a/src/root/overview.tt
+++ b/src/root/overview.tt
@@ -17,7 +17,31 @@
 
 [% IF projects.size != 0 %]
 
-<p>The following projects are hosted on this server:</p>
+<script type="text/javascript">
+  function showProjects() {
+    var showDisabled = $('#show-disabled').hasClass('active');
+    $('tr.project').map(function() {
+      var hide =
+        ($(this).hasClass('disabled-project') && !showDisabled);
+      if (hide) $(this).hide(); else $(this).show();
+    });
+    return false;
+  };
+
+  $(document).ready(function() {
+    $('#show-disabled').on('click', function(e) {
+      $(this).toggleClass('active');
+      showProjects();
+      return false;
+    });
+
+    showProjects();
+  });
+</script>
+
+<p>The following projects are hosted on this server:
+<label id="show-disabled" class="btn btn-small pull-right" data-toggle="button">Show disabled projects</label>
+</p>
 
 <table class="table table-condensed table-striped clickable-rows">
   <thead>
@@ -29,7 +53,7 @@
   </thead>
   <tbody>
     [% FOREACH p IN projects %]
-    <tr>
+    <tr class="project [% IF !p.enabled %]disabled-project[% END %]">
       <td><span class="[% IF !p.enabled %]disabled-project[% END %] [%+ IF p.hidden %]hidden-project[% END %]">[% INCLUDE renderProjectName project=p.name inRow=1 %]</span></td>
       <td>[% HTML.escape(p.displayname) %]</td>
       <td>[% WRAPPER maybeLink uri=p.homepage %][% HTML.escape(p.description) %][% END %]</td>
diff --git a/src/root/project.tt b/src/root/project.tt
index b48341e4..fd0dd999 100644
--- a/src/root/project.tt
+++ b/src/root/project.tt
@@ -53,9 +53,9 @@
   <div id="tabs-project" class="tab-pane active">
     [% IF project.jobsets %]
       <p>This project has the following jobsets:
-      <button id="show-disabled" type="button" class="btn btn-small pull-right active" data-toggle="button">Show disabled jobsets</button>
+      <label id="show-disabled" class="btn btn-small pull-right active" data-toggle="button">Show disabled jobsets</label>
       [% IF isProjectOwner %]
-        <button id="show-hidden" type="button" class="btn btn-small pull-right" data-toggle="button">Show hidden jobsets</button>
+        <label id="show-hidden" class="btn btn-small pull-right" data-toggle="button">Show hidden jobsets</label>
       [% END %]
       </p>
       [% INCLUDE renderJobsetOverview %]