From 912e3472945044eea2bc7863b094523981a3bb79 Mon Sep 17 00:00:00 2001
From: Meutel <meutel@meutel.net>
Date: Sun, 28 Jun 2015 20:04:18 +0200
Subject: [PATCH] improve design book, serie

---
 app/assets/stylesheets/custom.css.scss |  4 --
 app/views/books/show.html.erb          | 69 +++++++++++++++++++-------
 app/views/series/show.html.erb         | 30 ++++++++++-
 3 files changed, 80 insertions(+), 23 deletions(-)

diff --git a/app/assets/stylesheets/custom.css.scss b/app/assets/stylesheets/custom.css.scss
index 53f7491..7182061 100644
--- a/app/assets/stylesheets/custom.css.scss
+++ b/app/assets/stylesheets/custom.css.scss
@@ -34,15 +34,11 @@ h1 {
 	font-size: 3em;
 	letter-spacing: -2px;
 	margin-bottom: 30px;
-	text-align: center;
 }
 
 h2 {
-	font-size: 1.2em;
 	letter-spacing: -1px;
 	margin-bottom: 30px;
-	text-align: center;
-	font-weight: normal;
 	color: #777;
 }
 
diff --git a/app/views/books/show.html.erb b/app/views/books/show.html.erb
index f5a10cd..3eecd0f 100644
--- a/app/views/books/show.html.erb
+++ b/app/views/books/show.html.erb
@@ -1,19 +1,54 @@
-<h1><%= @book.title %></h1>
-<ul>
-	<% @book.authors.each do |author| %>
+<div class="page-header">
+	<h1>
+		<span class="glyphicon glyphicon-book"></span>
+		<%= @book.title %>
+	</h1>
+</div>
+<div class="row">
+	<h2>
+		<span class="glyphicon glyphicon-user"></span>
+		<%= "Author".pluralize(@book.authors.count) %>
+	</h2>
+	<ul>
+		<% @book.authors.each do |author| %>
 		<li><%= author.name %></li>
+		<% end %>
+	</ul>
+	<% if @book.serie %>
+	<h2>
+		<span class="glyphicon glyphicon-list"></span>
+		Serie
+	</h2>
+	<div>
+		<%= link_to @book.serie.name, @book.serie %>
+		<span class="badge"><%= @book.series_index.to_i %></span>
+	</div>
 	<% end %>
-</ul>
-<ul>
-	<% @book.languages.each do |language| %>
-		<li><%= language.lang_code %></li>
-	<% end %>
-</ul>
-<ul>
-	<% @book.tags.each do |tag| %>
-		<li><%= tag.name %></li>
-	<% end %>
-</ul>
-<p>Serie: <%= @book.serie.name %></p>
-<p>Publisher: <%= @book.publisher.name %></p>
-<p>Publishing date: <%= @book.pubdate.to_formatted_s(:date) %></p>
+	<h2>
+		<span class="glyphicon glyphicon-globe"></span>
+		<%= "Language".pluralize(@book.languages.count) %>
+	</h2>
+	<div>
+		<% @book.languages.each do |language| %>
+		<%= language.lang_code %>
+		<% end %>
+	</div>
+	<h2>
+		<span class="glyphicon glyphicon-tags"></span>
+		<%= "Tag".pluralize(@book.tags.count) %>
+	</h2>
+	<div>
+		<% @book.tags.each do |tag| %>
+		<span class="label label-info"><%= tag.name %></span>
+		<% end %>
+	</div>
+	<h2>Details</h2>
+	<ul>
+		<% if @book.pubdate %>
+		<li><strong>Publishing date</strong> <%= @book.pubdate.to_formatted_s(:date) %></li>
+		<% end %>
+		<% if @book.publisher %>
+		<li><strong>Publisher</strong> <%= @book.publisher.name %></li>
+		<% end %>
+	</ul>
+</div>
diff --git a/app/views/series/show.html.erb b/app/views/series/show.html.erb
index 2cbc211..e07316b 100644
--- a/app/views/series/show.html.erb
+++ b/app/views/series/show.html.erb
@@ -1,6 +1,32 @@
-<h1><%= @serie.name %></h1>
+<div class="page-header">
+	<h1>
+		<span class="glyphicon glyphicon-list"></span>
+		<%= @serie.name %>
+	</h1>
+</div>
+<h2>
+	<span class="glyphicon glyphicon-book"></span>
+	<%= "Book".pluralize(@serie.books.count) %>
+</h2>
 <ul>
 <% @serie.books.each do |book| %>
-	<li><%= link_to book.title, book %></li>
+	<li><%= book.series_index.to_i %>. <%= link_to book.title, book %></li>
 <% end %>
 </ul>
+<%
+authors = Array.new
+@serie.books.each do |book|
+	authors.concat book.authors
+end
+authors.uniq!.sort! { |x,y| x.sort <=> y.sort }
+if authors %>
+<h2>
+	<span class="glyphicon glyphicon-user"></span>
+	<%= "Author".pluralize(authors.count) %>
+</h2>
+<ul>
+<% authors.each do |author| %>
+	<li><%= link_to author.name, author %></li>
+<% end%>
+</ul>
+<% end %>