From d27404586264cd515e2865db5e1f7eb0da3366ee Mon Sep 17 00:00:00 2001
From: Sam Rijs <srijs@airpost.net>
Date: Sat, 27 Jan 2018 23:20:11 +1100
Subject: [PATCH] add summary per dependency table

---
 assets/static/style.css         |  6 ++++++
 src/server/views/status_html.rs | 29 +++++++++++++++++++++--------
 2 files changed, 27 insertions(+), 8 deletions(-)

diff --git a/assets/static/style.css b/assets/static/style.css
index 64817b8..1718c72 100644
--- a/assets/static/style.css
+++ b/assets/static/style.css
@@ -43,6 +43,12 @@ h1 code {
     letter-spacing: -0.02em;
 }
 
+h3 span.summary {
+    font-weight: normal;
+    font-style: italic;
+    font-size: .9em;
+}
+
 table {
     width: 100%;
     border: 0;
diff --git a/src/server/views/status_html.rs b/src/server/views/status_html.rs
index 327880f..5964857 100644
--- a/src/server/views/status_html.rs
+++ b/src/server/views/status_html.rs
@@ -1,3 +1,5 @@
+use std::collections::BTreeMap;
+
 use hyper::Response;
 use hyper::header::ContentType;
 use maud::{Markup, html};
@@ -8,8 +10,22 @@ use ::models::repo::RepoPath;
 
 const SELF_BASE_URL: &'static str = "https://shiny-robots.herokuapp.com";
 
-fn dependency_table<I: IntoIterator<Item=(CrateName, AnalyzedDependency)>>(deps: I) -> Markup {
+fn dependency_table(title: &str, deps: BTreeMap<CrateName, AnalyzedDependency>) -> Markup {
+    let count_total = deps.len();
+    let count_outdated = deps.iter().filter(|&(_, dep)| dep.is_outdated()).count();
+
     html! {
+        h3 {
+            (title)
+            span class="summary" {
+                @if count_outdated > 0 {
+                    (format!(" ({} total, {} up-to-date, {} outdated)", count_total, count_total - count_outdated, count_outdated))
+                } @else {
+                    (format!(" ({} total, all up-to-date)", count_total))
+                }
+            }
+        }
+
         table {
             tr {
                 th "Crate"
@@ -62,7 +78,7 @@ pub fn status_html(analysis_outcome: AnalyzeDependenciesOutcome, repo_path: Repo
                             code (format!("{}/{}", repo_path.qual.as_ref(), repo_path.name.as_ref()))
                         }
                     }
-                    
+
                     img src=(format!("/{}/status.svg", self_path));
 
                     pre {
@@ -75,18 +91,15 @@ pub fn status_html(analysis_outcome: AnalyzeDependenciesOutcome, repo_path: Repo
                     }
 
                     @if !analysis_outcome.deps.main.is_empty() {
-                        h3 "Dependencies"
-                        (dependency_table(analysis_outcome.deps.main))
+                        (dependency_table("Dependencies", analysis_outcome.deps.main))
                     }
 
                     @if !analysis_outcome.deps.dev.is_empty() {
-                        h3 "Dev dependencies"
-                        (dependency_table(analysis_outcome.deps.dev))
+                        (dependency_table("Dev dependencies", analysis_outcome.deps.dev))
                     }
 
                     @if !analysis_outcome.deps.build.is_empty() {
-                        h3 "Build dependencies"
-                        (dependency_table(analysis_outcome.deps.build))
+                        (dependency_table("Build dependencies", analysis_outcome.deps.build))
                     }
                 }
             }