ActionItem601: making sitemap look a little better

git-svn-id: https://svn.colivre.coop.br/svn/noosfero/trunk@2349 3f533792-8f58-4932-b0fe-aaf55b0a4547

ActionItem601: making sitemap look a little better
git-svn-id: https://svn.colivre.coop.br/svn/noosfero/trunk@2349 3f533792-8f58-4932-b0fe-aaf55b0a4547
AntonioTerceiro
1 parent a9e829e6
Showing 4 changed files with 28 additions and 11 deletions Show diff stats
app/helpers/folder_helper.rb
app/models/folder.rb
public/stylesheets/common.css
test/unit/folder_test.rb
 module FolderHelper
   def list_articles(articles, recursive = false)
-    articles.map {|item| display_article_in_listing(item, recursive, 0)}.join('')
+    content_tag(
+      'table',
+      content_tag('tr', content_tag('th', _('Title')) + content_tag('th', _('Last update'))) +
+      articles.map {|item| display_article_in_listing(item, recursive, 0)}.join('')
+    )
   end
   def display_article_in_listing(article, recursive = false, level = 0)
     result = content_tag(
-      'div',
-      link_to(('&nbsp;' * (level * 4) ) + image_tag(icon_for_article(article)) + article.name, article.url),
+      'tr',
+      content_tag('td', link_to(('&nbsp;' * (level * 4) ) + image_tag(icon_for_article(article)) + article.name, article.url))+
+      content_tag('td', show_date(article.updated_at), :class => 'last-update'),
       :class => 'sitemap-item'
     )
     if recursive
@@ -18,8 +18,9 @@ class Folder &lt; Article
   include ActionController::UrlWriter
   include ActionView::Helpers::AssetTagHelper
   include FolderHelper
+  include DatesHelper
   def to_html
-    content_tag('div', body) + tag('hr') + list_articles(children)
+    content_tag('div', body) + tag('hr') + (children.empty? ? content_tag('em', _('(empty folder)')) : list_articles(children))
   end
   def folder?
@@ -275,18 +275,22 @@ div.pending-tasks {
 }
 /* sitemap */
-div.sitemap-item a:link,
-div.sitemap-item a:visited {
+.sitemap-item a:link,
+.sitemap-item a:visited {
   display: block;
   border: none;
   text-decoration: none;
 }
-div.sitemap-item img {
+.sitemap-item img {
   border: none;
 }
-div.sitemap-item a:hover {
-  background-color: #f0f0f0;
+.sitemap-item a:hover {
+  background: #e0e0e0;
   color: red;
+  text-decoration: underline;
+}
+.last-update {
+  font-size: small;
 }
@@ -23,9 +23,16 @@ class FolderTest &lt; ActiveSupport::TestCase
     f = Folder.create!(:profile => p, :name => 'f')
     f.children.create!(:profile => p, :name => 'onearticle')
     f.children.create!(:profile => p, :name => 'otherarticle')
+    f.reload
-    assert_tag_in_string f.to_html, :tag => 'div', :descendant => { :tag => 'a', :attributes => { :href => /.*\/testuser\/f\/onearticle$/ } }, :content => /onearticle/
-    assert_tag_in_string f.to_html, :tag => 'div', :descendant => { :tag => 'a', :attributes => { :href => /.*\/testuser\/f\/otherarticle$/ } }, :content => /otherarticle/
+    assert_tag_in_string f.to_html, :tag => 'td', :descendant => { :tag => 'a', :attributes => { :href => /.*\/testuser\/f\/onearticle$/ } }, :content => /onearticle/
+    assert_tag_in_string f.to_html, :tag => 'td', :descendant => { :tag => 'a', :attributes => { :href => /.*\/testuser\/f\/otherarticle$/ } }, :content => /otherarticle/
+  end
+
+  should 'explictly advise if empty' do
+    p = create_user('testuser').person
+    f = Folder.create!(:profile => p, :name => 'f')
+    assert_tag_in_string f.to_html, :content => '(empty folder)'
   end
   should 'show text body in HTML content' do