Improve styling of the per-user notes list view

diff --git a/app/assets/stylesheets/common.css.scss b/app/assets/stylesheets/common.css.scss
index 5fe4ec97bfe183128617c0a6511f4d5a192da6e3..0d12b758f5c2e304d6c735e05e59c14b6d6eca69 100644 (file)
--- a/app/assets/stylesheets/common.css.scss
+++ b/app/assets/stylesheets/common.css.scss
@@ -1642,6 +1642,22 @@ a.button.submit {
   }
 }
 
+/* Rules for the user notes list */
+
+.note_list {
+  tr.creator {
+    background-color: #eeeeee;
+  }
+
+  td {
+    padding: 3px;
+  }
+
+  p {
+    margin-bottom: 0px;
+  }
+}
+
 /* Rules for the notes interface */
 
 .note {

diff --git a/app/controllers/notes_controller.rb b/app/controllers/notes_controller.rb
index d25d9a37e9a756d39dd5a438cb3dbc2858fb7605..11d0f8cb43aaf79aee313004c843c70b1084a08b 100644 (file)
--- a/app/controllers/notes_controller.rb
+++ b/app/controllers/notes_controller.rb
@@ -236,7 +236,7 @@ class NotesController < ApplicationController
       if @this_user = User.active.find_by_display_name(params[:display_name])
         @title =  t 'note.mine.title', :user => @this_user.display_name 
         @heading =  t 'note.mine.heading', :user => @this_user.display_name 
-        @description = t 'note.mine.description', :user => render_to_string(:partial => "user", :object => @this_user)
+        @description = t 'note.mine.subheading', :user => render_to_string(:partial => "user", :object => @this_user)
         @page = (params[:page] || 1).to_i 
         @page_size = 10
         @notes = @this_user.notes.order("updated_at DESC").offset((@page - 1) * @page_size).limit(@page_size).preload(:comments => :author)

diff --git a/app/helpers/note_helper.rb b/app/helpers/note_helper.rb
index bfd7f493dfb8103b9d85b1b9187ce2bb7ec8b162..f2bff861f7209ae4f47504579efbec546430b8c3 100644 (file)
--- a/app/helpers/note_helper.rb
+++ b/app/helpers/note_helper.rb
@@ -8,6 +8,10 @@ module NoteHelper
   end
 
   def note_author(author, link_options = {})
-    link_to h(author.display_name), link_options.merge({:controller => "user", :action => "view", :display_name => author.display_name})
+    if author.nil?
+      ""
+    else
+      link_to h(author.display_name), link_options.merge({:controller => "user", :action => "view", :display_name => author.display_name})
+    end
   end
 end

diff --git a/app/views/notes/mine.html.erb b/app/views/notes/mine.html.erb
index e385e942baad0c42473d0824defa8e0850a47450..d7d8663372a5abbb57f5a0a9abaf1e844c18adbd 100644 (file)
--- a/app/views/notes/mine.html.erb
+++ b/app/views/notes/mine.html.erb
@@ -1,19 +1,21 @@
-<h1><%= @heading %></h1>
-<p><%= raw @description %></p>
+<% content_for :heading do %>
+  <h2><%= @heading %></h2>
+  <p><%= raw @description %></p>
+<% end %>
 
 <%= render :partial => 'notes_paging_nav' %>
 
-<table id="note_list" cellpadding="3">
+<table class="note_list">
   <tr>
     <th></th>
     <th><%= t'note.mine.id' %></th>
-    <th><%= t'changeset.changesets.user' %></th>
-    <th><%= t'changeset.changesets.comment' %></th>
-    <th><%= t'changeset.changesets.saved_at' %></th>
+    <th><%= t'note.mine.creator' %></th>
+    <th><%= t'note.mine.description' %></th>
+    <th><%= t'note.mine.created_at' %></th>
     <th><%= t'note.mine.last_changed' %></th>
   </tr>
 <% @notes.each do |note| -%>
-  <tr<% if note.author != @user2 %> bgcolor="#EEEEEE"<% end %>>
+  <tr<% if note.author != @user2 %> class="creator"<% end %>>
     <td>
       <% if note.status == "closed" %>
         <%= image_tag("closed_note_marker.png", :alt => 'closed') %>
@@ -22,14 +24,10 @@
       <% end %>
     </td>
     <td><%= link_to note.id.to_s, :controller => "browse", :action => "note", :id => note.id %></td>
-    <% if note.author.nil? %> 
-      <td></td> 
-    <% else %> 
-      <td><%= link_to h(note.author.display_name), user_url(:display_name => note.author.display_name) %></td>
-    <% end %>
-    <td> <%= note.comments.first.body.to_html  %> </td>        
-    <td><%= l note.created_at %></td>
-    <td><%= l note.updated_at %></td>
+    <td><%= note_author(note.author) %></td>
+    <td><%= note.comments.first.body.to_html %></td>   
+    <td><%= friendly_date(note.created_at) %></td>
+    <td><%= friendly_date(note.updated_at) %></td>
   </tr>
 <% end -%>
 </table>

diff --git a/config/locales/en.yml b/config/locales/en.yml
index f64b833b06f7488d0272672f84ea68ed3c2fe9c2..217c43306c14ed680b5921bb4ab7a9502d26600e 100644 (file)
--- a/config/locales/en.yml
+++ b/config/locales/en.yml
@@ -1998,8 +1998,11 @@ en:
     mine:
       title: "Notes submitted or commented on by %{user}"
       heading: "%{user}'s notes"
-      description: "Notes submitted or commented on by %{user}"
+      subheading: "Notes submitted or commented on by %{user}"
       id: "Id"
+      creator: "Creator"
+      description: "Description"
+      created_at: "Created at"
       last_changed: "Last changed"
   javascripts:
     map: